1700948 - Also run all wasm tests explicitly with `--wasm-compiler=ion`

Status: RESOLVED FIXED

(Core :: JavaScript: WebAssembly, enhancement)

Description

[Julian Seward [:jseward]]

Now that we have Ion support for wasm on all Tier-1 targets, it's important to
ensure that wasm tests are always run on Ion. That's won't necessarily happen
in the situation that wasm via Cranelift is also supported in the build,
because optimising wasm compilation might be routed via Cranelift instead.
Hence, to guarantee test coverage of Ion, we need to add
test-also=--wasm-compiler=ion to all of the directives.txt files in
jit-test/tests/wasm.

Comment 1

[Julian Seward [:jseward]]

Attachment: Bug 1700948 - Also run all wasm tests explicitly with `--wasm-compiler=ion`. r=lth.

Now that we have Ion support for wasm on all Tier-1 targets, it's important to
ensure that wasm tests are always run on Ion. That won't necessarily happen
in the situation that wasm via Cranelift is also supported in the build,
because optimising wasm compilation might be routed via Cranelift instead.
Hence, to guarantee test coverage of Ion, we need to add
test-also=--wasm-compiler=ion to all of the directives.txt files in
jit-test/tests/wasm.

Depends on D109860

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8ec85768e9b1
Also run all wasm tests explicitly with `--wasm-compiler=ion`.  r=lth.

Comment 3

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/8ec85768e9b1