Changed inlined property load detection on ARM...

Changed inlined property load detection on ARM

Instaed of having a nop after all non-inlined calls to load IC use a different nop (mov r1, r1 instead of mov r0, r0) to detect an inlined load IC.

Added more infrastructure to the deferred code handling to make it possbile to block constant pool emitting in a deferred code block, including the branch instruction ending the deferred code block.

Addressed a couple of comments to http://codereview.chromium.org/1715003, including adding an assert to make sure that the patching of an ldr instruction is always possible.

Committed: http://code.google.com/p/v8/source/detail?r=4480

[Mads Ager (chromium), 2010-04-23 06:55:40]

LGTM

http://codereview.chromium.org/1758003/diff/11001/12003
File src/arm/assembler-arm.h (right):

http://codereview.chromium.org/1758003/diff/11001/12003#newcode899
src/arm/assembler-arm.h:899: void nop(int type = 0);
Create an enum for the types so they can be named?

http://codereview.chromium.org/1758003/diff/11001/12004
File src/arm/codegen-arm.cc (right):

http://codereview.chromium.org/1758003/diff/11001/12004#newcode5253
src/arm/codegen-arm.cc:5253: // inobject has been inlined..
Extra period at end of line.

[Søren Thygesen Gjesse, 2010-04-23 07:42:16]

http://codereview.chromium.org/1758003/diff/11001/12003
File src/arm/assembler-arm.h (right):

http://codereview.chromium.org/1758003/diff/11001/12003#newcode899
src/arm/assembler-arm.h:899: void nop(int type = 0);
On 2010/04/23 06:55:40, Mads Ager wrote:
> Create an enum for the types so they can be named?

Kept int here, but added an enum to the code generator for use when calling
nop(x).

http://codereview.chromium.org/1758003/diff/11001/12004
File src/arm/codegen-arm.cc (right):

http://codereview.chromium.org/1758003/diff/11001/12004#newcode5253
src/arm/codegen-arm.cc:5253: // inobject has been inlined..
On 2010/04/23 06:55:40, Mads Ager wrote:
> Extra period at end of line.

Done.