Implement inlined object allocation in Crankshaft.

Implement inlined object allocation in Crankshaft.

Generates inlined code for object allocation specific to the initial map
of the given constructor function. Also forces completion of inobject
slack tracking while crankshafting to finalize instance size of these
objects.

R=vegorov@chromium.org
TEST=mjsunit/compiler/alloc-object

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

[Michael Starzinger, 2012-02-09 14:12:12]

Depends on https://chromiumcodereview.appspot.com/9304001/.

[Michael Starzinger, 2012-02-28 11:27:52]

Rebased. Added new patch set.

[Vyacheslav Egorov (Chromium), 2012-02-28 18:00:32]

lgtm with test case that covers all cases (in object/out of object properties,
slack tracking in progress) added.

[Michael Starzinger, 2012-02-29 17:35:08]

Added new patch set.

[Michael Starzinger, 2012-03-01 09:15:54]

Added new patch set.

[Vyacheslav Egorov (Chromium), 2012-03-01 10:34:28]

LGTM with additional test case that tests case when constructor is marked with
ForbidInlineConstructor.

(e.g. when constructor has 256 assignments to this properties).

https://chromiumcodereview.appspot.com/9370019/diff/9001/src/arm/lithium-code...
File src/arm/lithium-codegen-arm.cc (right):

https://chromiumcodereview.appspot.com/9370019/diff/9001/src/arm/lithium-code...
src/arm/lithium-codegen-arm.cc:4381: for (int i = 0; i <
initial_map->inobject_properties(); i++) {
move this loop into the if. it's correct but a bit confusing when it's outside.

https://chromiumcodereview.appspot.com/9370019/diff/9001/src/ia32/lithium-cod...
File src/ia32/lithium-codegen-ia32.cc (right):

https://chromiumcodereview.appspot.com/9370019/diff/9001/src/ia32/lithium-cod...
src/ia32/lithium-codegen-ia32.cc:4268: __ mov(FieldOperand(result,
property_offset), undefined);
I't might be better to load undefined into a register (we have scratch?)

https://chromiumcodereview.appspot.com/9370019/diff/9001/src/x64/lithium-code...
File src/x64/lithium-codegen-x64.cc (right):

https://chromiumcodereview.appspot.com/9370019/diff/9001/src/x64/lithium-code...
src/x64/lithium-codegen-x64.cc:3999: __ Move(FieldOperand(result,
property_offset), undefined);
consider pre loading undefined into a register (it's a huge 64-bit constant).

[Michael Starzinger, 2012-03-01 11:11:24]

Addressed comments. Added new patch set. Rebased. Landed.

https://chromiumcodereview.appspot.com/9370019/diff/9001/src/arm/lithium-code...
File src/arm/lithium-codegen-arm.cc (right):

https://chromiumcodereview.appspot.com/9370019/diff/9001/src/arm/lithium-code...
src/arm/lithium-codegen-arm.cc:4381: for (int i = 0; i <
initial_map->inobject_properties(); i++) {
On 2012/03/01 10:34:28, Vyacheslav Egorov wrote:
> move this loop into the if. it's correct but a bit confusing when it's
outside.

Done.

https://chromiumcodereview.appspot.com/9370019/diff/9001/src/ia32/lithium-cod...
File src/ia32/lithium-codegen-ia32.cc (right):

https://chromiumcodereview.appspot.com/9370019/diff/9001/src/ia32/lithium-cod...
src/ia32/lithium-codegen-ia32.cc:4268: __ mov(FieldOperand(result,
property_offset), undefined);
On 2012/03/01 10:34:28, Vyacheslav Egorov wrote:
> I't might be better to load undefined into a register (we have scratch?)

Done.

https://chromiumcodereview.appspot.com/9370019/diff/9001/src/x64/lithium-code...
File src/x64/lithium-codegen-x64.cc (right):

https://chromiumcodereview.appspot.com/9370019/diff/9001/src/x64/lithium-code...
src/x64/lithium-codegen-x64.cc:3999: __ Move(FieldOperand(result,
property_offset), undefined);
On 2012/03/01 10:34:28, Vyacheslav Egorov wrote:
> consider pre loading undefined into a register (it's a huge 64-bit constant).

Done.