Reviewers: danno,

Description:
When processing collected uint32 instructions skip those with non-Integer32
representation.

[email protected]
BUG=test262 S15.4.4.8_A2_T3.js


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

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 f37902e9ded593582380351da97b03d417badc73..7a3e203beb759bad5cd333b715f4a7ef81285dbe 100644
--- a/src/hydrogen.cc
+++ b/src/hydrogen.cc
@@ -2933,7 +2933,9 @@ void HGraph::ComputeSafeUint32Operations() {
   Uint32Analysis analysis(zone());
   for (int i = 0; i < uint32_instructions_->length(); ++i) {
     HInstruction* current = uint32_instructions_->at(i);
-    if (current->IsLinked()) analysis.Analyze(current);
+    if (current->IsLinked() && current->representation().IsInteger32()) {
+      analysis.Analyze(current);
+    }
   }

   // Some phis might have been optimistically marked with kUint32 flag.


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

Reply via email to