Revision: 6051
Author: [email protected]
Date: Thu Dec 16 06:18:41 2010
Log: Fix an issue discovered in regress-969.

When running with 10 stress runs we get polymorphic stores.  This revealed
an incorrect bailout ID for the state following a polymorphic store.  The ID
should be the internal ID of the assignment side-effect, not the ID of the
end of the expression.

Review URL: http://codereview.chromium.org/5885003
http://code.google.com/p/v8/source/detail?r=6051

Modified:
 /branches/bleeding_edge/src/hydrogen.cc

=======================================
--- /branches/bleeding_edge/src/hydrogen.cc     Thu Dec 16 05:13:36 2010
+++ /branches/bleeding_edge/src/hydrogen.cc     Thu Dec 16 06:18:41 2010
@@ -3257,7 +3257,7 @@
     }

     HBasicBlock* new_exit_block =
-        BuildTypeSwitch(&maps, &subgraphs, object, expr->id());
+        BuildTypeSwitch(&maps, &subgraphs, object, expr->AssignmentId());
     subgraph()->set_exit_block(new_exit_block);
   }

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

Reply via email to