Revision: 19504
Author: [email protected]
Date: Thu Feb 20 11:22:33 2014 UTC
Log: A64: Port recent ARM optimizations in
LCodeGen::DoAccessArgumentsAt.
[email protected]
Review URL: https://codereview.chromium.org/172333004
http://code.google.com/p/v8/source/detail?r=19504
Modified:
/branches/bleeding_edge/src/a64/lithium-a64.cc
/branches/bleeding_edge/src/a64/lithium-a64.h
/branches/bleeding_edge/src/a64/lithium-codegen-a64.cc
=======================================
--- /branches/bleeding_edge/src/a64/lithium-a64.cc Wed Feb 19 13:51:49 2014
UTC
+++ /branches/bleeding_edge/src/a64/lithium-a64.cc Thu Feb 20 11:22:33 2014
UTC
@@ -812,12 +812,10 @@
LInstruction* LChunkBuilder::DoAccessArgumentsAt(HAccessArgumentsAt*
instr) {
- // TODO(all): Try to improve this, like ARM r17925.
info()->MarkAsRequiresFrame();
LOperand* args = NULL;
LOperand* length = NULL;
LOperand* index = NULL;
- LOperand* temp = NULL;
if (instr->length()->IsConstant() && instr->index()->IsConstant()) {
args = UseRegisterAtStart(instr->arguments());
@@ -827,11 +825,9 @@
args = UseRegister(instr->arguments());
length = UseRegisterAtStart(instr->length());
index = UseRegisterOrConstantAtStart(instr->index());
- temp = TempRegister();
}
- return DefineAsRegister(
- new(zone()) LAccessArgumentsAt(args, length, index, temp));
+ return DefineAsRegister(new(zone()) LAccessArgumentsAt(args, length,
index));
}
=======================================
--- /branches/bleeding_edge/src/a64/lithium-a64.h Wed Feb 19 13:51:49 2014
UTC
+++ /branches/bleeding_edge/src/a64/lithium-a64.h Thu Feb 20 11:22:33 2014
UTC
@@ -541,16 +541,14 @@
};
-class LAccessArgumentsAt V8_FINAL : public LTemplateInstruction<1, 3, 1> {
+class LAccessArgumentsAt V8_FINAL : public LTemplateInstruction<1, 3, 0> {
public:
LAccessArgumentsAt(LOperand* arguments,
LOperand* length,
- LOperand* index,
- LOperand* temp) {
+ LOperand* index) {
inputs_[0] = arguments;
inputs_[1] = length;
inputs_[2] = index;
- temps_[0] = temp;
}
DECLARE_CONCRETE_INSTRUCTION(AccessArgumentsAt, "access-arguments-at")
@@ -558,7 +556,6 @@
LOperand* arguments() { return inputs_[0]; }
LOperand* length() { return inputs_[1]; }
LOperand* index() { return inputs_[2]; }
- LOperand* temp() { return temps_[0]; }
virtual void PrintDataTo(StringStream* stream) V8_OVERRIDE;
};
=======================================
--- /branches/bleeding_edge/src/a64/lithium-codegen-a64.cc Thu Feb 20
10:08:04 2014 UTC
+++ /branches/bleeding_edge/src/a64/lithium-codegen-a64.cc Thu Feb 20
11:22:33 2014 UTC
@@ -1404,27 +1404,36 @@
void LCodeGen::DoAccessArgumentsAt(LAccessArgumentsAt* instr) {
- // TODO(all): Try to improve this, like ARM r17925.
Register arguments = ToRegister(instr->arguments());
Register result = ToRegister(instr->result());
+ // The pointer to the arguments array come from DoArgumentsElements.
+ // It does not point directly to the arguments and there is an offest of
+ // two words that we must take into account when accessing an argument.
+ // Subtracting the index from length accounts for one, so we add one
more.
+
if (instr->length()->IsConstantOperand() &&
instr->index()->IsConstantOperand()) {
- ASSERT(instr->temp() == NULL);
int index = ToInteger32(LConstantOperand::cast(instr->index()));
int length = ToInteger32(LConstantOperand::cast(instr->length()));
int offset = ((length - index) + 1) * kPointerSize;
__ Ldr(result, MemOperand(arguments, offset));
+ } else if (instr->index()->IsConstantOperand()) {
+ Register length = ToRegister32(instr->length());
+ int index = ToInteger32(LConstantOperand::cast(instr->index()));
+ int loc = index - 1;
+ if (loc != 0) {
+ __ Sub(result.W(), length, loc);
+ __ Ldr(result, MemOperand(arguments, result, UXTW,
kPointerSizeLog2));
+ } else {
+ __ Ldr(result, MemOperand(arguments, length, UXTW,
kPointerSizeLog2));
+ }
} else {
- ASSERT(instr->temp() != NULL);
- Register temp = ToRegister32(instr->temp());
Register length = ToRegister32(instr->length());
Operand index = ToOperand32I(instr->index());
- // There are two words between the frame pointer and the last
arguments.
- // Subtracting from length accounts for only one, so we add one more.
- __ Sub(temp, length, index);
- __ Add(temp, temp, 1);
- __ Ldr(result, MemOperand(arguments, temp, UXTW, kPointerSizeLog2));
+ __ Sub(result.W(), length, index);
+ __ Add(result.W(), result.W(), 1);
+ __ Ldr(result, MemOperand(arguments, result, UXTW, kPointerSizeLog2));
}
}
--
--
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.
