PPC/s390: [turbofan] Avoid allocating rest parameters for spread calls.

PPC/s390: [turbofan] Avoid allocating rest parameters for spread calls.

Port bfa319e5d3f902e504c00af5607117d1eb5d6273

Original Commit Message:

    We already had an optimization to turn Function.prototype.apply with
    arguments object, i.e.

      function foo() { return bar.apply(this, arguments); }

    into a special operator JSCallForwardVarargs, which avoids the
    allocation and deconstruction of the arguments object, but just passes
    along the incoming parameters. We can do the same for rest parameters
    and spread calls/constructs, i.e.

      class A extends B {
        constructor(...args) { super(...args); }
      }

    or

      function foo(...args) { return bar(1, 2, 3, ...args); }

    where we basically pass along the parameters (plus maybe additional
    statically known parameters).

    For this, we introduce a new JSConstructForwardVarargs operator and
    generalize the CallForwardVarargs builtins that are backing this.

R=bmeurer@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com
BUG=v8:6407, v8:6278, v8:6344
LOG=N

Review-Url: https://codereview.chromium.org/2887153004
Cr-Commit-Position: refs/heads/master@{#45402}
Committed: https://chromium.googlesource.com/v8/v8/+/f4da43618c18c0c0f6b3a4b488f822f8d40b04ab

[JaideepBajwa, 2017-05-18 18:43:20]

ptal

[john.yan, 2017-05-18 18:44:23]

lgtm

[JaideepBajwa, 2017-05-18 18:44:45]

The CQ bit was checked by bjaideep@ca.ibm.com

[commit-bot: I haz the power, 2017-05-18 18:44:55]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-18 19:21:04]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1495133085566140,
"parent_rev": "72edb7542562ffd6f50957681ecbc2f6fcd07f7a", "commit_rev":
"f4da43618c18c0c0f6b3a4b488f822f8d40b04ab"}

[commit-bot: I haz the power, 2017-05-18 19:21:12]

Description was changed from

==========
PPC/s390: [turbofan] Avoid allocating rest parameters for spread calls.

Port bfa319e5d3f902e504c00af5607117d1eb5d6273

Original Commit Message:

    We already had an optimization to turn Function.prototype.apply with
    arguments object, i.e.

      function foo() { return bar.apply(this, arguments); }

    into a special operator JSCallForwardVarargs, which avoids the
    allocation and deconstruction of the arguments object, but just passes
    along the incoming parameters. We can do the same for rest parameters
    and spread calls/constructs, i.e.

      class A extends B {
        constructor(...args) { super(...args); }
      }

    or

      function foo(...args) { return bar(1, 2, 3, ...args); }

    where we basically pass along the parameters (plus maybe additional
    statically known parameters).

    For this, we introduce a new JSConstructForwardVarargs operator and
    generalize the CallForwardVarargs builtins that are backing this.

R=bmeurer@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com,
michael_dawson@ca.ibm.com
BUG=v8:6407,v8:6278,v8:6344
LOG=N
==========

to

==========
PPC/s390: [turbofan] Avoid allocating rest parameters for spread calls.

Port bfa319e5d3f902e504c00af5607117d1eb5d6273

Original Commit Message:

    We already had an optimization to turn Function.prototype.apply with
    arguments object, i.e.

      function foo() { return bar.apply(this, arguments); }

    into a special operator JSCallForwardVarargs, which avoids the
    allocation and deconstruction of the arguments object, but just passes
    along the incoming parameters. We can do the same for rest parameters
    and spread calls/constructs, i.e.

      class A extends B {
        constructor(...args) { super(...args); }
      }

    or

      function foo(...args) { return bar(1, 2, 3, ...args); }

    where we basically pass along the parameters (plus maybe additional
    statically known parameters).

    For this, we introduce a new JSConstructForwardVarargs operator and
    generalize the CallForwardVarargs builtins that are backing this.

R=bmeurer@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com,
michael_dawson@ca.ibm.com
BUG=v8:6407,v8:6278,v8:6344
LOG=N

Review-Url: https://codereview.chromium.org/2887153004
Cr-Commit-Position: refs/heads/master@{#45402}
Committed:
https://chromium.googlesource.com/v8/v8/+/f4da43618c18c0c0f6b3a4b488f822f8d40...
==========

[commit-bot: I haz the power, 2017-05-18 19:21:13]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/v8/v8/+/f4da43618c18c0c0f6b3a4b488f822f8d40...