[es6] Consistently use %_GetSuperConstructor to implement super calls.

[es6] Consistently use %_GetSuperConstructor to implement super calls.

Following up on https://crrev.com/1517243002, we use the
%_GetSuperConstructor consistently for all super calls now
(inlining the intrinsic code in fullcodegen).

R=mstarzinger@chromium.org
BUG=v8:3330
LOG=n

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

[Benedikt Meurer, 2015-12-16 09:52:30]

Hey Michi,

This is follow up cleanup for super calls.
Please take a look.

Thanks,
Benedikt

[Michael Starzinger, 2015-12-16 10:48:47]

LGTM.

https://codereview.chromium.org/1529113002/diff/1/src/compiler/ast-graph-buil...
File src/compiler/ast-graph-builder.cc (right):

https://codereview.chromium.org/1529113002/diff/1/src/compiler/ast-graph-buil...
src/compiler/ast-graph-builder.cc:2504: // Prepare the callee to the super call.
The super constructor is stored as
nit: The second sentence in the comment is no longer necessary I think. Should
we drop it?

[Benedikt Meurer, 2015-12-16 10:49:40]

https://codereview.chromium.org/1529113002/diff/1/src/compiler/ast-graph-buil...
File src/compiler/ast-graph-builder.cc (right):

https://codereview.chromium.org/1529113002/diff/1/src/compiler/ast-graph-buil...
src/compiler/ast-graph-builder.cc:2504: // Prepare the callee to the super call.
The super constructor is stored as
On 2015/12/16 10:48:47, Michael Starzinger wrote:
> nit: The second sentence in the comment is no longer necessary I think. Should
> we drop it?

Done.

[Benedikt Meurer, 2015-12-16 10:49:54]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2015-12-16 10:49:54]

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

[commit-bot: I haz the power, 2015-12-16 10:49:59]

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

[commit-bot: I haz the power, 2015-12-16 11:08:04]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-16 11:08:12]

Description was changed from

==========
[es6] Consistently use %_GetSuperConstructor to implement super calls.

Following up on https://crrev.com/1517243002, we use the
%_GetSuperConstructor consistently for all super calls now
(inlining the intrinsic code in fullcodegen).

R=mstarzinger@chromium.org
BUG=v8:3330
LOG=n
==========

to

==========
[es6] Consistently use %_GetSuperConstructor to implement super calls.

Following up on https://crrev.com/1517243002, we use the
%_GetSuperConstructor consistently for all super calls now
(inlining the intrinsic code in fullcodegen).

R=mstarzinger@chromium.org
BUG=v8:3330
LOG=n

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

[commit-bot: I haz the power, 2015-12-16 11:08:13]

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