A64: Add support for loading cp from deferred if it's stored in new space

A64: Add support for loading cp from deferred if it's stored in new space

BUG=none
R=ulan@chromium.org, rodolph.perfetta@arm.com
LOG=n
TEST=mjsunit/regress/binop-in-effect-context-deop

Committed: https://code.google.com/p/v8/source/detail?r=19269

[ulan, 2014-02-11 10:49:04]

https://codereview.chromium.org/156733005/diff/1/src/a64/lithium-codegen-a64.cc
File src/a64/lithium-codegen-a64.cc (right):

https://codereview.chromium.org/156733005/diff/1/src/a64/lithium-codegen-a64....
src/a64/lithium-codegen-a64.cc:523: __ LoadHeapObject(cp,
ARM uses "Move(cp.." here.

[jochen (gone - plz use gerrit), 2014-02-11 10:50:31]

https://codereview.chromium.org/156733005/diff/1/src/a64/lithium-codegen-a64.cc
File src/a64/lithium-codegen-a64.cc (right):

https://codereview.chromium.org/156733005/diff/1/src/a64/lithium-codegen-a64....
src/a64/lithium-codegen-a64.cc:523: __ LoadHeapObject(cp,
On 2014/02/11 10:49:04, ulan wrote:
> ARM uses "Move(cp.." here.

Yes, but A64 doesn't have such a fancy move. However, the implementation of
Move(..Handle) and LoadHeapObject is exactly the same.

[jochen (gone - plz use gerrit), 2014-02-11 10:51:16]

On 2014/02/11 10:50:31, jochen wrote:
>
https://codereview.chromium.org/156733005/diff/1/src/a64/lithium-codegen-a64.cc
> File src/a64/lithium-codegen-a64.cc (right):
> 
>
https://codereview.chromium.org/156733005/diff/1/src/a64/lithium-codegen-a64....
> src/a64/lithium-codegen-a64.cc:523: __ LoadHeapObject(cp,
> On 2014/02/11 10:49:04, ulan wrote:
> > ARM uses "Move(cp.." here.
> 
> Yes, but A64 doesn't have such a fancy move. However, the implementation of
> Move(..Handle) and LoadHeapObject is exactly the same.

Actually, it's not exactly the same... updating

[jochen (gone - plz use gerrit), 2014-02-11 10:51:39]

On 2014/02/11 10:51:16, jochen wrote:
> On 2014/02/11 10:50:31, jochen wrote:
> >
>
https://codereview.chromium.org/156733005/diff/1/src/a64/lithium-codegen-a64.cc
> > File src/a64/lithium-codegen-a64.cc (right):
> > 
> >
>
https://codereview.chromium.org/156733005/diff/1/src/a64/lithium-codegen-a64....
> > src/a64/lithium-codegen-a64.cc:523: __ LoadHeapObject(cp,
> > On 2014/02/11 10:49:04, ulan wrote:
> > > ARM uses "Move(cp.." here.
> > 
> > Yes, but A64 doesn't have such a fancy move. However, the implementation of
> > Move(..Handle) and LoadHeapObject is exactly the same.
> 
> Actually, it's not exactly the same... updating

OTOH, the context can never be a SMI, so for the purpose of this method, it's
good enough[tm].

[ulan, 2014-02-11 10:58:19]

lgtm

[Rodolph Perfetta (ARM), 2014-02-11 10:58:47]

lgtm

[jochen (gone - plz use gerrit), 2014-02-11 10:58:52]

Committed patchset #1 manually as r19269 (presubmit successful).