[compiler] Don't canonicalize handles in Crankshaft.

[compiler] Don't canonicalize handles in Crankshaft.

R=mstarzinger@chromium.org
BUG=v8:5309
Committed: https://crrev.com/b550c07734674f5757be4c1060cf0e30a69e7928
Cr-Commit-Position: refs/heads/master@{#38903}

[Benedikt Meurer, 2016-08-25 10:41:30]

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

[commit-bot: I haz the power, 2016-08-25 10:41:40]

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-08-25 11:03:04]

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

[commit-bot: I haz the power, 2016-08-25 11:03:05]

Dry run: This issue passed the CQ dry run.

[Benedikt Meurer, 2016-08-25 11:14:27]

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

[commit-bot: I haz the power, 2016-08-25 11:14:33]

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

[Benedikt Meurer, 2016-08-25 11:17:31]

Michi: PTAL
Toon: FYI

[Michael Starzinger, 2016-08-25 11:44:20]

https://codereview.chromium.org/2274253003/diff/1/src/compiler.cc
File src/compiler.cc (right):

https://codereview.chromium.org/2274253003/diff/1/src/compiler.cc#newcode726
src/compiler.cc:726: CompilationHandleScope handle_scope(info);
Would it be possible to hoist both, the CompilationHandleScope as well as the
CannonicalHandleScope out into the caller. There the usr_turbofan predicate us
available without passing it around magically. Also it would allow us to see
that GetOptimizedCodeNow is in sync.

[Michael Starzinger, 2016-08-25 11:44:30]

LGTM. Thanks!

[commit-bot: I haz the power, 2016-08-25 11:45:41]

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

[commit-bot: I haz the power, 2016-08-25 11:45:42]

Dry run: This issue passed the CQ dry run.

[Benedikt Meurer, 2016-08-25 11:53:52]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-08-25 11:54:02]

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-08-25 11:56:18]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-25 11:56:37]

Description was changed from

==========
[compiler] Don't canonicalize handles in Crankshaft.

R=mstarzinger@chromium.org
BUG=v8:5309
==========

to

==========
[compiler] Don't canonicalize handles in Crankshaft.

R=mstarzinger@chromium.org
BUG=v8:5309

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

[commit-bot: I haz the power, 2016-08-25 11:56:38]

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