[turbofan] Use uint32 to store the number of control outputs instead of uint16.

[turbofan] Use uint32 to store the number of control outputs instead of uint16.

Using uint32 to store the the number of control outputs allows WebAssembly switches to have more than 2^16 case.

BUG=v8:5531
TEST=mjsunit/regress/wasm/regression-5531
R=titzer@chromium.org

Committed: https://crrev.com/2f3ca961c7bd525dd8aac3540d3b5b692b259a2b
Cr-Commit-Position: refs/heads/master@{#40420}

[ahaas, 2016-10-18 11:35:50]

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

[commit-bot: I haz the power, 2016-10-18 11:36:00]

Dry run: 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, 2016-10-18 12:02:30]

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

[commit-bot: I haz the power, 2016-10-18 12:02:31]

Dry run: This issue passed the CQ dry run.

[titzer, 2016-10-18 12:34:07]

https://codereview.chromium.org/2425983002/diff/1/src/wasm/ast-decoder.h
File src/wasm/ast-decoder.h (right):

https://codereview.chromium.org/2425983002/diff/1/src/wasm/ast-decoder.h#newc...
src/wasm/ast-decoder.h:225: if (table_count >
Can we instead fix the TurboFan operator? IIRC such switches may show up in the
wild.

[ahaas, 2016-10-18 12:58:42]

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

[commit-bot: I haz the power, 2016-10-18 12:58:53]

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

[ahaas, 2016-10-18 13:02:16]

Description was changed from

==========
[wasm] Bound the count of branch-table to uint16_max.

The bound is required by the turbofan compiler, because the number of
control outputs is stored in a uint16_t.

BUG=v8:5531
TEST=mjsunit/regress/wasm/regression-5531
R=titzer@chromium.org
==========

to

==========
[turbofan] Use uint32 to store the number of control outputs instead of uint16.

Using uint32 to store the the number of control outputs allows WebAssembly
switches to have more than 2^16 case. 

BUG=v8:5531
TEST=mjsunit/regress/wasm/regression-5531
R=titzer@chromium.org
==========

[ahaas, 2016-10-18 13:02:16]

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

[ahaas, 2016-10-18 13:02:32]

On 2016/10/18 at 12:34:07, titzer wrote:
> https://codereview.chromium.org/2425983002/diff/1/src/wasm/ast-decoder.h
> File src/wasm/ast-decoder.h (right):
> 
>
https://codereview.chromium.org/2425983002/diff/1/src/wasm/ast-decoder.h#newc...
> src/wasm/ast-decoder.h:225: if (table_count >
> Can we instead fix the TurboFan operator? IIRC such switches may show up in
the wild.

PTAL

[titzer, 2016-10-18 13:03:03]

lgtm

[commit-bot: I haz the power, 2016-10-18 13:26:57]

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

[commit-bot: I haz the power, 2016-10-18 13:26:58]

Dry run: This issue passed the CQ dry run.

[ahaas, 2016-10-19 07:10:49]

The CQ bit was checked by ahaas@chromium.org

[commit-bot: I haz the power, 2016-10-19 07:11:01]

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, 2016-10-19 07:25:24]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-17 22:05:40]

Description was changed from

==========
[turbofan] Use uint32 to store the number of control outputs instead of uint16.

Using uint32 to store the the number of control outputs allows WebAssembly
switches to have more than 2^16 case. 

BUG=v8:5531
TEST=mjsunit/regress/wasm/regression-5531
R=titzer@chromium.org
==========

to

==========
[turbofan] Use uint32 to store the number of control outputs instead of uint16.

Using uint32 to store the the number of control outputs allows WebAssembly
switches to have more than 2^16 case.

BUG=v8:5531
TEST=mjsunit/regress/wasm/regression-5531
R=titzer@chromium.org

Committed: https://crrev.com/2f3ca961c7bd525dd8aac3540d3b5b692b259a2b
Cr-Commit-Position: refs/heads/master@{#40420}
==========

[commit-bot: I haz the power, 2016-11-17 22:05:42]

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