From 3ffc6b5d751a17cbe2d0016650d8f8d6e2bfe402 Mon Sep 17 00:00:00 2001 From: "baptiste.afsa@arm.com" Date: Fri, 21 Feb 2014 15:36:16 +0000 Subject: [PATCH] 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 --- src/a64/code-stubs-a64.cc | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) 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); -- 2.7.4