[wasm] Fix double to int conversions.

[wasm] Fix double to int conversions.

R=ahaas@chromium.org
LOG=Y
BUG=chromium:576560
Committed: https://crrev.com/ed6fea15a9f025c4e928878822cfe7c5ee28d081
Cr-Commit-Position: refs/heads/master@{#33239}

[ahaas, 2016-01-12 13:35:10]

lgtm

https://codereview.chromium.org/1574063005/diff/1/src/wasm/asm-wasm-builder.cc
File src/wasm/asm-wasm-builder.cc (right):

https://codereview.chromium.org/1574063005/diff/1/src/wasm/asm-wasm-builder.c...
src/wasm/asm-wasm-builder.cc:625: return (TypeOf(expr->left()) == kAstI32) ?
kAsIs : kToInt;
Why is this change only required for MatchOr and MatchShr?

[titzer, 2016-01-12 13:39:48]

On 2016/01/12 13:35:10, ahaas wrote:
> lgtm
> 
> https://codereview.chromium.org/1574063005/diff/1/src/wasm/asm-wasm-builder.cc
> File src/wasm/asm-wasm-builder.cc (right):
> 
>
https://codereview.chromium.org/1574063005/diff/1/src/wasm/asm-wasm-builder.c...
> src/wasm/asm-wasm-builder.cc:625: return (TypeOf(expr->left()) == kAstI32) ?
> kAsIs : kToInt;
> Why is this change only required for MatchOr and MatchShr?

Because in asm.js, only |0 and >>>0 are type conversions (i.e. allow their input
operands to be non-int32), and the rest require them to be int32.

[titzer, 2016-01-12 13:39:51]

The CQ bit was checked by titzer@chromium.org

[commit-bot: I haz the power, 2016-01-12 13:40:00]

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

[commit-bot: I haz the power, 2016-01-12 13:45:38]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-12 13:47:06]

Description was changed from

==========
[wasm] Fix double to int conversions.

R=ahaas@chromium.org
LOG=Y
BUG=chromium:576560
==========

to

==========
[wasm] Fix double to int conversions.

R=ahaas@chromium.org
LOG=Y
BUG=chromium:576560

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

[commit-bot: I haz the power, 2016-01-12 13:47:06]

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