[compiler] Prepare for partially shipping Ignition.

[compiler] Prepare for partially shipping Ignition.

This prepares the code-base so that Ignition can be enabled on a certain
subset of compilations without setting the {FLAG_ignition} flag (which
enables Ignition on all compilations). We should not check the flag in
question explicitly anywhere outside of the compiler heuristics.

R=mvstanton@chromium.org

Committed: https://crrev.com/8549811ae3e67f1f7578aef7c3e1d8bbcf97105e
Cr-Commit-Position: refs/heads/master@{#40617}

[Michael Starzinger, 2016-10-21 12:40:17]

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

[commit-bot: I haz the power, 2016-10-21 12:40:19]

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-21 12:41:19]

FYI, this is subset of: https://codereview.chromium.org/2427953002/

[mvstanton, 2016-10-21 13:04:04]

Right on, rubber-stamp LGTM

[mvstanton, 2016-10-21 13:04:04]

Right on, rubber-stamp LGTM

[commit-bot: I haz the power, 2016-10-21 13:08:35]

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

[commit-bot: I haz the power, 2016-10-21 13:08:35]

Dry run: This issue passed the CQ dry run.

[Michael Starzinger, 2016-10-21 13:11:50]

The CQ bit was checked by mstarzinger@chromium.org

[commit-bot: I haz the power, 2016-10-21 13:11:55]

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-21 13:14:20]

Committed patchset #1 (id:1)

[Michael Starzinger, 2016-10-24 12:33:41]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2448443002/ by mstarzinger@chromium.org.

The reason for reverting is: Causes regressions:
https://bugs.chromium.org/p/chromium/issues/detail?id=658711.

[Michael Starzinger, 2016-10-27 10:49:51]

Description was changed from

==========
[compiler] Prepare for partially shipping Ignition.

This prepares the code-base so that Ignition can be enabled on a certain
subset of compilations without setting the {FLAG_ignition} flag (which
enables Ignition on all compilations). We should not check the flag in
question explicitly anywhere outside of the compiler heuristics.

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

to

==========
[compiler] Prepare for partially shipping Ignition.

This prepares the code-base so that Ignition can be enabled on a certain
subset of compilations without setting the {FLAG_ignition} flag (which
enables Ignition on all compilations). We should not check the flag in
question explicitly anywhere outside of the compiler heuristics.

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

[Michael Starzinger, 2016-10-27 10:58:38]

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

[commit-bot: I haz the power, 2016-10-27 10:58:49]

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-27 11:00:46]

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

[Michael Starzinger, 2016-10-27 11:00:46]

Rebased onto 7db6c79acd6ccf45984275136e8c58c4be0f0fd9 which should addressed the
performance regressions. Kudos go to Ross. Will reland this CL now.

[rmcilroy, 2016-10-27 11:13:14]

Uncomfortably excited :D.

[commit-bot: I haz the power, 2016-10-27 11:25:21]

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

[commit-bot: I haz the power, 2016-10-27 11:25:23]

Dry run: This issue passed the CQ dry run.

[Michael Starzinger, 2016-10-27 12:15:01]

The CQ bit was checked by mstarzinger@chromium.org

[Michael Starzinger, 2016-10-27 12:15:02]

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

[commit-bot: I haz the power, 2016-10-27 12:15:07]

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 12:17:19]

Description was changed from

==========
[compiler] Prepare for partially shipping Ignition.

This prepares the code-base so that Ignition can be enabled on a certain
subset of compilations without setting the {FLAG_ignition} flag (which
enables Ignition on all compilations). We should not check the flag in
question explicitly anywhere outside of the compiler heuristics.

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

to

==========
[compiler] Prepare for partially shipping Ignition.

This prepares the code-base so that Ignition can be enabled on a certain
subset of compilations without setting the {FLAG_ignition} flag (which
enables Ignition on all compilations). We should not check the flag in
question explicitly anywhere outside of the compiler heuristics.

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

[commit-bot: I haz the power, 2016-10-27 12:17:20]

Committed patchset #2 (id:20001)

[Michael Achenbach, 2016-10-27 20:13:29]

Any chance that this makes arm64 sim debug nosnap much slower:
https://build.chromium.org/p/client.v8.ports/builders/V8%20Linux%20-%20arm64%...

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

Description was changed from

==========
[compiler] Prepare for partially shipping Ignition.

This prepares the code-base so that Ignition can be enabled on a certain
subset of compilations without setting the {FLAG_ignition} flag (which
enables Ignition on all compilations). We should not check the flag in
question explicitly anywhere outside of the compiler heuristics.

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

to

==========
[compiler] Prepare for partially shipping Ignition.

This prepares the code-base so that Ignition can be enabled on a certain
subset of compilations without setting the {FLAG_ignition} flag (which
enables Ignition on all compilations). We should not check the flag in
question explicitly anywhere outside of the compiler heuristics.

R=mvstanton@chromium.org

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

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

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

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

Description was changed from

==========
[compiler] Prepare for partially shipping Ignition.

This prepares the code-base so that Ignition can be enabled on a certain
subset of compilations without setting the {FLAG_ignition} flag (which
enables Ignition on all compilations). We should not check the flag in
question explicitly anywhere outside of the compiler heuristics.

R=mvstanton@chromium.org

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

to

==========
[compiler] Prepare for partially shipping Ignition.

This prepares the code-base so that Ignition can be enabled on a certain
subset of compilations without setting the {FLAG_ignition} flag (which
enables Ignition on all compilations). We should not check the flag in
question explicitly anywhere outside of the compiler heuristics.

R=mvstanton@chromium.org

Committed: https://crrev.com/8549811ae3e67f1f7578aef7c3e1d8bbcf97105e
Cr-Commit-Position: refs/heads/master@{#40617}
==========

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

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