[fullcodegen] Remove deprecated generator implementation.

[fullcodegen] Remove deprecated generator implementation.

This removes the deprecated generator support for resumable functions
from {FullCodeGenerator}. The existing {AstNumbering} heuristic already
triggers Ignition for most resumable functions, with this change we make
said heuristic a hard choice and remove the deprecated code. This also
has the advantage that any suspended {JSGeneratorObject} instance on the
heap is guaranteed to have code based on a bytecode array.

R=bmeurer@chromium.org

Committed: https://crrev.com/09255541116e651690772f818592d5b6bef6a9d4
Cr-Commit-Position: refs/heads/master@{#41135}

[Michael Starzinger, 2016-11-16 13:09:38]

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

[commit-bot: I haz the power, 2016-11-16 13:09:42]

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

[Michael Starzinger, 2016-11-16 13:29:16]

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

[commit-bot: I haz the power, 2016-11-16 13:29:23]

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

[Michael Starzinger, 2016-11-16 13:30:13]

mstarzinger@chromium.org changed reviewers:
+ rmcilroy@chromium.org

[Michael Starzinger, 2016-11-16 13:30:14]

I plan to land this after the 5.6 branch has been created.

Benedikt: PTAL.
Ross: FYI.

[Michael Starzinger, 2016-11-16 13:32:17]

Description was changed from

==========
[fullcodegen] Remove deprecated generator implementation.

This removes the deprecated generator support for resumable functions
from {FullCodeGenerator}. The existing {AstNumbering} heuristic already
triggers Ignition for most resumable functions, with this change we make
said heuristic a hard choice and remove the deprecated code. This also
has the advantage that all suspended {JSGeneratorObject} instance on the
heap are guaranteed to have code based on a bytecode array.

R=bmeurer@chromium.org
==========

to

==========
[fullcodegen] Remove deprecated generator implementation.

This removes the deprecated generator support for resumable functions
from {FullCodeGenerator}. The existing {AstNumbering} heuristic already
triggers Ignition for most resumable functions, with this change we make
said heuristic a hard choice and remove the deprecated code. This also
has the advantage that any suspended {JSGeneratorObject} instance on the
heap is guaranteed to have code based on a bytecode array.

R=bmeurer@chromium.org
==========

[Michael Starzinger, 2016-11-16 13:38:37]

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

[commit-bot: I haz the power, 2016-11-16 13:38:43]

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

[rmcilroy, 2016-11-16 13:45:06]

Nice! :) LGTM FWIW

[Benedikt Meurer, 2016-11-16 13:45:30]

LGTM!!1!

[Michael Starzinger, 2016-11-16 14:27:58]

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

[commit-bot: I haz the power, 2016-11-16 14:28:07]

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, 2016-11-16 14:55:59]

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

[commit-bot: I haz the power, 2016-11-16 14:56:00]

Dry run: This issue passed the CQ dry run.

[Michael Starzinger, 2016-11-16 15:38:19]

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

[commit-bot: I haz the power, 2016-11-16 15:38:23]

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, 2016-11-16 15:40:33]

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

[commit-bot: I haz the power, 2016-11-16 15:40:34]

Dry run: This issue passed the CQ dry run.

[Michael Starzinger, 2016-11-21 11:52:58]

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

[commit-bot: I haz the power, 2016-11-21 11:53:01]

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, 2016-11-21 12:20:46]

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

[commit-bot: I haz the power, 2016-11-21 12:20:47]

Dry run: This issue passed the CQ dry run.

[Michael Starzinger, 2016-11-21 12:23:05]

The CQ bit was checked by mstarzinger@chromium.org

[Michael Starzinger, 2016-11-21 12:23:06]

The patchset sent to the CQ was uploaded after l-g-t-m from
bmeurer@chromium.org, rmcilroy@chromium.org
Link to the patchset: https://codereview.chromium.org/2504223002/#ps80001
(title: "Rebased.")

[commit-bot: I haz the power, 2016-11-21 12:23:15]

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, 2016-11-21 12:25:12]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1479730985808560,
"parent_rev": ["11a86caef1e586aa0f56181e3d047eedca23a7ca", null], "commit_rev":
["39f0670c143c9695820698019e1fabcec166c1cc", null]}

[commit-bot: I haz the power, 2016-11-21 12:25:17]

Description was changed from

==========
[fullcodegen] Remove deprecated generator implementation.

This removes the deprecated generator support for resumable functions
from {FullCodeGenerator}. The existing {AstNumbering} heuristic already
triggers Ignition for most resumable functions, with this change we make
said heuristic a hard choice and remove the deprecated code. This also
has the advantage that any suspended {JSGeneratorObject} instance on the
heap is guaranteed to have code based on a bytecode array.

R=bmeurer@chromium.org
==========

to

==========
[fullcodegen] Remove deprecated generator implementation.

This removes the deprecated generator support for resumable functions
from {FullCodeGenerator}. The existing {AstNumbering} heuristic already
triggers Ignition for most resumable functions, with this change we make
said heuristic a hard choice and remove the deprecated code. This also
has the advantage that any suspended {JSGeneratorObject} instance on the
heap is guaranteed to have code based on a bytecode array.

R=bmeurer@chromium.org
==========

[commit-bot: I haz the power, 2016-11-21 12:25:18]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-11-21 12:25:46]

Description was changed from

==========
[fullcodegen] Remove deprecated generator implementation.

This removes the deprecated generator support for resumable functions
from {FullCodeGenerator}. The existing {AstNumbering} heuristic already
triggers Ignition for most resumable functions, with this change we make
said heuristic a hard choice and remove the deprecated code. This also
has the advantage that any suspended {JSGeneratorObject} instance on the
heap is guaranteed to have code based on a bytecode array.

R=bmeurer@chromium.org
==========

to

==========
[fullcodegen] Remove deprecated generator implementation.

This removes the deprecated generator support for resumable functions
from {FullCodeGenerator}. The existing {AstNumbering} heuristic already
triggers Ignition for most resumable functions, with this change we make
said heuristic a hard choice and remove the deprecated code. This also
has the advantage that any suspended {JSGeneratorObject} instance on the
heap is guaranteed to have code based on a bytecode array.

R=bmeurer@chromium.org

Committed: https://crrev.com/09255541116e651690772f818592d5b6bef6a9d4
Cr-Commit-Position: refs/heads/master@{#41135}
==========

[commit-bot: I haz the power, 2016-11-21 12:25:48]

Patchset 5 (id:??) landed as
https://crrev.com/09255541116e651690772f818592d5b6bef6a9d4
Cr-Commit-Position: refs/heads/master@{#41135}