[compiler] Ship Ignition for all TurboFan code.

[compiler] Ship Ignition for all TurboFan code.

This enables Ignition unconditionally for all code that is destined for
optimization with TurboFan. This ensures all optimization attempts will
go through the BytecodeGraphBuilder and that the AstGraphBuilder pipe is
dried out in practice.

patch from issue 2427953002 at patchset 120001 (http://crrev.com/2427953002#ps120001)

R=mvstanton@chromium.org,rmcilroy@chromium.org

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

[Michael Starzinger, 2016-10-27 14:29:30]

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

[commit-bot: I haz the power, 2016-10-27 14:29:37]

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-10-27 15:00:32]

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

[commit-bot: I haz the power, 2016-10-27 15:00:33]

Dry run: This issue passed the CQ dry run.

[Michael Starzinger, 2016-10-28 08:15:17]

Description was changed from

==========
[compiler] Ship Ignition for all TurboFan code.

This enables Ignition unconditionally for all code that is destined for
optimization with TurboFan. This ensures all optimization attempts will
go through the BytecodeGraphBuilder and that the AstGraphBuilder pipe is
dried out in practice.

patch from issue 2427953002 at patchset 120001
(http://crrev.com/2427953002#ps120001)

R=mvstanton@chromium.org
==========

to

==========
[compiler] Ship Ignition for all TurboFan code.

This enables Ignition unconditionally for all code that is destined for
optimization with TurboFan. This ensures all optimization attempts will
go through the BytecodeGraphBuilder and that the AstGraphBuilder pipe is
dried out in practice.

patch from issue 2427953002 at patchset 120001
(http://crrev.com/2427953002#ps120001)

R=mvstanton@chromium.org,rmcilroy@chromium.org
==========

[Michael Starzinger, 2016-10-28 08:15:17]

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

[rmcilroy, 2016-10-28 15:42:25]

Awesome, LGTM, thanks!

[rmcilroy, 2016-10-28 15:42:41]

Missed one comment:

https://codereview.chromium.org/2453973004/diff/1/test/mjsunit/mjsunit.status
File test/mjsunit/mjsunit.status (right):

https://codereview.chromium.org/2453973004/diff/1/test/mjsunit/mjsunit.status...
test/mjsunit/mjsunit.status:66: # Issue 4680: The eval'ed code is piped through
Ignition and fails when being
Do we have a separate bug on this? Would be good to have so we can
release-block-stable on that bug.

[mvstanton, 2016-10-31 05:11:58]

Fantastic, LGTM :)

[Michael Starzinger, 2016-10-31 09:10:14]

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

[commit-bot: I haz the power, 2016-10-31 09:10: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-10-31 09:10:28]

Addressed comments. Landing.

https://codereview.chromium.org/2453973004/diff/1/test/mjsunit/mjsunit.status
File test/mjsunit/mjsunit.status (right):

https://codereview.chromium.org/2453973004/diff/1/test/mjsunit/mjsunit.status...
test/mjsunit/mjsunit.status:66: # Issue 4680: The eval'ed code is piped through
Ignition and fails when being
On 2016/10/28 15:42:41, rmcilroy wrote:
> Do we have a separate bug on this? Would be good to have so we can
> release-block-stable on that bug.

Done: https://bugs.chromium.org/p/v8/issues/detail?id=5587

[commit-bot: I haz the power, 2016-10-31 09:40:00]

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

[commit-bot: I haz the power, 2016-10-31 09:40:01]

Dry run: This issue passed the CQ dry run.

[Michael Starzinger, 2016-10-31 09:41:10]

The CQ bit was checked by mstarzinger@chromium.org

[Michael Starzinger, 2016-10-31 09:41:10]

The patchset sent to the CQ was uploaded after l-g-t-m from
mvstanton@chromium.org, rmcilroy@chromium.org
Link to the patchset: https://codereview.chromium.org/2453973004/#ps20001
(title: "Addressed comment.")

[commit-bot: I haz the power, 2016-10-31 09:41:20]

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-10-31 09:43:24]

Description was changed from

==========
[compiler] Ship Ignition for all TurboFan code.

This enables Ignition unconditionally for all code that is destined for
optimization with TurboFan. This ensures all optimization attempts will
go through the BytecodeGraphBuilder and that the AstGraphBuilder pipe is
dried out in practice.

patch from issue 2427953002 at patchset 120001
(http://crrev.com/2427953002#ps120001)

R=mvstanton@chromium.org,rmcilroy@chromium.org
==========

to

==========
[compiler] Ship Ignition for all TurboFan code.

This enables Ignition unconditionally for all code that is destined for
optimization with TurboFan. This ensures all optimization attempts will
go through the BytecodeGraphBuilder and that the AstGraphBuilder pipe is
dried out in practice.

patch from issue 2427953002 at patchset 120001
(http://crrev.com/2427953002#ps120001)

R=mvstanton@chromium.org,rmcilroy@chromium.org
==========

[commit-bot: I haz the power, 2016-10-31 09:43:25]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-17 22:17:38]

Description was changed from

==========
[compiler] Ship Ignition for all TurboFan code.

This enables Ignition unconditionally for all code that is destined for
optimization with TurboFan. This ensures all optimization attempts will
go through the BytecodeGraphBuilder and that the AstGraphBuilder pipe is
dried out in practice.

patch from issue 2427953002 at patchset 120001
(http://crrev.com/2427953002#ps120001)

R=mvstanton@chromium.org,rmcilroy@chromium.org
==========

to

==========
[compiler] Ship Ignition for all TurboFan code.

This enables Ignition unconditionally for all code that is destined for
optimization with TurboFan. This ensures all optimization attempts will
go through the BytecodeGraphBuilder and that the AstGraphBuilder pipe is
dried out in practice.

patch from issue 2427953002 at patchset 120001
(http://crrev.com/2427953002#ps120001)

R=mvstanton@chromium.org,rmcilroy@chromium.org

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

[commit-bot: I haz the power, 2016-11-17 22:17:40]

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