[compiler] Fix a bunch of wrong word size assumptions.

[compiler] Fix a bunch of wrong word size assumptions.

Operations on word size data must be word sized, and not word32.
Currently this only generates worse code, but in the future, it
might even generate wrong code, so we should better get this right
from the beginning.

R=yangguo@chromium.org

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

[Benedikt Meurer, 2016-03-01 06:49:22]

Hey Yang,

Bunch of bug fixes to make sure we use the correct machine operators for word
size operations. Currently this is semi-safe, but not by design.
Please take a look.

Thanks,
Benedikt

[Yang, 2016-03-01 06:50:07]

On 2016/03/01 06:49:22, Benedikt Meurer wrote:
> Hey Yang,
> 
> Bunch of bug fixes to make sure we use the correct machine operators for word
> size operations. Currently this is semi-safe, but not by design.
> Please take a look.
> 
> Thanks,
> Benedikt

lgtm.

[Benedikt Meurer, 2016-03-01 06:50:17]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-03-01 06:50:25]

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

[commit-bot: I haz the power, 2016-03-01 07:08:28]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-01 07:10:35]

Description was changed from

==========
[compiler] Fix a bunch of wrong word size assumptions.

Operations on word size data must be word sized, and not word32.
Currently this only generates worse code, but in the future, it
might even generate wrong code, so we should better get this right
from the beginning.

R=yangguo@chromium.org
==========

to

==========
[compiler] Fix a bunch of wrong word size assumptions.

Operations on word size data must be word sized, and not word32.
Currently this only generates worse code, but in the future, it
might even generate wrong code, so we should better get this right
from the beginning.

R=yangguo@chromium.org

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

[commit-bot: I haz the power, 2016-03-01 07:10:36]

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

[rmcilroy, 2016-03-01 19:28:55]

Thanks!

Random thought, is there any way we could run the code through a verification
phase before codegen to weed out invalid word size operations?