Skip to content

Commit

Permalink
Fix more tests on miri
Browse files Browse the repository at this point in the history
  • Loading branch information
alexcrichton committed Jan 15, 2025
1 parent 913d93b commit b451571
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
8 changes: 6 additions & 2 deletions crates/test-macros/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -210,9 +210,13 @@ fn expand(test_config: &TestConfig, func: Fn) -> Result<TokenStream> {

for strategy in &test_config.strategies {
let strategy_name = format!("{strategy:?}");
// Winch currently only offers support for x64.
// Winch currently only offers support for x64, and it requires
// signals-based-traps which MIRI disables so disable winch tests on MIRI
let target = if *strategy == Compiler::Winch {
quote! { #[cfg(target_arch = "x86_64")] }
quote! {
#[cfg(target_arch = "x86_64")]
#[cfg_attr(miri, ignore)]
}
} else {
quote! {}
};
Expand Down
1 change: 1 addition & 0 deletions tests/all/module.rs
Original file line number Diff line number Diff line change
Expand Up @@ -255,6 +255,7 @@ fn compile_a_component() -> Result<()> {
}

#[test]
#[cfg_attr(winch, ignore)]
fn tail_call_defaults() -> Result<()> {
let wasm_with_tail_calls = "(module (func $a return_call $a))";

Expand Down

0 comments on commit b451571

Please sign in to comment.