Separated smi check from HBoundsCheck.

src/code-stubs-hydrogen.cc

Index: src/code-stubs-hydrogen.cc
diff --git a/src/code-stubs-hydrogen.cc b/src/code-stubs-hydrogen.cc
index f837045eed037a9235d5cdac317dd84dde883421..0de0bff630daa1541d1560da49c03936ac9accbb 100644
--- a/src/code-stubs-hydrogen.cc
+++ b/src/code-stubs-hydrogen.cc
@@ -177,7 +177,9 @@ void CodeStubGraphBuilder<TransitionElementsKindStub>::BuildCodeStub() {
   HConstant* max_alloc_size =
       new(zone) HConstant(kMinFreeNewSpaceAfterGC, Representation::Integer32());
   AddInstruction(max_alloc_size);
-  AddInstruction(new(zone) HBoundsCheck(array_length, max_alloc_size));
+  AddInstruction(

[Jakob Kummerow, 2013/02/07 16:17:14]

Please add a comment here:
// Since we're forcing Integer32 representation for this HBoundsCheck,
// there's no need to Smi-check the index.

[Massi, 2013/02/11 10:35:55]

Done.

+      new(zone) HBoundsCheck(array_length, max_alloc_size,
+                             DONT_ALLOW_SMI_KEY, Representation::Integer32()));
 
   current_block()->UpdateEnvironment(new(zone) HEnvironment(zone));