X87: Revert of [turbofan] Take the immediate size in account when narrowing ia32/x64 word compariso…

X87: Revert of [turbofan] Take the immediate size in account when narrowing ia32/x64 word comparison operators. (patchset #1 id:1 of https://codereview.chromium.org/1968453002/ ).

  port 767c34dfae4b0e523c38b0f44e1db70b603d41bf (r36413)

  original commit message:
  Reason for revert:
  Breaks a KCS demo:

  BUG=chromium:611976
Original issue's description:
  > [turbofan] Take the immediate size in account when narrowing ia32/x64 word comparison operators.
  >
  > 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
  >
  > Committed: https://crrev.com/2da70f853d7f680d491c37c72d5ef04a85497ba9
  > Cr-Commit-Position: refs/heads/master@{#36136}

Committed: https://crrev.com/35e0f01fb9e9d191663e2993031829db38e706b5
Cr-Commit-Position: refs/heads/master@{#36456}

[zhengxing.li, 2016-05-24 03:52:34]

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

[zhengxing.li, 2016-05-24 03:52:34]

PTAL, thanks!

[Weiliang, 2016-05-24 03:54:45]

lgtm

[zhengxing.li, 2016-05-24 03:55:44]

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

[commit-bot: I haz the power, 2016-05-24 03:55:55]

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

[commit-bot: I haz the power, 2016-05-24 04:16:41]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-24 04:19:04]

Description was changed from

==========
X87: Revert of [turbofan] Take the immediate size in account when narrowing
ia32/x64 word comparison operators. (patchset #1 id:1 of
https://codereview.chromium.org/1968453002/ ).

  port 767c34dfae4b0e523c38b0f44e1db70b603d41bf (r36413)

  original commit message:
  Reason for revert:
  Breaks a KCS demo:

  BUG=chromium:611976

  Original issue's description:
  > [turbofan] Take the immediate size in account when narrowing ia32/x64 word
comparison operators.
  >
  > 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
  >
  > Committed: https://crrev.com/2da70f853d7f680d491c37c72d5ef04a85497ba9
  > Cr-Commit-Position: refs/heads/master@{#36136}
==========

to

==========
X87: Revert of [turbofan] Take the immediate size in account when narrowing
ia32/x64 word comparison operators. (patchset #1 id:1 of
https://codereview.chromium.org/1968453002/ ).

  port 767c34dfae4b0e523c38b0f44e1db70b603d41bf (r36413)

  original commit message:
  Reason for revert:
  Breaks a KCS demo:

  BUG=chromium:611976

  Original issue's description:
  > [turbofan] Take the immediate size in account when narrowing ia32/x64 word
comparison operators.
  >
  > 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
  >
  > Committed: https://crrev.com/2da70f853d7f680d491c37c72d5ef04a85497ba9
  > Cr-Commit-Position: refs/heads/master@{#36136}

Committed: https://crrev.com/35e0f01fb9e9d191663e2993031829db38e706b5
Cr-Commit-Position: refs/heads/master@{#36456}
==========

[commit-bot: I haz the power, 2016-05-24 04:19:05]

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