Revision: 19720
Author: [email protected]
Date: Fri Mar 7 12:02:52 2014 UTC
Log: A64 tweaks for division-like operations.
[email protected]
Review URL: https://codereview.chromium.org/190603003
http://code.google.com/p/v8/source/detail?r=19720
Modified:
/branches/bleeding_edge/src/a64/lithium-codegen-a64.cc
/branches/bleeding_edge/src/a64/macro-assembler-a64.cc
=======================================
--- /branches/bleeding_edge/src/a64/lithium-codegen-a64.cc Fri Mar 7
11:44:04 2014 UTC
+++ /branches/bleeding_edge/src/a64/lithium-codegen-a64.cc Fri Mar 7
12:02:52 2014 UTC
@@ -2630,8 +2630,7 @@
HDiv* hdiv = instr->hydrogen();
if (hdiv->CheckFlag(HValue::kBailoutOnMinusZero) &&
hdiv->left()->RangeCanInclude(0) && divisor < 0) {
- __ Cmp(dividend, 0);
- DeoptimizeIf(eq, instr->environment());
+ DeoptimizeIfZero(dividend, instr->environment());
}
__ FlooringDiv(result, dividend, Abs(divisor));
=======================================
--- /branches/bleeding_edge/src/a64/macro-assembler-a64.cc Fri Mar 7
11:44:04 2014 UTC
+++ /branches/bleeding_edge/src/a64/macro-assembler-a64.cc Fri Mar 7
12:02:52 2014 UTC
@@ -4936,18 +4936,15 @@
Register dividend,
int32_t divisor) {
Register tmp = WTmp0();
- ASSERT(!AreAliased(dividend, result, tmp));
+ ASSERT(!AreAliased(result, dividend, tmp));
+ ASSERT(result.Is32Bits() && dividend.Is32Bits());
MultiplierAndShift ms(divisor);
Mov(tmp, Operand(ms.multiplier()));
Smull(result.X(), dividend, tmp);
- Mov(result.X(), Operand(result.X(), ASR, 32));
- if (divisor > 0 && ms.multiplier() < 0) {
- Add(result, result, Operand(dividend));
- }
- if (divisor < 0 && ms.multiplier() > 0) {
- Sub(result, result, Operand(dividend));
- }
- if (ms.shift() > 0) Mov(result, Operand(result, ASR, ms.shift()));
+ Asr(result.X(), result.X(), 32);
+ if (divisor > 0 && ms.multiplier() < 0) Add(result, result, dividend);
+ if (divisor < 0 && ms.multiplier() > 0) Sub(result, result, dividend);
+ if (ms.shift() > 0) Asr(result, result, ms.shift());
Add(result, result, Operand(dividend, LSR, 31));
}
--
--
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/d/optout.
