[arm64] We must not overwrite registers for binop results that are used in frame states.

[arm64] We must not overwrite registers for binop results that are used in frame states.

The ARM64 instruction selector can generate code like this

  negs w0, w1
  b.vs deopt

but then reference the old value of w0 in the frame state, which will
obviously lead to wrong results.

R=jarin@chromium.org
BUG=v8:5158
Committed: https://crrev.com/29da5460fc5c82efea03278074fd63223997ade5
Cr-Commit-Position: refs/heads/master@{#37322}

[Jarin, 2016-06-28 09:49:55]

lgtm

[Benedikt Meurer, 2016-06-28 09:50:12]

Description was changed from

==========
[arm64] We must not overwrite registers for binop results that are used in frame
states.

The ARM64 instruction selector can generate code like this

  negs w0, w1
  b.vs deopt

but then reference the old value of w0 in the frame state, which will
obviously lead to wrong results.

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

to

==========
[arm64] We must not overwrite registers for binop results that are used in frame
states.

The ARM64 instruction selector can generate code like this

  negs w0, w1
  b.vs deopt

but then reference the old value of w0 in the frame state, which will
obviously lead to wrong results.

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

[Benedikt Meurer, 2016-06-28 09:50:12]

bmeurer@chromium.org changed reviewers:
+ georgia.kouveli@arm.com, rodolph.perfetta@arm.com

[Benedikt Meurer, 2016-06-28 09:50:18]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-06-28 09:50: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-06-28 10:11:02]

Description was changed from

==========
[arm64] We must not overwrite registers for binop results that are used in frame
states.

The ARM64 instruction selector can generate code like this

  negs w0, w1
  b.vs deopt

but then reference the old value of w0 in the frame state, which will
obviously lead to wrong results.

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

to

==========
[arm64] We must not overwrite registers for binop results that are used in frame
states.

The ARM64 instruction selector can generate code like this

  negs w0, w1
  b.vs deopt

but then reference the old value of w0 in the frame state, which will
obviously lead to wrong results.

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

[commit-bot: I haz the power, 2016-06-28 10:11:03]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-28 10:11:24]

Description was changed from

==========
[arm64] We must not overwrite registers for binop results that are used in frame
states.

The ARM64 instruction selector can generate code like this

  negs w0, w1
  b.vs deopt

but then reference the old value of w0 in the frame state, which will
obviously lead to wrong results.

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

to

==========
[arm64] We must not overwrite registers for binop results that are used in frame
states.

The ARM64 instruction selector can generate code like this

  negs w0, w1
  b.vs deopt

but then reference the old value of w0 in the frame state, which will
obviously lead to wrong results.

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

Committed: https://crrev.com/29da5460fc5c82efea03278074fd63223997ade5
Cr-Commit-Position: refs/heads/master@{#37322}
==========

[commit-bot: I haz the power, 2016-06-28 10:11:25]

Patchset 1 (id:??) landed as
https://crrev.com/29da5460fc5c82efea03278074fd63223997ade5
Cr-Commit-Position: refs/heads/master@{#37322}