[wasm] Change the default lowering of stores.

[wasm] Change the default lowering of stores.

The int64-lowering only lowered store instructions with a word64 store
representation. For all other stores the default lowering applied. The
default lowering replaces all input nodes with both their replacement
nodes, which can change the number of input nodes of the lowered node.

In WebAssembly there exist stores which take an I64 input and store it
with a different representation, e.g. I32. In TurboFan this translates
to a store node with word32 store representation and a word64 value
input. The default lowering replaces the word64 value input to become
two word32 value inputs, which makes the number of inputs of the store
node invalid. This CL discards the high word replacement of the value
input so that the number of input nodes of a store node does not change
in the default lowering.

R=titzer@chromium.org
CC=rossberg@chromium.org

BUG=

Review-Url: https://codereview.chromium.org/2668023004
Cr-Commit-Position: refs/heads/master@{#42860}
Committed: https://chromium.googlesource.com/v8/v8/+/dd51dd926eff96e23b3022998f5a486f29860fac

[ahaas, 2017-02-01 15:18:20]

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

[commit-bot: I haz the power, 2017-02-01 15:18:29]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[titzer, 2017-02-01 15:31:14]

lgtm

[commit-bot: I haz the power, 2017-02-01 15:43:54]

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

[commit-bot: I haz the power, 2017-02-01 15:43:55]

Dry run: This issue passed the CQ dry run.

[ahaas, 2017-02-01 16:25:10]

The CQ bit was checked by ahaas@chromium.org

[commit-bot: I haz the power, 2017-02-01 16:25:16]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-01 16:27:12]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485966310112310, "parent_rev":
"93f181b6b9bfd0f48c4e8a8fc13ad6e662b6faa6", "commit_rev":
"dd51dd926eff96e23b3022998f5a486f29860fac"}

[commit-bot: I haz the power, 2017-02-01 16:27:17]

Description was changed from

==========
[wasm] Change the default lowering of stores.

The int64-lowering only lowered store instructions with a word64 store
representation. For all other stores the default lowering applied. The
default lowering replaces all input nodes with both their replacement
nodes, which can change the number of input nodes of the lowered node.

In WebAssembly there exist stores which take an I64 input and store it
with a different representation, e.g. I32. In TurboFan this translates
to a store node with word32 store representation and a word64 value
input. The default lowering replaces the word64 value input to become
two word32 value inputs, which makes the number of inputs of the store
node invalid. This CL discards the high word replacement of the value
input so that the number of input nodes of a store node does not change
in the default lowering.

R=titzer@chromium.org
CC=rossberg@chromium.org

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

to

==========
[wasm] Change the default lowering of stores.

The int64-lowering only lowered store instructions with a word64 store
representation. For all other stores the default lowering applied. The
default lowering replaces all input nodes with both their replacement
nodes, which can change the number of input nodes of the lowered node.

In WebAssembly there exist stores which take an I64 input and store it
with a different representation, e.g. I32. In TurboFan this translates
to a store node with word32 store representation and a word64 value
input. The default lowering replaces the word64 value input to become
two word32 value inputs, which makes the number of inputs of the store
node invalid. This CL discards the high word replacement of the value
input so that the number of input nodes of a store node does not change
in the default lowering.

R=titzer@chromium.org
CC=rossberg@chromium.org

BUG=

Review-Url: https://codereview.chromium.org/2668023004
Cr-Commit-Position: refs/heads/master@{#42860}
Committed:
https://chromium.googlesource.com/v8/v8/+/dd51dd926eff96e23b3022998f5a486f298...
==========

[commit-bot: I haz the power, 2017-02-01 16:27:18]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/dd51dd926eff96e23b3022998f5a486f298...