diff --git a/rs/embedders/src/wasm_utils/validation.rs b/rs/embedders/src/wasm_utils/validation.rs index db90cacfb87..f350588bd8e 100644 --- a/rs/embedders/src/wasm_utils/validation.rs +++ b/rs/embedders/src/wasm_utils/validation.rs @@ -1364,9 +1364,8 @@ pub fn wasmtime_validation_config(embedders_config: &EmbeddersConfig) -> wasmtim // implementation. config.wasm_multi_memory(false); config.wasm_reference_types(true); - // The SIMD instructions are disable for determinism. + // The relaxed SIMD instructions are disable for determinism. config.wasm_relaxed_simd(false); - config.wasm_simd(false); // Tail calls may be enabled in the future. config.wasm_tail_call(false); // Threads are disabled for determinism. diff --git a/rs/embedders/src/wasmtime_embedder/wasmtime_embedder_tests.rs b/rs/embedders/src/wasmtime_embedder/wasmtime_embedder_tests.rs index 87009956b88..470f5b1dff3 100644 --- a/rs/embedders/src/wasmtime_embedder/wasmtime_embedder_tests.rs +++ b/rs/embedders/src/wasmtime_embedder/wasmtime_embedder_tests.rs @@ -171,10 +171,10 @@ fn test_initial_wasmtime_config() { "tail calls support is not enabled", ), ( - "simd", + "relaxed_simd", "https://github.com/WebAssembly/relaxed-simd/", - "(module (func $f (drop (v128.const i64x2 0 0))))", - "SIMD support is not enabled", + "(module (func $f (param v128) (drop (f64x2.relaxed_madd (local.get 0) (local.get 0) (local.get 0)))))", + "relaxed SIMD support is not enabled", ), ( "threads", @@ -216,10 +216,9 @@ fn test_initial_wasmtime_config() { // Memory control // GC ] { - let wasm_binary = BinaryEncodedWasm::new( - wat::parse_str(wat) - .unwrap_or_else(|_| panic!("Error parsing proposal `{proposal}` code snippet")), - ); + let wasm_binary = BinaryEncodedWasm::new(wat::parse_str(wat).unwrap_or_else(|err| { + panic!("Error parsing proposal `{proposal}` code snippet: {err}") + })); let err = validate_and_instrument_for_testing( &WasmtimeEmbedder::new(EmbeddersConfig::default(), no_op_logger()), &wasm_binary,