[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

Drive-by-fixes: Make sure the JSRegExp::lastIndex field stays Smi
if possible (otherwise we tank the regexp benchmark in Octane).

CQ_INCLUDE_TRYBOTS=master.tryserver.v8:v8_mac64_rel,v8_mac64_dbg
R=jarin@chromium.org
BUG=v8:5267
Committed: https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca99c0ef
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Committed: https://crrev.com/ddf792beb3a72f6dba83e94fc8ada03ebf1630bd
Committed: https://crrev.com/19b3943607521f63fe44774ef36becf747cb818e
Cr-Original-Original-Commit-Position: refs/heads/master@{#39692}
Cr-Original-Commit-Position: refs/heads/master@{#39748}
Cr-Commit-Position: refs/heads/master@{#39803}

[Benedikt Meurer, 2016-09-23 05:14:19]

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

[commit-bot: I haz the power, 2016-09-23 05:14:29]

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-23 05:19:59]

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

[commit-bot: I haz the power, 2016-09-23 05:20:00]

Dry run: Try jobs failed on following builders:
  v8_linux64_gyp_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng/build...)

[Benedikt Meurer, 2016-09-23 05:20:50]

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

[commit-bot: I haz the power, 2016-09-23 05:21:00]

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-23 05:42:13]

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

[commit-bot: I haz the power, 2016-09-23 05:42:14]

Dry run: Try jobs failed on following builders:
  v8_linux64_avx2_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel_ng/buil...)
  v8_linux64_avx2_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel_ng_trig...)

[Benedikt Meurer, 2016-09-23 05:45:47]

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

[commit-bot: I haz the power, 2016-09-23 05:45:53]

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-23 07:46:17]

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

[commit-bot: I haz the power, 2016-09-23 07:46:18]

Dry run: Try jobs failed on following builders:
  v8_mac_rel_ng on master.tryserver.v8 (JOB_TIMED_OUT, no build URL)
  v8_win64_rel_ng on master.tryserver.v8 (JOB_TIMED_OUT, no build URL)
  v8_win_compile_dbg on master.tryserver.v8 (JOB_TIMED_OUT, no build URL)
  v8_win_nosnap_shared_rel_ng on master.tryserver.v8 (JOB_TIMED_OUT, no build
URL)
  v8_win_rel_ng on master.tryserver.v8 (JOB_TIMED_OUT, no build URL)

[Benedikt Meurer, 2016-09-23 08:20:58]

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

[commit-bot: I haz the power, 2016-09-23 08:21:02]

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

[Jarin, 2016-09-23 08:35:17]

lgtm

[commit-bot: I haz the power, 2016-09-23 10:21:25]

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

[commit-bot: I haz the power, 2016-09-23 10:21:26]

Dry run: Try jobs failed on following builders:
  v8_win_compile_dbg on master.tryserver.v8 (JOB_TIMED_OUT, no build URL)
  v8_win_nosnap_shared_rel_ng on master.tryserver.v8 (JOB_TIMED_OUT, no build
URL)

[Benedikt Meurer, 2016-09-23 10:28:41]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-09-23 10:28:55]

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

[Benedikt Meurer, 2016-09-23 10:31:42]

The CQ bit was unchecked by bmeurer@chromium.org

[Benedikt Meurer, 2016-09-23 10:31:44]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-09-23 10:31:47]

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-23 10:37:36]

Description was changed from

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

R=jarin@chromium.org
BUG=v8:5267
==========

to

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

R=jarin@chromium.org
BUG=v8:5267

Committed: https://crrev.com/6a939714e991ebf10d56ddbd2869325cca99c0ef
Cr-Commit-Position: refs/heads/master@{#39655}
==========

[commit-bot: I haz the power, 2016-09-23 10:37:37]

Patchset 3 (id:??) landed as
https://crrev.com/6a939714e991ebf10d56ddbd2869325cca99c0ef
Cr-Commit-Position: refs/heads/master@{#39655}

[Benedikt Meurer, 2016-09-23 10:37:46]

Description was changed from

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

R=jarin@chromium.org
BUG=v8:5267

Committed: https://crrev.com/6a939714e991ebf10d56ddbd2869325cca99c0ef
Cr-Commit-Position: refs/heads/master@{#39655}
==========

to

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
==========

[Benedikt Meurer, 2016-09-23 10:37:47]

Committed patchset #3 (id:40001) manually as
6a939714e991ebf10d56ddbd2869325cca99c0ef (presubmit successful).

[Benedikt Meurer, 2016-09-23 11:23:32]

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

The reason for reverting is: Causes tree breakage:

https://build.chromium.org/p/client.v8/builders/V8%20Win64/builds/12916/steps....

[Benedikt Meurer, 2016-09-23 17:44:31]

Description was changed from

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
==========

to

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
==========

[Benedikt Meurer, 2016-09-23 17:46:36]

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

[commit-bot: I haz the power, 2016-09-23 17:46:43]

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-23 17:48:48]

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

[commit-bot: I haz the power, 2016-09-23 17:48:53]

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-23 18:19:22]

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

[commit-bot: I haz the power, 2016-09-23 18:19:23]

Dry run: This issue passed the CQ dry run.

[Benedikt Meurer, 2016-09-23 18:57:54]

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

[commit-bot: I haz the power, 2016-09-23 18:57:59]

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-23 19:14:21]

The CQ bit was unchecked by bmeurer@chromium.org

[Benedikt Meurer, 2016-09-23 19:14:30]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2016-09-23 19:14:31]

The patchset sent to the CQ was uploaded after l-g-t-m from jarin@chromium.org
Link to the patchset: https://codereview.chromium.org/2367593003/#ps80001
(title: "RegExp mitigation")

[commit-bot: I haz the power, 2016-09-23 19:14: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-09-23 19:26:59]

Description was changed from

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
==========

to

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
==========

[commit-bot: I haz the power, 2016-09-23 19:27:00]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-09-23 19:27:25]

Description was changed from

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
==========

to

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Cr-Commit-Position: refs/heads/master@{#39692}
==========

[commit-bot: I haz the power, 2016-09-23 19:27:26]

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

[Benedikt Meurer, 2016-09-23 20:16:27]

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

The reason for reverting is: Failing on the main waterfall again (again for
typedarray, even tho it passed trybots?):

https://build.chromium.org/p/client.v8/builders/V8%20Mac64/builds/11515/steps....

[Benedikt Meurer, 2016-09-27 04:00:21]

Description was changed from

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Cr-Commit-Position: refs/heads/master@{#39692}
==========

to

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Cr-Commit-Position: refs/heads/master@{#39692}
==========

[Benedikt Meurer, 2016-09-27 04:04:10]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2016-09-27 04:04:11]

The patchset sent to the CQ was uploaded after l-g-t-m from jarin@chromium.org
Link to the patchset: https://codereview.chromium.org/2367593003/#ps100001
(title: "REBASE onto https://codereview.chromium.org/2371963002")

[commit-bot: I haz the power, 2016-09-27 04:04:18]

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-27 04:47:34]

Description was changed from

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Cr-Commit-Position: refs/heads/master@{#39692}
==========

to

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Cr-Commit-Position: refs/heads/master@{#39692}
==========

[commit-bot: I haz the power, 2016-09-27 04:47:35]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-09-27 04:47:47]

Description was changed from

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Cr-Commit-Position: refs/heads/master@{#39692}
==========

to

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Committed: https://crrev.com/ddf792beb3a72f6dba83e94fc8ada03ebf1630bd
Cr-Original-Commit-Position: refs/heads/master@{#39692}
Cr-Commit-Position: refs/heads/master@{#39748}
==========

[commit-bot: I haz the power, 2016-09-27 04:47:48]

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

[Benedikt Meurer, 2016-09-27 05:10:29]

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

The reason for reverting is: Still blows up on the main waterfall even after
Jakob's fix:

https://build.chromium.org/p/client.v8/builders/V8%20Mac64/builds/11557/steps...
https://build.chromium.org/p/client.v8/builders/V8%20Mac64/builds/11557/steps...
https://build.chromium.org/p/client.v8/builders/V8%20Win64/builds/12982/steps....

[Benedikt Meurer, 2016-09-27 05:28:39]

Description was changed from

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Committed: https://crrev.com/ddf792beb3a72f6dba83e94fc8ada03ebf1630bd
Cr-Original-Commit-Position: refs/heads/master@{#39692}
Cr-Commit-Position: refs/heads/master@{#39748}
==========

to

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Committed: https://crrev.com/ddf792beb3a72f6dba83e94fc8ada03ebf1630bd
Cr-Original-Commit-Position: refs/heads/master@{#39692}
Cr-Commit-Position: refs/heads/master@{#39748}
==========

[Benedikt Meurer, 2016-09-27 06:23:57]

Description was changed from

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Committed: https://crrev.com/ddf792beb3a72f6dba83e94fc8ada03ebf1630bd
Cr-Original-Commit-Position: refs/heads/master@{#39692}
Cr-Commit-Position: refs/heads/master@{#39748}
==========

to

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

CQ_INCLUDE_TRYBOTS=master.tryserver.v8:v8_mac64_rel
R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Committed: https://crrev.com/ddf792beb3a72f6dba83e94fc8ada03ebf1630bd
Cr-Original-Commit-Position: refs/heads/master@{#39692}
Cr-Commit-Position: refs/heads/master@{#39748}
==========

[Benedikt Meurer, 2016-09-27 12:44:09]

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

[commit-bot: I haz the power, 2016-09-27 12:44:14]

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-27 12:44:38]

The CQ bit was unchecked by bmeurer@chromium.org

[Benedikt Meurer, 2016-09-27 12:45:12]

Description was changed from

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

CQ_INCLUDE_TRYBOTS=master.tryserver.v8:v8_mac64_rel
R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Committed: https://crrev.com/ddf792beb3a72f6dba83e94fc8ada03ebf1630bd
Cr-Original-Commit-Position: refs/heads/master@{#39692}
Cr-Commit-Position: refs/heads/master@{#39748}
==========

to

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

CQ_INCLUDE_TRYBOTS=master.tryserver.v8:v8_mac64_rel,v8_mac64_dbg
R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Committed: https://crrev.com/ddf792beb3a72f6dba83e94fc8ada03ebf1630bd
Cr-Original-Commit-Position: refs/heads/master@{#39692}
Cr-Commit-Position: refs/heads/master@{#39748}
==========

[Benedikt Meurer, 2016-09-27 13:10:46]

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

[commit-bot: I haz the power, 2016-09-27 13:10:53]

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-27 13:33:30]

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

[commit-bot: I haz the power, 2016-09-27 13:33:32]

Dry run: Try jobs failed on following builders:
  v8_mac64_dbg on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac64_dbg/builds/113)

[Benedikt Meurer, 2016-09-27 17:22:11]

Description was changed from

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

CQ_INCLUDE_TRYBOTS=master.tryserver.v8:v8_mac64_rel,v8_mac64_dbg
R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Committed: https://crrev.com/ddf792beb3a72f6dba83e94fc8ada03ebf1630bd
Cr-Original-Commit-Position: refs/heads/master@{#39692}
Cr-Commit-Position: refs/heads/master@{#39748}
==========

to

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

Drive-by-fixes: Make sure the JSRegExp::lastIndex field stays Smi
if possible (otherwise we tank the regexp benchmark in Octane).
Also ensure that the JSTypedArray::length field is properly initialized
with a Smi always.

CQ_INCLUDE_TRYBOTS=master.tryserver.v8:v8_mac64_rel,v8_mac64_dbg
R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Committed: https://crrev.com/ddf792beb3a72f6dba83e94fc8ada03ebf1630bd
Cr-Original-Commit-Position: refs/heads/master@{#39692}
Cr-Commit-Position: refs/heads/master@{#39748}
==========

[Benedikt Meurer, 2016-09-27 17:22:19]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2016-09-27 17:22:19]

The patchset sent to the CQ was uploaded after l-g-t-m from jarin@chromium.org
Link to the patchset: https://codereview.chromium.org/2367593003/#ps200001
(title: "Enforce Smi representation for JSTypedArray::length in
%TypedArrayInitializeFromArrayLike.")

[commit-bot: I haz the power, 2016-09-27 17:22:29]

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-27 17:39:54]

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

[commit-bot: I haz the power, 2016-09-27 17:39:55]

Try jobs failed on following builders:
  v8_mac64_dbg on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac64_dbg/builds/114)

[Benedikt Meurer, 2016-09-27 17:51:23]

Description was changed from

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

Drive-by-fixes: Make sure the JSRegExp::lastIndex field stays Smi
if possible (otherwise we tank the regexp benchmark in Octane).
Also ensure that the JSTypedArray::length field is properly initialized
with a Smi always.

CQ_INCLUDE_TRYBOTS=master.tryserver.v8:v8_mac64_rel,v8_mac64_dbg
R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Committed: https://crrev.com/ddf792beb3a72f6dba83e94fc8ada03ebf1630bd
Cr-Original-Commit-Position: refs/heads/master@{#39692}
Cr-Commit-Position: refs/heads/master@{#39748}
==========

to

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

Drive-by-fixes: Make sure the JSRegExp::lastIndex field stays Smi
if possible (otherwise we tank the regexp benchmark in Octane).
Also ensure that the JSTypedArray::length field is properly initialized
with a Smi always.

R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Committed: https://crrev.com/ddf792beb3a72f6dba83e94fc8ada03ebf1630bd
Cr-Original-Commit-Position: refs/heads/master@{#39692}
Cr-Commit-Position: refs/heads/master@{#39748}
==========

[Benedikt Meurer, 2016-09-27 17:51:32]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-09-27 17:51:45]

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-27 18:28:08]

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

[commit-bot: I haz the power, 2016-09-27 18:28:10]

Try jobs failed on following builders:
  v8_linux_dbg_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg_ng/builds/13590)
  v8_linux_dbg_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg_ng_triggered/b...)

[Benedikt Meurer, 2016-09-27 18:35:12]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2016-09-27 18:35:13]

The patchset sent to the CQ was uploaded after l-g-t-m from jarin@chromium.org
Link to the patchset: https://codereview.chromium.org/2367593003/#ps220001
(title: "Blacklist SecondaryStubCache test for --turbo and --always-opt.")

[commit-bot: I haz the power, 2016-09-27 18:35:21]

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

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

[commit-bot: I haz the power, 2016-09-27 19:12:37]

Try jobs failed on following builders:
  v8_linux_dbg_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg_ng/builds/13594)
  v8_linux_dbg_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg_ng_triggered/b...)

[Benedikt Meurer, 2016-09-28 04:40:07]

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

[commit-bot: I haz the power, 2016-09-28 04:40:12]

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 04:45:14]

Description was changed from

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

Drive-by-fixes: Make sure the JSRegExp::lastIndex field stays Smi
if possible (otherwise we tank the regexp benchmark in Octane).
Also ensure that the JSTypedArray::length field is properly initialized
with a Smi always.

R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Committed: https://crrev.com/ddf792beb3a72f6dba83e94fc8ada03ebf1630bd
Cr-Original-Commit-Position: refs/heads/master@{#39692}
Cr-Commit-Position: refs/heads/master@{#39748}
==========

to

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

Drive-by-fixes: Make sure the JSRegExp::lastIndex field stays Smi
if possible (otherwise we tank the regexp benchmark in Octane).
Also ensure that the JSTypedArray::length field is properly initialized
with a Smi always.

CQ_INCLUDE_TRYBOTS=master.tryserver.v8:v8_mac64_rel,v8_mac64_dbg
R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Committed: https://crrev.com/ddf792beb3a72f6dba83e94fc8ada03ebf1630bd
Cr-Original-Commit-Position: refs/heads/master@{#39692}
Cr-Commit-Position: refs/heads/master@{#39748}
==========

[Benedikt Meurer, 2016-09-28 04:45:41]

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

[commit-bot: I haz the power, 2016-09-28 04:45:52]

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 04:46:34]

Patchset #13 (id:240001) has been deleted

[Benedikt Meurer, 2016-09-28 04:46:44]

Patchset #12 (id:220001) has been deleted

[Benedikt Meurer, 2016-09-28 04:46:54]

Patchset #7 (id:120001) has been deleted

[Benedikt Meurer, 2016-09-28 04:47:07]

Patchset #4 (id:60001) has been deleted

[Benedikt Meurer, 2016-09-28 04:47:18]

Patchset #2 (id:20001) has been deleted

[Benedikt Meurer, 2016-09-28 04:47:28]

Patchset #5 (id:140001) has been deleted

[Benedikt Meurer, 2016-09-28 04:47:38]

Patchset #5 (id:160001) has been deleted

[Benedikt Meurer, 2016-09-28 04:47:49]

Patchset #5 (id:180001) has been deleted

[Benedikt Meurer, 2016-09-28 04:47:58]

The CQ bit was unchecked by bmeurer@chromium.org

[Benedikt Meurer, 2016-09-28 04:48:06]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2016-09-28 04:48:07]

The patchset sent to the CQ was uploaded after l-g-t-m from jarin@chromium.org
Link to the patchset: https://codereview.chromium.org/2367593003/#ps260001
(title: "Fix the stub cache tests.")

[commit-bot: I haz the power, 2016-09-28 04:48:13]

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

The CQ bit was unchecked by bmeurer@chromium.org

[Benedikt Meurer, 2016-09-28 05:37:03]

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:37:09]

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:37:59]

Description was changed from

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

Drive-by-fixes: Make sure the JSRegExp::lastIndex field stays Smi
if possible (otherwise we tank the regexp benchmark in Octane).
Also ensure that the JSTypedArray::length field is properly initialized
with a Smi always.

CQ_INCLUDE_TRYBOTS=master.tryserver.v8:v8_mac64_rel,v8_mac64_dbg
R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Committed: https://crrev.com/ddf792beb3a72f6dba83e94fc8ada03ebf1630bd
Cr-Original-Commit-Position: refs/heads/master@{#39692}
Cr-Commit-Position: refs/heads/master@{#39748}
==========

to

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

Drive-by-fixes: Make sure the JSRegExp::lastIndex field stays Smi
if possible (otherwise we tank the regexp benchmark in Octane).

CQ_INCLUDE_TRYBOTS=master.tryserver.v8:v8_mac64_rel,v8_mac64_dbg
R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Committed: https://crrev.com/ddf792beb3a72f6dba83e94fc8ada03ebf1630bd
Cr-Original-Commit-Position: refs/heads/master@{#39692}
Cr-Commit-Position: refs/heads/master@{#39748}
==========

[Benedikt Meurer, 2016-09-28 05:50:04]

The CQ bit was unchecked by bmeurer@chromium.org

[Benedikt Meurer, 2016-09-28 05:50:10]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2016-09-28 05:50:10]

The patchset sent to the CQ was uploaded after l-g-t-m from jarin@chromium.org
Link to the patchset: https://codereview.chromium.org/2367593003/#ps280001
(title: "REBASE")

[commit-bot: I haz the power, 2016-09-28 05:50:19]

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:07:46]

Description was changed from

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

Drive-by-fixes: Make sure the JSRegExp::lastIndex field stays Smi
if possible (otherwise we tank the regexp benchmark in Octane).

CQ_INCLUDE_TRYBOTS=master.tryserver.v8:v8_mac64_rel,v8_mac64_dbg
R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Committed: https://crrev.com/ddf792beb3a72f6dba83e94fc8ada03ebf1630bd
Cr-Original-Commit-Position: refs/heads/master@{#39692}
Cr-Commit-Position: refs/heads/master@{#39748}
==========

to

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

Drive-by-fixes: Make sure the JSRegExp::lastIndex field stays Smi
if possible (otherwise we tank the regexp benchmark in Octane).

CQ_INCLUDE_TRYBOTS=master.tryserver.v8:v8_mac64_rel,v8_mac64_dbg
R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Committed: https://crrev.com/ddf792beb3a72f6dba83e94fc8ada03ebf1630bd
Cr-Original-Commit-Position: refs/heads/master@{#39692}
Cr-Commit-Position: refs/heads/master@{#39748}
==========

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

Committed patchset #7 (id:280001)

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

Description was changed from

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

Drive-by-fixes: Make sure the JSRegExp::lastIndex field stays Smi
if possible (otherwise we tank the regexp benchmark in Octane).

CQ_INCLUDE_TRYBOTS=master.tryserver.v8:v8_mac64_rel,v8_mac64_dbg
R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Committed: https://crrev.com/ddf792beb3a72f6dba83e94fc8ada03ebf1630bd
Cr-Original-Commit-Position: refs/heads/master@{#39692}
Cr-Commit-Position: refs/heads/master@{#39748}
==========

to

==========
[turbofan] ChangeFloat64ToTagged shouldn't canonicalize.

This matches current Crankshaft/fullcodegen behavior more closely and
thus reduces the chances that we run into unnecessary polymorphism due
to the field representation tracking in our object model.

Drive-by-fixes: Make sure the JSRegExp::lastIndex field stays Smi
if possible (otherwise we tank the regexp benchmark in Octane).

CQ_INCLUDE_TRYBOTS=master.tryserver.v8:v8_mac64_rel,v8_mac64_dbg
R=jarin@chromium.org
BUG=v8:5267

Committed:
https://chromium.googlesource.com/v8/v8/+/6a939714e991ebf10d56ddbd2869325cca9...
Committed: https://crrev.com/ee158e6c4cc896479a32245432a3c2fdd31bcb73
Committed: https://crrev.com/ddf792beb3a72f6dba83e94fc8ada03ebf1630bd
Committed: https://crrev.com/19b3943607521f63fe44774ef36becf747cb818e
Cr-Original-Original-Commit-Position: refs/heads/master@{#39692}
Cr-Original-Commit-Position: refs/heads/master@{#39748}
Cr-Commit-Position: refs/heads/master@{#39803}
==========

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

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