Only allow |0 and *1.0 for asm validator foreign variables.

Only allow |0 and *1.0 for asm validator foreign variables.

Require that foreign variable imports have a correct type annotation,
_including_ checking the value of the multiplier is 0 or 1.0 as appropriate.

BUG= https://code.google.com/p/v8/issues/detail?id=4203
TEST=test-asm-validator
R=aseemgarg@chromium.org,titzer@chromium.org
LOG=N

Committed: https://crrev.com/413c266817f49419765ede4049bfc6ba2f3994e5
Cr-Commit-Position: refs/heads/master@{#33923}

[bradn, 2016-02-12 00:38:41]

bradnelson@google.com changed reviewers:
+ bradnelson@google.com

[aseemgarg, 2016-02-12 01:18:21]

lgtm

We should also try to get +X be a new kind of node instead of X*1.0. This will
always have a difference from Spec. Same for ~~X

[bradn, 2016-02-12 01:28:53]

Agreed, on my list :-)

[bradn, 2016-02-12 01:28:58]

The CQ bit was checked by bradnelson@google.com

[commit-bot: I haz the power, 2016-02-12 01:29:12]

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

[commit-bot: I haz the power, 2016-02-12 01:50:27]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-12 01:50:50]

Description was changed from

==========
Only allow |0 and *1.0 for asm validator foreign variables.

Require that foreign variable imports have a correct type annotation,
_including_ checking the value of the multiplier is 0 or 1.0 as appropriate.

BUG= https://code.google.com/p/v8/issues/detail?id=4203
TEST=test-asm-validator
R=aseemgarg@chromium.org,titzer@chromium.org
LOG=N
==========

to

==========
Only allow |0 and *1.0 for asm validator foreign variables.

Require that foreign variable imports have a correct type annotation,
_including_ checking the value of the multiplier is 0 or 1.0 as appropriate.

BUG= https://code.google.com/p/v8/issues/detail?id=4203
TEST=test-asm-validator
R=aseemgarg@chromium.org,titzer@chromium.org
LOG=N

Committed: https://crrev.com/413c266817f49419765ede4049bfc6ba2f3994e5
Cr-Commit-Position: refs/heads/master@{#33923}
==========

[commit-bot: I haz the power, 2016-02-12 01:50:51]

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