[turbofan] Disable inlining of derived class constructors.

[turbofan] Disable inlining of derived class constructors.

The inlining logic doesn't account for the fact that the derived
constructor could return a primitive, thus leaking the implicit
receiver (which is the hole).

R=jarin@chromium.org
BUG=chromium:706642
Review-Url: https://codereview.chromium.org/2788603002
Cr-Commit-Position: refs/heads/master@{#44264}
Committed: https://chromium.googlesource.com/v8/v8/+/c019e53cbb0e3249d28c2d88eca64bf73df8de56

[Benedikt Meurer, 2017-03-30 09:54:20]

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

[commit-bot: I haz the power, 2017-03-30 09:54:31]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Benedikt Meurer, 2017-03-30 09:55:58]

bmeurer@chromium.org changed reviewers:
+ gsathya@chromium.org

[Benedikt Meurer, 2017-03-30 09:55:59]

Disabling the optimization for now (back-mergable), so that we have time to
really address the underlying inconsistency.
Please take a look.

Thanks,
Benedikt

[Jarin, 2017-03-30 09:57:49]

lgtm

[Benedikt Meurer, 2017-03-30 09:58:00]

The CQ bit was unchecked by bmeurer@chromium.org

[Benedikt Meurer, 2017-03-30 09:58:03]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2017-03-30 09:58:10]

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-03-30 10:17:10]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490867883658550, "parent_rev":
"eef2a462ad9a1ee5a25bec09d9a4a446b124557d", "commit_rev":
"c019e53cbb0e3249d28c2d88eca64bf73df8de56"}

[commit-bot: I haz the power, 2017-03-30 10:17:17]

Description was changed from

==========
[turbofan] Disable inlining of derived class constructors.

The inlining logic doesn't account for the fact that the derived
constructor could return a primitive, thus leaking the implicit
receiver (which is the hole).

R=jarin@chromium.org
BUG=chromium:706642
==========

to

==========
[turbofan] Disable inlining of derived class constructors.

The inlining logic doesn't account for the fact that the derived
constructor could return a primitive, thus leaking the implicit
receiver (which is the hole).

R=jarin@chromium.org
BUG=chromium:706642

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

[commit-bot: I haz the power, 2017-03-30 10:17:18]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/c019e53cbb0e3249d28c2d88eca64bf73df...