[turbofan] Disable inline allocation of closures.

[turbofan] Disable inline allocation of closures.

This puts lowering of {JSCreateClosure} operations behind a flag. For
now the benefit of inline allocating such closures is negligible, it
does increase code size, and breaks in combination with inlining based
on {SharedFunctionInfo}.

R=jarin@chromium.org
BUG=v8:2206
Review-Url: https://codereview.chromium.org/2636493002
Cr-Commit-Position: refs/heads/master@{#42331}
Committed: https://chromium.googlesource.com/v8/v8/+/83f19e7d972c9564b523da49782eb41b0939d8ce

[Michael Starzinger, 2017-01-13 12:26:54]

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

[commit-bot: I haz the power, 2017-01-13 12:26:59]

Dry run: 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-01-13 12:55:16]

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

[commit-bot: I haz the power, 2017-01-13 12:55:17]

Dry run: Try jobs failed on following builders:
  v8_win64_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win64_rel_ng/builds/20721)
  v8_win64_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win64_rel_ng_triggered/b...)

[Michael Starzinger, 2017-01-13 13:02:24]

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

[commit-bot: I haz the power, 2017-01-13 13:02:29]

Dry run: 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-01-13 13:49:55]

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

[commit-bot: I haz the power, 2017-01-13 13:49:56]

Dry run: This issue passed the CQ dry run.

[Jarin, 2017-01-13 14:01:42]

jarin@chromium.org changed reviewers:
+ jarin@chromium.org

[Jarin, 2017-01-13 14:01:43]

lgtm

[Michael Starzinger, 2017-01-13 14:04:41]

Description was changed from

==========
[turbofan] Disable inline allocation of closures.

This puts lowering of {JSCreateClosure} operations behind a flag. For
now the benefit of inline allocating such closures is negligible, it
does increase code size, and breaks in combination with inlining based
on {SharedFunctionInfo}.

R=bmeurer@chromium.org
BUG=v8:2206
==========

to

==========
[turbofan] Disable inline allocation of closures.

This puts lowering of {JSCreateClosure} operations behind a flag. For
now the benefit of inline allocating such closures is negligible, it
does increase code size, and breaks in combination with inlining based
on {SharedFunctionInfo}.

R=jarin@chromium.org
BUG=v8:2206
==========

[Michael Starzinger, 2017-01-13 14:04:54]

The CQ bit was checked by mstarzinger@chromium.org

[commit-bot: I haz the power, 2017-01-13 14:05:01]

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-01-13 14:07:03]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484316294717990,
"parent_rev": "aca17974f6811ba5132e8a47a94ae29ba912011a", "commit_rev":
"83f19e7d972c9564b523da49782eb41b0939d8ce"}

[commit-bot: I haz the power, 2017-01-13 14:07:08]

Description was changed from

==========
[turbofan] Disable inline allocation of closures.

This puts lowering of {JSCreateClosure} operations behind a flag. For
now the benefit of inline allocating such closures is negligible, it
does increase code size, and breaks in combination with inlining based
on {SharedFunctionInfo}.

R=jarin@chromium.org
BUG=v8:2206
==========

to

==========
[turbofan] Disable inline allocation of closures.

This puts lowering of {JSCreateClosure} operations behind a flag. For
now the benefit of inline allocating such closures is negligible, it
does increase code size, and breaks in combination with inlining based
on {SharedFunctionInfo}.

R=jarin@chromium.org
BUG=v8:2206

Review-Url: https://codereview.chromium.org/2636493002
Cr-Commit-Position: refs/heads/master@{#42331}
Committed:
https://chromium.googlesource.com/v8/v8/+/83f19e7d972c9564b523da49782eb41b093...
==========

[commit-bot: I haz the power, 2017-01-13 14:07:09]

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