Avoid creating unnecessary branches in Hydrogen

src/hydrogen-instructions.cc

Index: src/hydrogen-instructions.cc
diff --git a/src/hydrogen-instructions.cc b/src/hydrogen-instructions.cc
index 2d4b82bf16ba1efe84bbc0915111921069ad9384..82c002d88c6d6e768495c831109dbadb0fafb9cf 100644
--- a/src/hydrogen-instructions.cc
+++ b/src/hydrogen-instructions.cc
@@ -2744,8 +2744,7 @@ bool HStoreKeyed::NeedsCanonicalization() {
 
 
 #define H_CONSTANT_INT32(val)                                                  \
-new(zone) HConstant(FACTORY->NewNumberFromInt(val, TENURED),                   \
-                    Representation::Integer32())
+new(zone) HConstant(val, Representation::Integer32())
 #define H_CONSTANT_DOUBLE(val)                                                 \
 new(zone) HConstant(FACTORY->NewNumber(val, TENURED),                          \

[Yang, 2013/02/19 14:39:07]

As discussed offline, there is no need to allocate in the double case either.

[Jakob Kummerow, 2013/02/21 08:46:26]

Done.

                     Representation::Double())
@@ -2893,7 +2892,8 @@ HInstruction* HShr::NewHShr(Zone* zone,
         return H_CONSTANT_DOUBLE(
             static_cast<double>(static_cast<uint32_t>(left_val)));
       }
-      return H_CONSTANT_INT32(static_cast<uint32_t>(left_val) >> right_val);
+      return H_CONSTANT_INT32(static_cast<int32_t>(
+          static_cast<uint32_t>(left_val) >> right_val));
     }
   }
   return new(zone) HShr(context, left, right);