Reviewers: Jakob,

Message:
PTAL

Description:
Don't explicitly pass the Smi-representation to IfCompare.

BUG=

Please review this at https://chromiumcodereview.appspot.com/15891006/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files:
  M src/hydrogen.cc


Index: src/hydrogen.cc
diff --git a/src/hydrogen.cc b/src/hydrogen.cc
index d0c898afe97dee3384bc453dfb0f33c00a3bfc73..fa7b1b4ff4775e8bdedc27d7bd47fd16fd43af95 100644
--- a/src/hydrogen.cc
+++ b/src/hydrogen.cc
@@ -1128,7 +1128,7 @@ HValue* HGraphBuilder::BuildCheckForCapacityGrow(HValue* object,
   Zone* zone = this->zone();
   IfBuilder length_checker(this);

-  length_checker.IfCompare(length, key, Token::EQ, Representation::Smi());
+  length_checker.IfCompare(length, key, Token::EQ);
   length_checker.Then();

   HValue* current_capacity =
@@ -1136,8 +1136,7 @@ HValue* HGraphBuilder::BuildCheckForCapacityGrow(HValue* object,

   IfBuilder capacity_checker(this);

-  capacity_checker.IfCompare(
-      length, current_capacity, Token::EQ, Representation::Smi());
+  capacity_checker.IfCompare(length, current_capacity, Token::EQ);
   capacity_checker.Then();

   HValue* context = environment()->LookupContext();
@@ -1259,11 +1258,11 @@ HInstruction* HGraphBuilder::BuildUncheckedMonomorphicElementAccess(
           new(zone) HLoadExternalArrayPointer(elements);
       AddInstruction(external_elements);
       IfBuilder length_checker(this);
- length_checker.IfCompare(key, length, Token::LT, Representation::Smi());
+      length_checker.IfCompare(key, length, Token::LT);
       length_checker.Then();
       IfBuilder negative_checker(this);
       HValue* bounds_check = negative_checker.IfCompare(
-          key, graph()->GetConstant0(), Token::GTE, Representation::Smi());
+          key, graph()->GetConstant0(), Token::GTE);
       negative_checker.Then();
       HInstruction* result = BuildExternalArrayElementAccess(
           external_elements, key, val, bounds_check,


--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to