Optimistically assume that elements IC only transition once.

test/mjsunit/elements-transition-hoisting.js

Index: test/mjsunit/elements-transition-hoisting.js
diff --git a/test/mjsunit/elements-transition-hoisting.js b/test/mjsunit/elements-transition-hoisting.js
index 50ca2a16e2b453967636e82d094c3584ed029d8d..f70efe4fd7a7994a47af427cb7d4df454445edc0 100644
--- a/test/mjsunit/elements-transition-hoisting.js
+++ b/test/mjsunit/elements-transition-hoisting.js
@@ -58,6 +58,9 @@ if (support_smi_only_arrays) {
   }
 
   testDoubleConversion4(new Array(5));
+  testDoubleConversion4(new Array(5)); // Call twice to make sure that second

[Jakob Kummerow, 2012/06/08 12:18:30]

nit: Two spaces between ; and //, and a full stop at the end.
Consider putting the comment on its own line to save a line break.

More of the same below.

+                                       // store is a transition and not
+                                       // optimistically MONOMORPHIC
   %OptimizeFunctionOnNextCall(testDoubleConversion4);
   testDoubleConversion4(new Array(5));
   testDoubleConversion4(new Array(5));
@@ -80,6 +83,9 @@ if (support_smi_only_arrays) {
   }
 
   testExactMapHoisting(new Array(5));
+  testExactMapHoisting(new Array(5)); // Call twice to make sure that second
+                                      // store is a transition and not
+                                      // optimistically MONOMORPHIC
   %OptimizeFunctionOnNextCall(testExactMapHoisting);
   testExactMapHoisting(new Array(5));
   testExactMapHoisting(new Array(5));
@@ -107,6 +113,9 @@ if (support_smi_only_arrays) {
   }
 
   testExactMapHoisting2(new Array(5));
+  testExactMapHoisting2(new Array(5)); // Call twice to make sure that second
+                                       // store is a transition and not
+                                       // optimistically MONOMORPHIC
   %OptimizeFunctionOnNextCall(testExactMapHoisting2);
   testExactMapHoisting2(new Array(5));
   testExactMapHoisting2(new Array(5));
@@ -132,6 +141,9 @@ if (support_smi_only_arrays) {
   add_transition.foo = 0;
   add_transition[0] = new Object(); // For FAST_ELEMENT transition to be created
   testExactMapHoisting3(new Array(5));
+  testExactMapHoisting3(new Array(5)); // Call twice to make sure that second
+                                       // store is a transition and not
+                                       // optimistically MONOMORPHIC
   %OptimizeFunctionOnNextCall(testExactMapHoisting3);
   testExactMapHoisting3(new Array(5));
   testExactMapHoisting3(new Array(5));
@@ -150,6 +162,10 @@ if (support_smi_only_arrays) {
   }
 
   testDominatingTransitionHoisting1(new Array(5));
+  testDominatingTransitionHoisting1(new Array(5)); // Call twice to make sure
+                                                   // that second store is a
+                                                   // transition and not
+                                                   // optimistically MONOMORPHIC
   %OptimizeFunctionOnNextCall(testDominatingTransitionHoisting1);
   testDominatingTransitionHoisting1(new Array(5));
   testDominatingTransitionHoisting1(new Array(5));
@@ -166,6 +182,9 @@ if (support_smi_only_arrays) {
   }
 
   testHoistingWithSideEffect(new Array(5));
+  testHoistingWithSideEffect(new Array(5)); // Call twice to make sure that
+                                            // second store is a transition and
+                                            // not optimistically MONOMORPHIC
   %OptimizeFunctionOnNextCall(testHoistingWithSideEffect);
   testHoistingWithSideEffect(new Array(5));
   testHoistingWithSideEffect(new Array(5));