Revision: 19529
Author: [email protected]
Date: Fri Feb 21 15:36:16 2014 UTC
Log: A64: Replace a Subs by a Sub where flags don't need to be updated.
[email protected]
Review URL: https://codereview.chromium.org/170553002
http://code.google.com/p/v8/source/detail?r=19529
Modified:
/branches/bleeding_edge/src/a64/code-stubs-a64.cc
=======================================
--- /branches/bleeding_edge/src/a64/code-stubs-a64.cc Fri Feb 21 15:34:39
2014 UTC
+++ /branches/bleeding_edge/src/a64/code-stubs-a64.cc Fri Feb 21 15:36:16
2014 UTC
@@ -3638,10 +3638,9 @@
// 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);
--
--
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.
