From: baptiste.afsa@arm.com <baptiste.afsa@arm.com@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Date: Fri, 21 Feb 2014 15:36:16 +0000 (+0000)
Subject: A64: Replace a Subs by a Sub where flags don't need to be updated.
X-Git-Tag: upstream/4.7.83~10580
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=3ffc6b5d751a17cbe2d0016650d8f8d6e2bfe402;p=platform%2Fupstream%2Fv8.git

A64: Replace a Subs by a Sub where flags don't need to be updated.

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
---

diff --git a/src/a64/code-stubs-a64.cc b/src/a64/code-stubs-a64.cc
index 883f29b..b8f5809 100644
--- a/src/a64/code-stubs-a64.cc
+++ b/src/a64/code-stubs-a64.cc
@@ -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);