Reviewers: fschneider,

Description:
Don't take UnkownOSRValues into account when infering Phi's representation.

[email protected]


Please review this at https://chromiumcodereview.appspot.com/9664070/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files:
  M src/hydrogen-instructions.h


Index: src/hydrogen-instructions.h
diff --git a/src/hydrogen-instructions.h b/src/hydrogen-instructions.h
index 53f0de1d9756a545ca24dc82732ed4f4e455014d..9903ff3fa3d85e4f522a72e53ebe61ca22e09f31 100644
--- a/src/hydrogen-instructions.h
+++ b/src/hydrogen-instructions.h
@@ -2266,6 +2266,7 @@ class HPhi: public HValue {
     bool int32_occurred = false;
     for (int i = 0; i < OperandCount(); ++i) {
       HValue* value = OperandAt(i);
+      if (value->IsUnknownOSRValue()) continue;
       if (value->representation().IsDouble()) double_occurred = true;
       if (value->representation().IsInteger32()) int32_occurred = true;
if (value->representation().IsTagged()) return Representation::Tagged();


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to