Reviewers: Jakob, fschneider,
Description:
Re-added missing return in HInferRepresentation::TryChange.
It mysteriously disappeared about 800 revisions ago... :-} Thanks to Jakob
for
spotting this!
Please review this at http://codereview.chromium.org/8574036/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files:
M src/hydrogen.cc
Index: src/hydrogen.cc
diff --git a/src/hydrogen.cc b/src/hydrogen.cc
index
4b7528b596f6f02b78cdbcb33d59aa015146a09f..d0380c44b116320118fd8ae6c644dc3de4b596a1
100644
--- a/src/hydrogen.cc
+++ b/src/hydrogen.cc
@@ -1678,7 +1678,7 @@ Representation
HInferRepresentation::TryChange(HValue* value) {
}
// Prefer unboxing over boxing, the latter is more expensive.
- if (tagged_count > non_tagged_count) Representation::None();
+ if (tagged_count > non_tagged_count) return Representation::None();
// Prefer Integer32 over Double, if possible.
if (int32_count > 0 && value->IsConvertibleToInteger()) {
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
