[turbofan] Fixing IMulHigh, which has a temp.

[turbofan] Fixing IMulHigh, which has a temp.

On Intel, imul clobbers {r|e}ax.  We're missing that in the representation
of the MulHigh intermediate instructions. Fixing, by adding it as a temp,
akin VisitDiv does.

Committed: https://crrev.com/a685180d38fe6c8b8878e8b0638e475221be37fe
Cr-Commit-Position: refs/heads/master@{#33535}

[Mircea Trofin, 2016-01-27 04:19:07]

Patchset #1 (id:1) has been deleted

[Mircea Trofin, 2016-01-27 04:24:21]

Description was changed from

==========
[turbofan] intel imul has 2 operands.

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

to

==========
On Intel, imul clobbers {r|e}ax.  We're missing that in the representation
of the MulHigh intermediate instructions. Fixing, by adding it as a temp,
akin VisitDiv does.

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

[Mircea Trofin, 2016-01-27 04:24:21]

mtrofin@chromium.org changed reviewers:
+ bmeurer@chromium.org, jarin@chromium.org

[Benedikt Meurer, 2016-01-27 05:07:29]

Description was changed from

==========
On Intel, imul clobbers {r|e}ax.  We're missing that in the representation
of the MulHigh intermediate instructions. Fixing, by adding it as a temp,
akin VisitDiv does.

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

to

==========
[turbofan] Fixing IMulHigh, which has a temp.

On Intel, imul clobbers {r|e}ax.  We're missing that in the representation
of the MulHigh intermediate instructions. Fixing, by adding it as a temp,
akin VisitDiv does.
==========

[Benedikt Meurer, 2016-01-27 05:08:42]

Nice. LGTM.

[Mircea Trofin, 2016-01-27 06:38:02]

The CQ bit was checked by mtrofin@chromium.org

[commit-bot: I haz the power, 2016-01-27 06:38:07]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1631973003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1631973003/40001

[commit-bot: I haz the power, 2016-01-27 06:39:49]

Description was changed from

==========
[turbofan] Fixing IMulHigh, which has a temp.

On Intel, imul clobbers {r|e}ax.  We're missing that in the representation
of the MulHigh intermediate instructions. Fixing, by adding it as a temp,
akin VisitDiv does.
==========

to

==========
[turbofan] Fixing IMulHigh, which has a temp.

On Intel, imul clobbers {r|e}ax.  We're missing that in the representation
of the MulHigh intermediate instructions. Fixing, by adding it as a temp,
akin VisitDiv does.
==========

[commit-bot: I haz the power, 2016-01-27 06:39:49]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-01-27 06:40:18]

Description was changed from

==========
[turbofan] Fixing IMulHigh, which has a temp.

On Intel, imul clobbers {r|e}ax.  We're missing that in the representation
of the MulHigh intermediate instructions. Fixing, by adding it as a temp,
akin VisitDiv does.
==========

to

==========
[turbofan] Fixing IMulHigh, which has a temp.

On Intel, imul clobbers {r|e}ax.  We're missing that in the representation
of the MulHigh intermediate instructions. Fixing, by adding it as a temp,
akin VisitDiv does.

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

[commit-bot: I haz the power, 2016-01-27 06:40:19]

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