Run cargo build against wasm32-unknown-unknown target to catch MIR …

…failures (#86) rust-lang/rust#49292
mkeeter · Apr 15, 2024 · a53276d · a53276d
1 parent 058a3cb
commit a53276d
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/.github/workflows/check-wasm.yml b/.github/workflows/check-wasm.yml
@@ -23,4 +23,6 @@ jobs:
     - name: Install wasm target
       run: rustup target add ${{ matrix.target }}
     - name: Check
-      run: cargo check --target=${{ matrix.target }} -pfidget --no-default-features --features="rhai,render,mesh"
+      # cargo check doesn't find MIR diagnostics, so we use cargo build instead
+      # (https://github.com/rust-lang/rust/issues/49292)
+      run: cargo build --target=${{ matrix.target }} -pfidget --no-default-features --features="rhai,render,mesh"
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/fidget/src/mesh/cell.rs b/fidget/src/mesh/cell.rs
@@ -19,10 +19,12 @@ impl From<Cell> for CellData {
             Cell::Empty => 1,
             Cell::Full => 2,
             Cell::Branch { index, thread } => {
+                #[cfg(not(target_arch = "wasm32"))]
                 debug_assert!(index < (1 << 54));
                 0b10 << 62 | ((thread as u64) << 54) | index as u64
             }
             Cell::Leaf(Leaf { mask, index }) => {
+                #[cfg(not(target_arch = "wasm32"))]
                 debug_assert!(index < (1 << 54));
                 (0b11 << 62) | ((mask as u64) << 54) | index as u64
             }