[Interpreter] Don't require a CanonicalHandleScope for parse / renumbering

[Interpreter] Don't require a CanonicalHandleScope for parse / renumbering

Removes the need for a CanonicalHandleScope for parsing and renumbering
phases when using Ignition. Since AST strings are canonicalized by the
AST value factory, we only need to make sure we use the same canonical
handles for any other handles we add to the bytecode generator.

This avoids a regression when enabling Ignition for all Turbofan code, and
improves CodeLoad on for Ignition by about 5%.

BUG=v8:4280
Committed: https://crrev.com/7db6c79acd6ccf45984275136e8c58c4be0f0fd9
Cr-Commit-Position: refs/heads/master@{#40595}

[rmcilroy, 2016-10-26 13:40:00]

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

[commit-bot: I haz the power, 2016-10-26 13:40:09]

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

[rmcilroy, 2016-10-26 13:44:48]

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

[commit-bot: I haz the power, 2016-10-26 13:44:59]

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

[rmcilroy, 2016-10-26 14:02:51]

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

[rmcilroy, 2016-10-26 14:02:52]

Michi / Leszek, PTAL, thanks.

[commit-bot: I haz the power, 2016-10-26 14:10:05]

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

[commit-bot: I haz the power, 2016-10-26 14:10:06]

Dry run: This issue passed the CQ dry run.

[Michael Starzinger, 2016-10-26 14:35:53]

LGTM. Thanks!

[Leszek Swirski, 2016-10-26 14:36:32]

lgtm

[rmcilroy, 2016-10-26 15:08:25]

The CQ bit was checked by rmcilroy@chromium.org

[commit-bot: I haz the power, 2016-10-26 15:08:38]

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-26 15:11:06]

Committed patchset #2 (id:20001)

[Toon Verwaest, 2016-10-26 16:09:31]

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

[Toon Verwaest, 2016-10-26 16:09:31]

Awesome! :-)

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

Description was changed from

==========
[Interpreter] Don't require a CanonicalHandleScope for parse / renumbering

Removes the need for a CanonicalHandleScope for parsing and renumbering
phases when using Ignition. Since AST strings are canonicalized by the
AST value factory, we only need to make sure we use the same canonical
handles for any other handles we add to the bytecode generator.

This avoids a regression when enabling Ignition for all Turbofan code, and
improves CodeLoad on for Ignition by about 5%.

BUG=v8:4280
==========

to

==========
[Interpreter] Don't require a CanonicalHandleScope for parse / renumbering

Removes the need for a CanonicalHandleScope for parsing and renumbering
phases when using Ignition. Since AST strings are canonicalized by the
AST value factory, we only need to make sure we use the same canonical
handles for any other handles we add to the bytecode generator.

This avoids a regression when enabling Ignition for all Turbofan code, and
improves CodeLoad on for Ignition by about 5%.

BUG=v8:4280

Committed: https://crrev.com/7db6c79acd6ccf45984275136e8c58c4be0f0fd9
Cr-Commit-Position: refs/heads/master@{#40595}
==========

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

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