Add for loop to asm-to-wasm

Add for loop to asm-to-wasm

TEST=asm-wasm.js
R=titzer@chromium.org,bradnelson@google.com
BUG=

Committed: https://crrev.com/1c8130bd89d41536669ac0529bf2340f8e8ad405
Cr-Commit-Position: refs/heads/master@{#32895}

[aseemgarg, 2015-12-15 00:21:52]

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

[commit-bot: I haz the power, 2015-12-15 00:21:57]

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

[commit-bot: I haz the power, 2015-12-15 00:21:58]

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

[commit-bot: I haz the power, 2015-12-15 00:22:00]

Dry run: No L-G-T-M from a valid reviewer yet. Only full committers are
accepted.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[bradnelson, 2015-12-15 03:07:03]

bradnelson@chromium.org changed reviewers:
+ bradnelson@chromium.org

[bradnelson, 2015-12-15 03:07:03]

The CQ bit was checked by bradnelson@chromium.org

[bradnelson, 2015-12-15 03:07:04]

lgtm

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

https://codereview.chromium.org/1523843003/diff/1/src/wasm/asm-wasm-builder.c...
src/wasm/asm-wasm-builder.cc:264: void VisitForInStatement(ForInStatement* stmt)
{
Unrelated to this CL, but since for..in, for..of, try..catch, try..finally are
forbidden by the typer, maybe switch them to UNREACHABLE(); in another CL.

[commit-bot: I haz the power, 2015-12-15 03:07:12]

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

[commit-bot: I haz the power, 2015-12-15 03:07:13]

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

[commit-bot: I haz the power, 2015-12-15 03:07:13]

No L-G-T-M from a valid reviewer yet. Only full committers are accepted.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[bradn, 2015-12-15 03:09:53]

The CQ bit was checked by bradnelson@google.com

[bradn, 2015-12-15 03:09:53]

lgtm

[commit-bot: I haz the power, 2015-12-15 03:10:02]

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

[commit-bot: I haz the power, 2015-12-15 03:10:03]

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

[commit-bot: I haz the power, 2015-12-15 03:10:04]

No L-G-T-M from a valid reviewer yet. Only full committers are accepted.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[ahaas, 2015-12-16 11:48:51]

ahaas@chromium.org changed reviewers:
+ ahaas@chromium.org

[ahaas, 2015-12-16 11:48:51]

lgtm

[bradn, 2015-12-16 11:50:01]

The CQ bit was checked by bradnelson@google.com

[commit-bot: I haz the power, 2015-12-16 11:50:09]

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

[commit-bot: I haz the power, 2015-12-16 12:10:19]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-16 12:10:41]

Description was changed from

==========
Add for loop to asm-to-wasm

TEST=asm-wasm.js
R=titzer@chromium.org,bradnelson@google.com
BUG=
==========

to

==========
Add for loop to asm-to-wasm

TEST=asm-wasm.js
R=titzer@chromium.org,bradnelson@google.com
BUG=

Committed: https://crrev.com/1c8130bd89d41536669ac0529bf2340f8e8ad405
Cr-Commit-Position: refs/heads/master@{#32895}
==========

[commit-bot: I haz the power, 2015-12-16 12:10:42]

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