Reviewers: ,
Message:
Modulo http://code.google.com/p/v8/issues/detail?id=1572, which exists for
me on
bleeding_edge, it passes the test.py.
Description:
InferRange simplifications
* src/hydrogen-instructions.cc (HConstant::InferRange)
(HPhi::InferRange): Remove redundant setting of default values on
range constructors.
BUG=
TEST=tools/test.py --arch=x64 passes
Please review this at http://codereview.chromium.org/7516001/
SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge
Affected files:
M src/hydrogen-instructions.cc
Index: src/hydrogen-instructions.cc
diff --git a/src/hydrogen-instructions.cc b/src/hydrogen-instructions.cc
index
2be2a032574b6bf9af1f8560e19fe95d1694bab7..66de2f58a2bbaa101d5c0b79d669b9e8e616fe22
100644
--- a/src/hydrogen-instructions.cc
+++ b/src/hydrogen-instructions.cc
@@ -880,9 +880,7 @@ Range* HValue::InferRange() {
Range* HConstant::InferRange() {
if (has_int32_value_) {
- Range* result = new Range(int32_value_, int32_value_);
- result->set_can_be_minus_zero(false);
- return result;
+ return new Range(int32_value_, int32_value_);
}
return HValue::InferRange();
}
@@ -891,8 +889,7 @@ Range* HConstant::InferRange() {
Range* HPhi::InferRange() {
if (representation().IsInteger32()) {
if (block()->IsLoopHeader()) {
- Range* range = new Range(kMinInt, kMaxInt);
- return range;
+ return new Range();
} else {
Range* range = OperandAt(0)->range()->Copy();
for (int i = 1; i < OperandCount(); ++i) {
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
