[builtins] Remove bunch of uses of %_Arguments and %_ArgumentsLength.

[builtins] Remove bunch of uses of %_Arguments and %_ArgumentsLength.

There are a bunch of places in our builtins where we use %_Arguments and
%_ArgumentsLength for no good reason, as arguments object and/or rest
parameter is as good and performant in these cases. Now the only uses
of %_Arguments and %_ArgumentsLength left are in string.js, which
requires dedicated investigation.

CQ_INCLUDE_TRYBOTS=tryserver.v8:v8_linux_nosnap_dbg
R=yangguo@chromium.org

Committed: https://crrev.com/2160429fd458e3c095475e718c97f77ac90d906f
Cr-Commit-Position: refs/heads/master@{#33834}

Committed: https://crrev.com/f6db05660ee062964fd1af73383e05fe97295328
Cr-Commit-Position: refs/heads/master@{#33881}

[Benedikt Meurer, 2016-02-09 06:04:55]

Hey Yang,

The awfulness is almost done.
Please take a look.

Thanks,
Benedikt

[Yang, 2016-02-09 06:06:45]

On 2016/02/09 06:04:55, Benedikt Meurer wrote:
> Hey Yang,
> 
> The awfulness is almost done.
> Please take a look.
> 
> Thanks,
> Benedikt

lgtm.

[Benedikt Meurer, 2016-02-09 06:10:01]

The CQ bit was checked by bmeurer@chromium.org

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

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

[commit-bot: I haz the power, 2016-02-09 07:17:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-09 07:18:00]

Description was changed from

==========
[builtins] Remove bunch of uses of %_Arguments and %_ArgumentsLength.

There are a bunch of places in our builtins where we use %_Arguments and
%_ArgumentsLength for no good reason, as arguments object and/or rest
parameter is as good and performant in these cases. Now the only uses
of %_Arguments and %_ArgumentsLength left are in string.js, which
requires dedicated investigation.

R=yangguo@chromium.org
==========

to

==========
[builtins] Remove bunch of uses of %_Arguments and %_ArgumentsLength.

There are a bunch of places in our builtins where we use %_Arguments and
%_ArgumentsLength for no good reason, as arguments object and/or rest
parameter is as good and performant in these cases. Now the only uses
of %_Arguments and %_ArgumentsLength left are in string.js, which
requires dedicated investigation.

R=yangguo@chromium.org

Committed: https://crrev.com/2160429fd458e3c095475e718c97f77ac90d906f
Cr-Commit-Position: refs/heads/master@{#33834}
==========

[commit-bot: I haz the power, 2016-02-09 07:18:01]

Patchset 1 (id:??) landed as
https://crrev.com/2160429fd458e3c095475e718c97f77ac90d906f
Cr-Commit-Position: refs/heads/master@{#33834}

[Benedikt Meurer, 2016-02-09 08:01:59]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1677063005/ by bmeurer@chromium.org.

The reason for reverting is: Breaks tree.

[Benedikt Meurer, 2016-02-09 08:03:14]

Description was changed from

==========
[builtins] Remove bunch of uses of %_Arguments and %_ArgumentsLength.

There are a bunch of places in our builtins where we use %_Arguments and
%_ArgumentsLength for no good reason, as arguments object and/or rest
parameter is as good and performant in these cases. Now the only uses
of %_Arguments and %_ArgumentsLength left are in string.js, which
requires dedicated investigation.

R=yangguo@chromium.org

Committed: https://crrev.com/2160429fd458e3c095475e718c97f77ac90d906f
Cr-Commit-Position: refs/heads/master@{#33834}
==========

to

==========
[builtins] Remove bunch of uses of %_Arguments and %_ArgumentsLength.

There are a bunch of places in our builtins where we use %_Arguments and
%_ArgumentsLength for no good reason, as arguments object and/or rest
parameter is as good and performant in these cases. Now the only uses
of %_Arguments and %_ArgumentsLength left are in string.js, which
requires dedicated investigation.

R=yangguo@chromium.org

Committed: https://crrev.com/2160429fd458e3c095475e718c97f77ac90d906f
Cr-Commit-Position: refs/heads/master@{#33834}
==========

[Benedikt Meurer, 2016-02-09 08:48:44]

Description was changed from

==========
[builtins] Remove bunch of uses of %_Arguments and %_ArgumentsLength.

There are a bunch of places in our builtins where we use %_Arguments and
%_ArgumentsLength for no good reason, as arguments object and/or rest
parameter is as good and performant in these cases. Now the only uses
of %_Arguments and %_ArgumentsLength left are in string.js, which
requires dedicated investigation.

R=yangguo@chromium.org

Committed: https://crrev.com/2160429fd458e3c095475e718c97f77ac90d906f
Cr-Commit-Position: refs/heads/master@{#33834}
==========

to

==========
[builtins] Remove bunch of uses of %_Arguments and %_ArgumentsLength.

There are a bunch of places in our builtins where we use %_Arguments and
%_ArgumentsLength for no good reason, as arguments object and/or rest
parameter is as good and performant in these cases. Now the only uses
of %_Arguments and %_ArgumentsLength left are in string.js, which
requires dedicated investigation.

CQ_INCLUDE_TRYBOTS=tryserver.v8:v8_linux_nosnap_dbg
R=yangguo@chromium.org

Committed: https://crrev.com/2160429fd458e3c095475e718c97f77ac90d906f
Cr-Commit-Position: refs/heads/master@{#33834}
==========

[Benedikt Meurer, 2016-02-09 08:48:49]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2016-02-09 08:48:49]

The patchset sent to the CQ was uploaded after l-g-t-m from yangguo@chromium.org
Link to the patchset: https://codereview.chromium.org/1678953004/#ps20001
(title: "Fix")

[commit-bot: I haz the power, 2016-02-09 08:48:56]

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

[commit-bot: I haz the power, 2016-02-09 10:14:13]

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

[commit-bot: I haz the power, 2016-02-09 10:14:13]

Try jobs failed on following builders:
  v8_linux_nosnap_dbg on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_nosnap_dbg/builds/...)

[Benedikt Meurer, 2016-02-11 06:32:25]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2016-02-11 06:32:25]

The patchset sent to the CQ was uploaded after l-g-t-m from yangguo@chromium.org
Link to the patchset: https://codereview.chromium.org/1678953004/#ps40001
(title: "REBASE. Fix deoptimizer adapted arguments materialization for
builtins.")

[commit-bot: I haz the power, 2016-02-11 06:32:27]

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

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

Description was changed from

==========
[builtins] Remove bunch of uses of %_Arguments and %_ArgumentsLength.

There are a bunch of places in our builtins where we use %_Arguments and
%_ArgumentsLength for no good reason, as arguments object and/or rest
parameter is as good and performant in these cases. Now the only uses
of %_Arguments and %_ArgumentsLength left are in string.js, which
requires dedicated investigation.

CQ_INCLUDE_TRYBOTS=tryserver.v8:v8_linux_nosnap_dbg
R=yangguo@chromium.org

Committed: https://crrev.com/2160429fd458e3c095475e718c97f77ac90d906f
Cr-Commit-Position: refs/heads/master@{#33834}
==========

to

==========
[builtins] Remove bunch of uses of %_Arguments and %_ArgumentsLength.

There are a bunch of places in our builtins where we use %_Arguments and
%_ArgumentsLength for no good reason, as arguments object and/or rest
parameter is as good and performant in these cases. Now the only uses
of %_Arguments and %_ArgumentsLength left are in string.js, which
requires dedicated investigation.

CQ_INCLUDE_TRYBOTS=tryserver.v8:v8_linux_nosnap_dbg
R=yangguo@chromium.org

Committed: https://crrev.com/2160429fd458e3c095475e718c97f77ac90d906f
Cr-Commit-Position: refs/heads/master@{#33834}
==========

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

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-02-11 10:41:06]

Description was changed from

==========
[builtins] Remove bunch of uses of %_Arguments and %_ArgumentsLength.

There are a bunch of places in our builtins where we use %_Arguments and
%_ArgumentsLength for no good reason, as arguments object and/or rest
parameter is as good and performant in these cases. Now the only uses
of %_Arguments and %_ArgumentsLength left are in string.js, which
requires dedicated investigation.

CQ_INCLUDE_TRYBOTS=tryserver.v8:v8_linux_nosnap_dbg
R=yangguo@chromium.org

Committed: https://crrev.com/2160429fd458e3c095475e718c97f77ac90d906f
Cr-Commit-Position: refs/heads/master@{#33834}
==========

to

==========
[builtins] Remove bunch of uses of %_Arguments and %_ArgumentsLength.

There are a bunch of places in our builtins where we use %_Arguments and
%_ArgumentsLength for no good reason, as arguments object and/or rest
parameter is as good and performant in these cases. Now the only uses
of %_Arguments and %_ArgumentsLength left are in string.js, which
requires dedicated investigation.

CQ_INCLUDE_TRYBOTS=tryserver.v8:v8_linux_nosnap_dbg
R=yangguo@chromium.org

Committed: https://crrev.com/2160429fd458e3c095475e718c97f77ac90d906f
Cr-Commit-Position: refs/heads/master@{#33834}

Committed: https://crrev.com/f6db05660ee062964fd1af73383e05fe97295328
Cr-Commit-Position: refs/heads/master@{#33881}
==========

[commit-bot: I haz the power, 2016-02-11 10:41:06]

Patchset 3 (id:??) landed as
https://crrev.com/f6db05660ee062964fd1af73383e05fe97295328
Cr-Commit-Position: refs/heads/master@{#33881}