[crankshaft] do not restore rsi in stub for deferred number conversions

[crankshaft] do not restore rsi in stub for deferred number conversions

BUG=v8:5188, chromium:626719
Committed: https://crrev.com/9aa8051f1a8f9b05be0decdb1b09729bc4f5c91e
Cr-Commit-Position: refs/heads/master@{#37766}

[Camillo Bruni, 2016-07-07 09:45:12]

Description was changed from

==========
[crankshaft] do not restore rsi in stub for deferred numver conversions

BUG=
==========

to

==========
[crankshaft] do not restore rsi in stub for deferred number conversions

BUG=
==========

[Camillo Bruni, 2016-07-07 09:45:20]

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

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

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-07-07 10:02:11]

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

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

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

[Camillo Bruni, 2016-07-07 12:40:44]

Description was changed from

==========
[crankshaft] do not restore rsi in stub for deferred number conversions

BUG=
==========

to

==========
[crankshaft] do not restore rsi in stub for deferred number conversions

BUG=v8:5188
==========

[Camillo Bruni, 2016-07-11 11:59:12]

Description was changed from

==========
[crankshaft] do not restore rsi in stub for deferred number conversions

BUG=v8:5188
==========

to

==========
[crankshaft] do not restore rsi in stub for deferred number conversions

BUG=v8:5188, chromium:626719
==========

[Camillo Bruni, 2016-07-13 12:53:30]

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

[commit-bot: I haz the power, 2016-07-13 12:53:35]

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-07-13 13:00:54]

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

[commit-bot: I haz the power, 2016-07-13 13:00:55]

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

[Camillo Bruni, 2016-07-13 15:49:12]

cbruni@chromium.org changed reviewers:
+ jkummerow@chromium.org

[Camillo Bruni, 2016-07-13 15:49:13]

PTAL

[Jakob Kummerow, 2016-07-13 15:56:56]

LGTM with nit

https://codereview.chromium.org/2122953002/diff/80001/src/crankshaft/ia32/lit...
File src/crankshaft/ia32/lithium-codegen-ia32.cc (right):

https://codereview.chromium.org/2122953002/diff/80001/src/crankshaft/ia32/lit...
src/crankshaft/ia32/lithium-codegen-ia32.cc:4412: // Smi to XMM conversion.
Clobbering a temp is faster than -tagging the
accidental edit?

[Camillo Bruni, 2016-07-14 07:54:08]

The CQ bit was checked by cbruni@chromium.org

[Camillo Bruni, 2016-07-14 07:54:08]

The patchset sent to the CQ was uploaded after l-g-t-m from
jkummerow@chromium.org
Link to the patchset: https://codereview.chromium.org/2122953002/#ps100001
(title: "addressing nit")

[commit-bot: I haz the power, 2016-07-14 07:54:10]

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-07-14 07:55:39]

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

[commit-bot: I haz the power, 2016-07-14 07:55:41]

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_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_rel_ng/builds/9067)
  v8_linux_dbg_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg_ng/builds/9037)

[Camillo Bruni, 2016-07-14 08:13:02]

The CQ bit was checked by cbruni@chromium.org

[Camillo Bruni, 2016-07-14 08:13:02]

The patchset sent to the CQ was uploaded after l-g-t-m from
jkummerow@chromium.org
Link to the patchset: https://codereview.chromium.org/2122953002/#ps120001
(title: "rebase onto master")

[commit-bot: I haz the power, 2016-07-14 08:13:06]

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-07-14 08:19:34]

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

[commit-bot: I haz the power, 2016-07-14 08:19:35]

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

[Camillo Bruni, 2016-07-14 12:59:13]

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

[commit-bot: I haz the power, 2016-07-14 12:59:21]

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-07-14 13:28:10]

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

[commit-bot: I haz the power, 2016-07-14 13:28:11]

Dry run: This issue passed the CQ dry run.

[Camillo Bruni, 2016-07-14 13:39:45]

The CQ bit was checked by cbruni@chromium.org

[Camillo Bruni, 2016-07-14 13:39:45]

The patchset sent to the CQ was uploaded after l-g-t-m from
jkummerow@chromium.org
Link to the patchset: https://codereview.chromium.org/2122953002/#ps140001
(title: "fixing register overlap on arm64")

[commit-bot: I haz the power, 2016-07-14 13:39:52]

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-07-14 13:41:55]

Committed patchset #8 (id:140001)

[commit-bot: I haz the power, 2016-07-14 13:44:41]

Description was changed from

==========
[crankshaft] do not restore rsi in stub for deferred number conversions

BUG=v8:5188, chromium:626719
==========

to

==========
[crankshaft] do not restore rsi in stub for deferred number conversions

BUG=v8:5188, chromium:626719

Committed: https://crrev.com/9aa8051f1a8f9b05be0decdb1b09729bc4f5c91e
Cr-Commit-Position: refs/heads/master@{#37766}
==========

[commit-bot: I haz the power, 2016-07-14 13:44:42]

Patchset 8 (id:??) landed as
https://crrev.com/9aa8051f1a8f9b05be0decdb1b09729bc4f5c91e
Cr-Commit-Position: refs/heads/master@{#37766}