wasm-module-builder: Fix left-over use of kWasmStmt

[backes]

This was renamed to kWasmVoid in 2f5a7c5.

[sbc100]

Is this something that we could land upstream? (I guess its fine here too since we hope to merge soon.. so lgtm either way).

[backes]

Is this something that we could land upstream?

For some reason this was only a problem in this repo. Upstream still uses kWasmStmt. This was renamed to kWasmVoid in the exnref repo, and only here we have the left-over use of kWasmStmt. So probably a rebase artifact.

[sbc100]

I believe the issue is present on the upstream wasm-3.0 branch:

$ git grep kWasmStmt origin/wasm-3.0
origin/wasm-3.0:test/js-api/wasm-module-builder.js:        type == kWasmF64 || type == kWasmS128 || type == kWasmStmt) {

And we inherited since we are based on that branch.

[backes]

Oh, right, I forgot that the upstream branch is wasm-3.0 and not main.

[rossberg]

Can I motivate a V8 person to fix this upstream?

[backes]

I lost track of which repo or branch is based on which other one, and which branches are irrelevant by now. But if you tell me to fix this in the spec's wasm-3.0 branch, I can do that.

[backes]

WebAssembly/spec#1812