Generate the TransitionElementsStub using Crankshaft

src/ia32/codegen-ia32.cc

Index: src/ia32/codegen-ia32.cc
diff --git a/src/ia32/codegen-ia32.cc b/src/ia32/codegen-ia32.cc
index 7dc37689a0bb3e204f0e78339e85a83adb90ad35..e7e815d6aa8c5113786dae28f1c01ef8928266d6 100644
--- a/src/ia32/codegen-ia32.cc
+++ b/src/ia32/codegen-ia32.cc
@@ -401,8 +401,8 @@ void ElementsTransitionGenerator::GenerateMapChangeElementsTransition(
   // -----------------------------------
   if (mode == TRACK_ALLOCATION_SITE) {
     ASSERT(allocation_site_info_found != NULL);
-    masm->TestJSArrayForAllocationSiteInfo(edx, edi,
-                                           allocation_site_info_found);
+    masm->TestJSArrayForAllocationSiteInfo(edx, edi);

[Michael Starzinger, 2013/01/31 13:15:10]

I like this change. But use "__" instead of "masm->" here. And in the two
occurrences below.

[danno, 2013/01/31 15:53:25]

Done.

+    __ j(equal, allocation_site_info_found);
   }
 
   // Set transitioned map.
@@ -429,7 +429,8 @@ void ElementsTransitionGenerator::GenerateSmiToDouble(
   Label loop, entry, convert_hole, gc_required, only_change_map;
 
   if (mode == TRACK_ALLOCATION_SITE) {
-    masm->TestJSArrayForAllocationSiteInfo(edx, edi, fail);
+    masm->TestJSArrayForAllocationSiteInfo(edx, edi);
+    __ j(equal, fail);
   }
 
   // Check for empty arrays, which only require a map transition and no changes
@@ -568,7 +569,8 @@ void ElementsTransitionGenerator::GenerateDoubleToObject(
   Label loop, entry, convert_hole, gc_required, only_change_map, success;
 
   if (mode == TRACK_ALLOCATION_SITE) {
-    masm->TestJSArrayForAllocationSiteInfo(edx, edi, fail);
+    masm->TestJSArrayForAllocationSiteInfo(edx, edi);
+    __ j(equal, fail);
   }
 
   // Check for empty arrays, which only require a map transition and no changes