Reviewers: Rodolph Perfetta (ARM), ulan,
Description:
A64: Add support for loading cp from deferred if it's stored in new space
BUG=none
[email protected],[email protected]
LOG=n
TEST=mjsunit/regress/binop-in-effect-context-deop
Please review this at https://codereview.chromium.org/156733005/
SVN Base: https://v8.googlecode.com/svn/branches/experimental/a64
Affected files (+2, -3 lines):
M src/a64/lithium-codegen-a64.cc
Index: src/a64/lithium-codegen-a64.cc
diff --git a/src/a64/lithium-codegen-a64.cc b/src/a64/lithium-codegen-a64.cc
index
64da578e5ccf5667e7f44cf9063dd64781341319..da03ccc214700a949048b590d342a434f84521c2
100644
--- a/src/a64/lithium-codegen-a64.cc
+++ b/src/a64/lithium-codegen-a64.cc
@@ -520,9 +520,8 @@ void LCodeGen::LoadContextFromDeferred(LOperand*
context) {
} else if (context->IsConstantOperand()) {
HConstant* constant =
chunk_->LookupConstant(LConstantOperand::cast(context));
- // TODO(all): on ARM this move can handle object in new space, not in
A64.
- // Check if this can be a problem.
- __ Mov(cp, Operand(Handle<Object>::cast(constant->handle(isolate()))));
+ __ LoadHeapObject(cp,
+
Handle<HeapObject>::cast(constant->handle(isolate())));
} else {
UNREACHABLE();
}
--
--
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.
