Skip to content

Commit

Permalink
Only include misc_testsuite/winch when testing Winch (bytecodeallia…
Browse files Browse the repository at this point in the history
…nce#6905)

This commit optimizes the use of CI resources by avoiding unnecessary
duplication of WebAssembly spec tests when using the Cranelift compiler
strategy. Previously, Cranelift was tested against both the official spec test
suite and Winch's test suite, the latter being a subset of the former. This
commit eliminates this redundancy.
  • Loading branch information
saulecabrera authored and eduardomourar committed Sep 6, 2023
1 parent 18525e9 commit e4df7a9
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion build.rs
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,13 @@ fn main() -> anyhow::Result<()> {
test_directory_module(out, "tests/misc_testsuite/memory64", strategy)?;
test_directory_module(out, "tests/misc_testsuite/component-model", strategy)?;
test_directory_module(out, "tests/misc_testsuite/function-references", strategy)?;
test_directory_module(out, "tests/misc_testsuite/winch", strategy)?;
// The testsuite of Winch is a subset of the official
// WebAssembly test suite, until parity is reached. This
// check is in place to prevent Cranelift from duplicating
// tests.
if *strategy == "Winch" {
test_directory_module(out, "tests/misc_testsuite/winch", strategy)?;
}
Ok(())
})?;

Expand Down

0 comments on commit e4df7a9

Please sign in to comment.