chore: try to make simd ops more likely to autovectorize

Signed-off-by: Henry <mail@henrygressmann.de>

Author: explodingcamera

.github/workflows/test.yaml

@@ -13,21 +13,22 @@ jobs:
     name: Build wasm
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
         with:
           submodules: true
       - name: Install Rust toolchain
-        uses: actions-rust-lang/setup-rust-toolchain@v1
+        uses: actions-rust-lang/setup-rust-toolchain@150fca883cd4034361b621bd4e6a9d34e5143606 # v1.15.4
         with:
-          toolchain: stable
+          toolchain: nightly
+          components: rust-src
           rustflags: ""
           target: wasm32-unknown-unknown
       - name: Install Binaryen and WABT
         run: sudo apt-get install -y binaryen wabt
       - name: Build wasm
         run: ./examples/rust/build.sh
       - name: Save artifacts
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@bbbca2ddaa5d8feaa63e36b76fdaad77386f024f # v7.0.0
         with:
           name: wasm
           path: examples/rust/out
@@ -67,20 +68,20 @@ jobs:
     runs-on: ${{ matrix.os }}
     steps:
       - name: Checkout code
-        uses: actions/checkout@v4
+        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
         with:
           submodules: true
 
       - name: Install Rust toolchain
-        uses: actions-rust-lang/setup-rust-toolchain@v1
+        uses: actions-rust-lang/setup-rust-toolchain@150fca883cd4034361b621bd4e6a9d34e5143606 # v1.15.4
         with:
           toolchain: ${{ matrix.rust }}
           rustflags: ""
           components: rustfmt, clippy
         if: matrix.target == ''
 
       - name: Load wasm
-        uses: actions/download-artifact@v4
+        uses: actions/download-artifact@3e5f45b2cfb9172054b4087a40e8e0b5a5461e7c # v8.0.1
         with:
           name: wasm
           path: examples/rust/out
@@ -94,7 +95,7 @@ jobs:
         if: matrix.target == ''
 
       - name: Run tests (${{ matrix.target }})
-        uses: houseabsolute/actions-rust-cross@v0.0.13
+        uses: houseabsolute/actions-rust-cross@a8cc74d61047fa553b4e908b4b10e70029f00ca6 # v1.0.6
         with:
           command: test
           target: ${{ matrix.target }}

crates/tinywasm/benches/argon2id.rs

@@ -31,11 +31,11 @@ fn argon2id_run(module: TinyWasmModule) -> Result<()> {
 
 fn criterion_benchmark(c: &mut Criterion) {
     let module = argon2id_parse().expect("argon2id_parse");
-    let _twasm = argon2id_to_twasm(&module).expect("argon2id_to_twasm");
+    let twasm = argon2id_to_twasm(&module).expect("argon2id_to_twasm");
 
-    // c.bench_function("argon2id_parse", |b| b.iter(argon2id_parse));
-    // c.bench_function("argon2id_to_twasm", |b| b.iter(|| argon2id_to_twasm(&module)));
-    // c.bench_function("argon2id_from_twasm", |b| b.iter(|| argon2id_from_twasm(&twasm)));
+    c.bench_function("argon2id_parse", |b| b.iter(argon2id_parse));
+    c.bench_function("argon2id_to_twasm", |b| b.iter(|| argon2id_to_twasm(&module)));
+    c.bench_function("argon2id_from_twasm", |b| b.iter(|| argon2id_from_twasm(&twasm)));
     c.bench_function("argon2id", |b| b.iter(|| argon2id_run(module.clone())));
 }
 

crates/tinywasm/benches/fibonacci.rs

@@ -36,11 +36,11 @@ fn fibonacci_run(module: TinyWasmModule, recursive: bool, n: i32) -> Result<()>
 
 fn criterion_benchmark(c: &mut Criterion) {
     let module = fibonacci_parse().expect("fibonacci_parse");
-    let _twasm = fibonacci_to_twasm(&module).expect("fibonacci_to_twasm");
+    let twasm = fibonacci_to_twasm(&module).expect("fibonacci_to_twasm");
 
-    // c.bench_function("fibonacci_parse", |b| b.iter(fibonacci_parse));
-    // c.bench_function("fibonacci_to_twasm", |b| b.iter(|| fibonacci_to_twasm(&module)));
-    // c.bench_function("fibonacci_from_twasm", |b| b.iter(|| fibonacci_from_twasm(&twasm)));
+    c.bench_function("fibonacci_parse", |b| b.iter(fibonacci_parse));
+    c.bench_function("fibonacci_to_twasm", |b| b.iter(|| fibonacci_to_twasm(&module)));
+    c.bench_function("fibonacci_from_twasm", |b| b.iter(|| fibonacci_from_twasm(&twasm)));
     c.bench_function("fibonacci_iterative_60", |b| b.iter(|| fibonacci_run(module.clone(), false, 60)));
     c.bench_function("fibonacci_recursive_26", |b| b.iter(|| fibonacci_run(module.clone(), true, 26)));
 }

crates/tinywasm/benches/tinywasm.rs

@@ -33,13 +33,13 @@ fn tinywasm_run(module: TinyWasmModule) -> Result<()> {
 
 fn criterion_benchmark(c: &mut Criterion) {
     let module = tinywasm_parse().expect("tinywasm_parse");
-    let _twasm = tinywasm_to_twasm(&module).expect("tinywasm_to_twasm");
+    let twasm = tinywasm_to_twasm(&module).expect("tinywasm_to_twasm");
     let mut group = c.benchmark_group("tinywasm");
     group.measurement_time(std::time::Duration::from_secs(10));
 
-    // group.bench_function("tinywasm_parse", |b| b.iter(tinywasm_parse));
-    // group.bench_function("tinywasm_to_twasm", |b| b.iter(|| tinywasm_to_twasm(&module)));
-    // group.bench_function("tinywasm_from_twasm", |b| b.iter(|| tinywasm_from_twasm(&twasm)));
+    group.bench_function("tinywasm_parse", |b| b.iter(tinywasm_parse));
+    group.bench_function("tinywasm_to_twasm", |b| b.iter(|| tinywasm_to_twasm(&module)));
+    group.bench_function("tinywasm_from_twasm", |b| b.iter(|| tinywasm_from_twasm(&twasm)));
     group.bench_function("tinywasm", |b| b.iter(|| tinywasm_run(module.clone())));
 }
 

crates/tinywasm/src/engine.rs

@@ -42,19 +42,15 @@ pub(crate) struct EngineInner {
 /// Fuel accounting policy for budgeted execution.
 #[derive(Debug, Clone, Copy)]
 #[non_exhaustive]
+#[derive(Default)]
 pub enum FuelPolicy {
     /// Charge one fuel unit per retired instruction.
+    #[default]
     PerInstruction,
     /// Charge one fuel unit per instruction plus predefined extra cost for specific operations.
     Weighted,
 }
 
-impl Default for FuelPolicy {
-    fn default() -> Self {
-        Self::PerInstruction
-    }
-}
-
 /// Default initial size for the 32-bit value stack (i32, f32 values).
 pub const DEFAULT_VALUE_STACK_32_SIZE: usize = 64 * 1024; // 64k slots