X87: [turbofan] Take the immediate size in account when narrowing ia32/x64 word comparison operator…

X87: [turbofan] Take the immediate size in account when narrowing ia32/x64 word comparison operators.

  port 2da70f853d7f680d491c37c72d5ef04a85497ba9 (r36136)

  original commit message:
  Trying to re-land http://crrev.com/1948453002 after fixing assembler-x64.cc in http://crrev.com/1962563003.

  Before this patch, we would emit a cmp or test with a memory operand only if both of the operands in the IR were loads.
  Now if either of them is a load and the other one is an immediate, we can use a memory operand if the load representation machine size is wide enough to represent the latter.

BUG=

Committed: https://crrev.com/9af1f35fcc0f13d63ba8a985baccebb38dd031ed
Cr-Commit-Position: refs/heads/master@{#36159}

[zhengxing.li, 2016-05-11 07:16:14]

zhengxing.li@intel.com changed reviewers:
+ chunyang.dai@intel.com, epertoso@chromium.org, weiliang.lin@intel.com

[zhengxing.li, 2016-05-11 07:16:14]

PTAL, thanks!

[Weiliang, 2016-05-11 07:42:46]

lgtm

lgtm

[Weiliang, 2016-05-11 07:42:47]

lgtm

[zhengxing.li, 2016-05-11 07:53:50]

The CQ bit was checked by zhengxing.li@intel.com

[commit-bot: I haz the power, 2016-05-11 07:54:02]

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

[commit-bot: I haz the power, 2016-05-11 08:19:45]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-11 08:22:15]

Description was changed from

==========
X87: [turbofan] Take the immediate size in account when narrowing ia32/x64 word
comparison operators.

  port 2da70f853d7f680d491c37c72d5ef04a85497ba9 (r36136)

  original commit message:
  Trying to re-land http://crrev.com/1948453002 after fixing assembler-x64.cc in
http://crrev.com/1962563003.

  Before this patch, we would emit a cmp or test with a memory operand only if
both of the operands in the IR were loads.
  Now if either of them is a load and the other one is an immediate, we can use
a memory operand if the load representation machine size is wide enough to
represent the latter.

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

to

==========
X87: [turbofan] Take the immediate size in account when narrowing ia32/x64 word
comparison operators.

  port 2da70f853d7f680d491c37c72d5ef04a85497ba9 (r36136)

  original commit message:
  Trying to re-land http://crrev.com/1948453002 after fixing assembler-x64.cc in
http://crrev.com/1962563003.

  Before this patch, we would emit a cmp or test with a memory operand only if
both of the operands in the IR were loads.
  Now if either of them is a load and the other one is an immediate, we can use
a memory operand if the load representation machine size is wide enough to
represent the latter.

BUG=

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

[commit-bot: I haz the power, 2016-05-11 08:22:16]

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