A64: Replace a Subs by a Sub where flags don't need to be updated.
authorbaptiste.afsa@arm.com <baptiste.afsa@arm.com@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Fri, 21 Feb 2014 15:36:16 +0000 (15:36 +0000)
committerbaptiste.afsa@arm.com <baptiste.afsa@arm.com@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Fri, 21 Feb 2014 15:36:16 +0000 (15:36 +0000)
R=jochen@chromium.org

Review URL: https://codereview.chromium.org/170553002

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@19529 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

src/a64/code-stubs-a64.cc

index 883f29b..b8f5809 100644 (file)
@@ -3638,10 +3638,9 @@ void ICCompareStub::GenerateSmis(MacroAssembler* masm) {
   // Bail out (to 'miss') unless both x0 and x1 are smis.
   __ JumpIfEitherNotSmi(x0, x1, &miss);
 
-  // TODO(jbramley): Why do we only set the flags for EQ?
   if (GetCondition() == eq) {
     // For equality we do not care about the sign of the result.
-    __ Subs(x0, x0, x1);
+    __ Sub(x0, x0, x1);
   } else {
     // Untag before subtracting to avoid handling overflow.
     __ SmiUntag(x1);