Partial revert of rest parameter desugaring.

Partial revert of rest parameter desugaring.

We'll be able to optimize rest parameters in TurboFan similarly to the arguments array. This CL restores the previous behavior, and a follow-on will enable TurboFan optimization.

(TBR for rossberg since we discussed the revert beforehand. The only changes are a few lines related to tests and rebasing.)

TBR=rossberg@chromium.org
BUG=

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

[mvstanton, 2015-12-22 17:34:52]

Description was changed from

==========
Partial revert of rest parameter desugaring.

BUG=
==========

to

==========
Partial revert of rest parameter desugaring.

We'll be able to optimize rest parameters in TurboFan similarly to the arguments
array. This CL restores the previous behavior, and a follow-on will enable
TurboFan optimization.

BUG=
==========

[mvstanton, 2015-12-22 17:40:48]

mvstanton@chromium.org changed reviewers:
+ bmeurer@chromium.org, neis@chromium.org

[mvstanton, 2015-12-22 17:40:49]

mvstanton@chromium.org changed required reviewers:
+ bmeurer@chromium.org, neis@chromium.org

[mvstanton, 2015-12-22 17:40:49]

Hi guys,
Here is step 1 of the restructuring of rest args. It reverts the
desugaring-style solution (mostly this CL:
https://codereview.chromium.org/1272673003), and a follow-on CL will enable
TurboFan optimizations. Thanks for the look!
--Mike

[caitp (gmail), 2015-12-22 17:57:15]

On 2015/12/22 17:40:49, mvstanton wrote:
> Hi guys,
> Here is step 1 of the restructuring of rest args. It reverts the
> desugaring-style solution (mostly this CL:
> https://codereview.chromium.org/1272673003), and a follow-on CL will enable
> TurboFan optimizations. Thanks for the look!
> --Mike

looks like the lazy parsing issue strikes again

[caitp (gmail), 2015-12-22 18:13:30]

On 2015/12/22 17:57:15, caitp wrote:
> On 2015/12/22 17:40:49, mvstanton wrote:
> > Hi guys,
> > Here is step 1 of the restructuring of rest args. It reverts the
> > desugaring-style solution (mostly this CL:
> > https://codereview.chromium.org/1272673003), and a follow-on CL will enable
> > TurboFan optimizations. Thanks for the look!
> > --Mike
> 
> looks like the lazy parsing issue strikes again

Or I guess, not related to that. that's good news

[Benedikt Meurer, 2015-12-23 04:56:23]

LGTM on the approach as discussed offline yesterday modulo test failures.

[mvstanton, 2015-12-23 12:01:06]

Description was changed from

==========
Partial revert of rest parameter desugaring.

We'll be able to optimize rest parameters in TurboFan similarly to the arguments
array. This CL restores the previous behavior, and a follow-on will enable
TurboFan optimization.

BUG=
==========

to

==========
Partial revert of rest parameter desugaring.

We'll be able to optimize rest parameters in TurboFan similarly to the arguments
array. This CL restores the previous behavior, and a follow-on will enable
TurboFan optimization.

(TBR for rossberg since we discussed the revert beforehand. The only changes are
a few lines related to tests and rebasing.)

TBR=rossberg@chromium.org
BUG=
==========

[mvstanton, 2015-12-23 12:01:06]

mvstanton@chromium.org changed reviewers:
- neis@chromium.org

[mvstanton, 2015-12-23 12:01:07]

mvstanton@chromium.org changed required reviewers:
- neis@chromium.org

[mvstanton, 2015-12-23 12:01:25]

The CQ bit was checked by mvstanton@chromium.org

[mvstanton, 2015-12-23 12:01:25]

The patchset sent to the CQ was uploaded after l-g-t-m from bmeurer@chromium.org
Link to the patchset: https://codereview.chromium.org/1537683002/#ps100001
(title: "Fix test failures.")

[commit-bot: I haz the power, 2015-12-23 12:01:30]

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

[commit-bot: I haz the power, 2015-12-23 12:07:08]

Description was changed from

==========
Partial revert of rest parameter desugaring.

We'll be able to optimize rest parameters in TurboFan similarly to the arguments
array. This CL restores the previous behavior, and a follow-on will enable
TurboFan optimization.

(TBR for rossberg since we discussed the revert beforehand. The only changes are
a few lines related to tests and rebasing.)

TBR=rossberg@chromium.org
BUG=
==========

to

==========
Partial revert of rest parameter desugaring.

We'll be able to optimize rest parameters in TurboFan similarly to the arguments
array. This CL restores the previous behavior, and a follow-on will enable
TurboFan optimization.

(TBR for rossberg since we discussed the revert beforehand. The only changes are
a few lines related to tests and rebasing.)

TBR=rossberg@chromium.org
BUG=
==========

[commit-bot: I haz the power, 2015-12-23 12:07:09]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2015-12-23 12:07:36]

Description was changed from

==========
Partial revert of rest parameter desugaring.

We'll be able to optimize rest parameters in TurboFan similarly to the arguments
array. This CL restores the previous behavior, and a follow-on will enable
TurboFan optimization.

(TBR for rossberg since we discussed the revert beforehand. The only changes are
a few lines related to tests and rebasing.)

TBR=rossberg@chromium.org
BUG=
==========

to

==========
Partial revert of rest parameter desugaring.

We'll be able to optimize rest parameters in TurboFan similarly to the arguments
array. This CL restores the previous behavior, and a follow-on will enable
TurboFan optimization.

(TBR for rossberg since we discussed the revert beforehand. The only changes are
a few lines related to tests and rebasing.)

TBR=rossberg@chromium.org
BUG=

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

[commit-bot: I haz the power, 2015-12-23 12:07:36]

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