[ia32] Byte and word memory operands in ia32 cmp/test.

[ia32] Byte and word memory operands in ia32 cmp/test.

Currently, if the size of two cmp or test operands is a byte or a word, we sign-extend or zero-extend each of them into a 32-bit register before doing the comparison, even when the conditions for the use of a memory operand are met.

This CL makes it possible to load only one of them into a register and address the other as a memory operand.

The tricky bit is that, unlike as in the x64 counterpart http://crrev.com/1780193003, not all registers can be accessed as bytes.

BUG=

Committed: https://crrev.com/3dd3beb06676fe8030d1fef9086cbe63e4a4376d
Cr-Commit-Position: refs/heads/master@{#35199}

[epertoso, 2016-03-30 15:36:41]

Patchset #1 (id:1) has been deleted

[epertoso, 2016-03-30 15:43:37]

Description was changed from

==========
[turbofan] Byte and word memory operands in ia64 cmp/test.

Currently, if the size of two cmp or test operands is a byte or a word, we
sign-extend or zero-extend each of them into a 32-bit register before doing the
comparison, even when the conditions for the use of a memory operand are met.

This CL makes it possible to load only one of them into a register and address
the other as a memory operand.

The tricky part is that, unlike as in the x64 counterpart
http://crrev.com/1780193003, not all registers can be accessed as bytes.


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

to

==========
[turbofan] Byte and word memory operands in ia64 cmp/test.

Currently, if the size of two cmp or test operands is a byte or a word, we
sign-extend or zero-extend each of them into a 32-bit register before doing the
comparison, even when the conditions for the use of a memory operand are met.

This CL makes it possible to load only one of them into a register and address
the other as a memory operand.

The tricky bit is that, unlike as in the x64 counterpart
http://crrev.com/1780193003, not all registers can be accessed as bytes.


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

[epertoso, 2016-03-30 15:43:56]

Description was changed from

==========
[turbofan] Byte and word memory operands in ia64 cmp/test.

Currently, if the size of two cmp or test operands is a byte or a word, we
sign-extend or zero-extend each of them into a 32-bit register before doing the
comparison, even when the conditions for the use of a memory operand are met.

This CL makes it possible to load only one of them into a register and address
the other as a memory operand.

The tricky bit is that, unlike as in the x64 counterpart
http://crrev.com/1780193003, not all registers can be accessed as bytes.


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

to

==========
[turbofan] Byte and word memory operands in ia32 cmp/test.

Currently, if the size of two cmp or test operands is a byte or a word, we
sign-extend or zero-extend each of them into a 32-bit register before doing the
comparison, even when the conditions for the use of a memory operand are met.

This CL makes it possible to load only one of them into a register and address
the other as a memory operand.

The tricky bit is that, unlike as in the x64 counterpart
http://crrev.com/1780193003, not all registers can be accessed as bytes.


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

[epertoso, 2016-03-30 15:44:09]

epertoso@chromium.org changed reviewers:
+ jarin@chromium.org

[epertoso, 2016-03-30 15:44:26]

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

[commit-bot: I haz the power, 2016-03-30 15:44:35]

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

[commit-bot: I haz the power, 2016-03-30 16:07:54]

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

[commit-bot: I haz the power, 2016-03-30 16:07:55]

Dry run: This issue passed the CQ dry run.

[epertoso, 2016-04-01 08:28:59]

epertoso@chromium.org changed reviewers:
+ bmeurer@chromium.org

[epertoso, 2016-04-01 09:17:42]

Description was changed from

==========
[turbofan] Byte and word memory operands in ia32 cmp/test.

Currently, if the size of two cmp or test operands is a byte or a word, we
sign-extend or zero-extend each of them into a 32-bit register before doing the
comparison, even when the conditions for the use of a memory operand are met.

This CL makes it possible to load only one of them into a register and address
the other as a memory operand.

The tricky bit is that, unlike as in the x64 counterpart
http://crrev.com/1780193003, not all registers can be accessed as bytes.


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

to

==========
[ia32] Byte and word memory operands in ia32 cmp/test.

Currently, if the size of two cmp or test operands is a byte or a word, we
sign-extend or zero-extend each of them into a 32-bit register before doing the
comparison, even when the conditions for the use of a memory operand are met.

This CL makes it possible to load only one of them into a register and address
the other as a memory operand.

The tricky bit is that, unlike as in the x64 counterpart
http://crrev.com/1780193003, not all registers can be accessed as bytes.


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

[epertoso, 2016-04-01 09:30:16]

PTAL

[Benedikt Meurer, 2016-04-01 10:40:12]

LGTM.

[epertoso, 2016-04-01 10:44:08]

The CQ bit was checked by epertoso@chromium.org

[commit-bot: I haz the power, 2016-04-01 10:44:13]

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

[commit-bot: I haz the power, 2016-04-01 11:07:36]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-04-01 11:09:06]

Description was changed from

==========
[ia32] Byte and word memory operands in ia32 cmp/test.

Currently, if the size of two cmp or test operands is a byte or a word, we
sign-extend or zero-extend each of them into a 32-bit register before doing the
comparison, even when the conditions for the use of a memory operand are met.

This CL makes it possible to load only one of them into a register and address
the other as a memory operand.

The tricky bit is that, unlike as in the x64 counterpart
http://crrev.com/1780193003, not all registers can be accessed as bytes.


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

to

==========
[ia32] Byte and word memory operands in ia32 cmp/test.

Currently, if the size of two cmp or test operands is a byte or a word, we
sign-extend or zero-extend each of them into a 32-bit register before doing the
comparison, even when the conditions for the use of a memory operand are met.

This CL makes it possible to load only one of them into a register and address
the other as a memory operand.

The tricky bit is that, unlike as in the x64 counterpart
http://crrev.com/1780193003, not all registers can be accessed as bytes.

BUG=

Committed: https://crrev.com/3dd3beb06676fe8030d1fef9086cbe63e4a4376d
Cr-Commit-Position: refs/heads/master@{#35199}
==========

[commit-bot: I haz the power, 2016-04-01 11:09:07]

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