[stubs] Don't unconditionally canonicalize in ChangeFloat64ToTagged.

[stubs] Don't unconditionally canonicalize in ChangeFloat64ToTagged.

Add a CanonicalizationMode to CodeStubAssembler::ChangeFloat64ToTagged,
so clients can request Smi canonicalization when desired, but otherwise
get Crankshaft/Fullcodegen compatible behavior of just boxing the double
into a HeapNumber.

R=verwaest@chromium.org
BUG=v8:5268
Committed: https://crrev.com/06eef6e6d8199df8317df8469d767092472f3fe0
Cr-Commit-Position: refs/heads/master@{#39804}

[Benedikt Meurer, 2016-09-28 05:35:43]

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

[commit-bot: I haz the power, 2016-09-28 05:35:47]

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-09-28 05:43:52]

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

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

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

[commit-bot: I haz the power, 2016-09-28 06:10:35]

Dry run: This issue passed the CQ dry run.

[Toon Verwaest, 2016-09-28 06:39:49]

lgtm

[Jarin, 2016-09-28 06:40:00]

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

[Jarin, 2016-09-28 06:40:00]

lgtm

[Benedikt Meurer, 2016-09-28 06:40:11]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-09-28 06:40:14]

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-09-28 06:42:52]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-28 06:43:08]

Description was changed from

==========
[stubs] Don't unconditionally canonicalize in ChangeFloat64ToTagged.

Add a CanonicalizationMode to CodeStubAssembler::ChangeFloat64ToTagged,
so clients can request Smi canonicalization when desired, but otherwise
get Crankshaft/Fullcodegen compatible behavior of just boxing the double
into a HeapNumber.

R=verwaest@chromium.org
BUG=v8:5268
==========

to

==========
[stubs] Don't unconditionally canonicalize in ChangeFloat64ToTagged.

Add a CanonicalizationMode to CodeStubAssembler::ChangeFloat64ToTagged,
so clients can request Smi canonicalization when desired, but otherwise
get Crankshaft/Fullcodegen compatible behavior of just boxing the double
into a HeapNumber.

R=verwaest@chromium.org
BUG=v8:5268

Committed: https://crrev.com/06eef6e6d8199df8317df8469d767092472f3fe0
Cr-Commit-Position: refs/heads/master@{#39804}
==========

[commit-bot: I haz the power, 2016-09-28 06:43:09]

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

[Benedikt Meurer, 2016-09-28 09:03:32]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/2373253003/ by bmeurer@chromium.org.

The reason for reverting is: Tanks ai-astar in Kraken pretty badly (some deopt
loop)..