Bug 1753923 Comment 0 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

SIMD has shipped and seems stable and I don't think we need the run-time switches any more:

* javascript.options.wasm_simd in about:config
* --no-wasm-simd at the shell

These should probably be removed (or we should understand why they are still needed).  All code in the engine that tests this setting should be removed.

The testing predicate and the ifdef remain, since we don't have SIMD on all platforms.

Removing the --no-wasm-simd switch may require another "state" in public/WasmFeatures.h, so there's a little bit of design work.

Obviously this is not urgent, it's just cleanup that should be done to simplify the code.
SIMD has shipped and seems stable and I don't think we need the run-time switches any more:

* javascript.options.wasm_simd in about:config
* --no-wasm-simd at the shell

These should probably be removed (or we should understand why they are still needed).  All code in the engine that tests this setting should be removed.

The TestingFunctions.cpp predicate and the ifdef remain, since we don't have SIMD on all platforms.

Removing the --no-wasm-simd switch may require another "state" in public/WasmFeatures.h, so there's a little bit of design work.

Obviously this is not urgent, it's just cleanup that should be done to simplify the code.
SIMD has shipped and seems stable and I don't think we need the run-time switches any more:

* javascript.options.wasm_simd in about:config
* --no-wasm-simd at the shell

These should probably be removed (or we should understand why they are still needed).  All code in the engine that tests this setting should be removed.

The TestingFunctions.cpp predicate and the ifdef remain, since we don't have SIMD on all platforms.

Removing the --no-wasm-simd switch may require another "state" in public/WasmFeatures.h, so there's a little bit of design work. (Edit: it probably just means removing SIMD from WasmFeatures.h)

Obviously this is not urgent, it's just cleanup that should be done to simplify the code.

Back to Bug 1753923 Comment 0