[runtime] Introduce FastNewStrictArgumentsStub to optimize strict arguments.

[runtime] Introduce FastNewStrictArgumentsStub to optimize strict arguments.

The FastNewStrictArgumentsStub is very similar to the recently added
FastNewRestParameterStub, it's actually almost a copy of it, except that
it doesn't have the fast case we have for the empty rest parameter. This
patch improves strict arguments in TurboFan and fullcodegen by up to 10x
compared to the previous version.

Also introduce proper JSSloppyArgumentsObject and JSStrictArgumentsObject
for the in-object properties instead of having them as constants in the
Heap class.

Drive-by-fix: Use this stub and the FastNewRestParameterStub in the
interpreter to avoid the runtime call overhead for strict arguments
and rest parameter creation.

R=jarin@chromium.org
TBR=mstarzinger@chromium.org

Committed: https://crrev.com/09d845354742a90fc0596262eb5cbaac169e5ed9
Cr-Commit-Position: refs/heads/master@{#33925}

[Benedikt Meurer, 2016-02-11 17:39:59]

The CQ bit was checked by bmeurer@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-11 17:40:07]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1693513002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1693513002/1

[Benedikt Meurer, 2016-02-11 17:41:46]

Hey Jaro,

Here's the next baby. It's very similar to the FastNewRestParameterStub
(actually more or less a copy with removing the fast case and changing the frame
detection logic).
Please take a look.

Thanks,
Benedikt

[Benedikt Meurer, 2016-02-11 17:46:58]

The CQ bit was checked by bmeurer@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-11 17:47:04]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1693513002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1693513002/20001

[akos.palfi.imgtec, 2016-02-11 17:53:37]

akos.palfi@imgtec.com changed reviewers:
+ akos.palfi@imgtec.com

[akos.palfi.imgtec, 2016-02-11 17:53:38]

MIPS part l-g-t-m, thanks for doing the port and fixing the at/t0 thing! :)

[Benedikt Meurer, 2016-02-11 18:00:25]

On 2016/02/11 17:53:38, akos.palfi.imgtec wrote:
> MIPS part l-g-t-m, thanks for doing the port and fixing the at/t0 thing! :)

Thanks for the pointer. :-)

[commit-bot: I haz the power, 2016-02-11 18:24:18]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-11 18:24:18]

Dry run: This issue passed the CQ dry run.

[Jarin, 2016-02-12 05:07:11]

lgtm, although it would be good to understand why the ArgumentsAccessStub was
not doing its job - it looks like it was doing almost the same thing as your new
stub.

[Benedikt Meurer, 2016-02-12 05:08:41]

Description was changed from

==========
[runtime] Introduce FastNewStrictArgumentsStub to optimize strict arguments.

The FastNewStrictArgumentsStub is very similar to the recently added
FastNewRestParameterStub, it's actually almost a copy of it, except that
it doesn't have the fast case we have for the empty rest parameter. This
patch improves strict arguments in TurboFan and fullcodegen by up to 10x
compared to the previous version.

Also introduce proper JSSloppyArgumentsObject and JSStrictArgumentsObject
for the in-object properties instead of having them as constants in the
Heap class.

Drive-by-fix: Use this stub and the FastNewRestParameterStub in the
interpreter to avoid the runtime call overhead for strict arguments
and rest parameter creation.

R=jarin@chromium.org,mstarzinger@chromium.org
==========

to

==========
[runtime] Introduce FastNewStrictArgumentsStub to optimize strict arguments.

The FastNewStrictArgumentsStub is very similar to the recently added
FastNewRestParameterStub, it's actually almost a copy of it, except that
it doesn't have the fast case we have for the empty rest parameter. This
patch improves strict arguments in TurboFan and fullcodegen by up to 10x
compared to the previous version.

Also introduce proper JSSloppyArgumentsObject and JSStrictArgumentsObject
for the in-object properties instead of having them as constants in the
Heap class.

Drive-by-fix: Use this stub and the FastNewRestParameterStub in the
interpreter to avoid the runtime call overhead for strict arguments
and rest parameter creation.

R=jarin@chromium.org
TBR=mstarzinger@chromium.org
==========

[Benedikt Meurer, 2016-02-12 05:08:48]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-02-12 05:09:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1693513002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1693513002/20001

[commit-bot: I haz the power, 2016-02-12 05:10:45]

Description was changed from

==========
[runtime] Introduce FastNewStrictArgumentsStub to optimize strict arguments.

The FastNewStrictArgumentsStub is very similar to the recently added
FastNewRestParameterStub, it's actually almost a copy of it, except that
it doesn't have the fast case we have for the empty rest parameter. This
patch improves strict arguments in TurboFan and fullcodegen by up to 10x
compared to the previous version.

Also introduce proper JSSloppyArgumentsObject and JSStrictArgumentsObject
for the in-object properties instead of having them as constants in the
Heap class.

Drive-by-fix: Use this stub and the FastNewRestParameterStub in the
interpreter to avoid the runtime call overhead for strict arguments
and rest parameter creation.

R=jarin@chromium.org
TBR=mstarzinger@chromium.org
==========

to

==========
[runtime] Introduce FastNewStrictArgumentsStub to optimize strict arguments.

The FastNewStrictArgumentsStub is very similar to the recently added
FastNewRestParameterStub, it's actually almost a copy of it, except that
it doesn't have the fast case we have for the empty rest parameter. This
patch improves strict arguments in TurboFan and fullcodegen by up to 10x
compared to the previous version.

Also introduce proper JSSloppyArgumentsObject and JSStrictArgumentsObject
for the in-object properties instead of having them as constants in the
Heap class.

Drive-by-fix: Use this stub and the FastNewRestParameterStub in the
interpreter to avoid the runtime call overhead for strict arguments
and rest parameter creation.

R=jarin@chromium.org
TBR=mstarzinger@chromium.org
==========

[commit-bot: I haz the power, 2016-02-12 05:10:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-12 05:11:12]

Description was changed from

==========
[runtime] Introduce FastNewStrictArgumentsStub to optimize strict arguments.

The FastNewStrictArgumentsStub is very similar to the recently added
FastNewRestParameterStub, it's actually almost a copy of it, except that
it doesn't have the fast case we have for the empty rest parameter. This
patch improves strict arguments in TurboFan and fullcodegen by up to 10x
compared to the previous version.

Also introduce proper JSSloppyArgumentsObject and JSStrictArgumentsObject
for the in-object properties instead of having them as constants in the
Heap class.

Drive-by-fix: Use this stub and the FastNewRestParameterStub in the
interpreter to avoid the runtime call overhead for strict arguments
and rest parameter creation.

R=jarin@chromium.org
TBR=mstarzinger@chromium.org
==========

to

==========
[runtime] Introduce FastNewStrictArgumentsStub to optimize strict arguments.

The FastNewStrictArgumentsStub is very similar to the recently added
FastNewRestParameterStub, it's actually almost a copy of it, except that
it doesn't have the fast case we have for the empty rest parameter. This
patch improves strict arguments in TurboFan and fullcodegen by up to 10x
compared to the previous version.

Also introduce proper JSSloppyArgumentsObject and JSStrictArgumentsObject
for the in-object properties instead of having them as constants in the
Heap class.

Drive-by-fix: Use this stub and the FastNewRestParameterStub in the
interpreter to avoid the runtime call overhead for strict arguments
and rest parameter creation.

R=jarin@chromium.org
TBR=mstarzinger@chromium.org

Committed: https://crrev.com/09d845354742a90fc0596262eb5cbaac169e5ed9
Cr-Commit-Position: refs/heads/master@{#33925}
==========

[commit-bot: I haz the power, 2016-02-12 05:11:13]

Patchset 2 (id:??) landed as
https://crrev.com/09d845354742a90fc0596262eb5cbaac169e5ed9
Cr-Commit-Position: refs/heads/master@{#33925}