From f9e5f274e8a195e8485999e67356795e2df4f7dd Mon Sep 17 00:00:00 2001 From: Afonso Bordado Date: Wed, 6 Jul 2022 23:52:59 +0100 Subject: [PATCH] cranelift: Disable pinned_reg runtests (#4392) We've had some ABI issues with pinned reg in runtests. See #4376 for more info. --- cranelift/filetests/src/test_run.rs | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/cranelift/filetests/src/test_run.rs b/cranelift/filetests/src/test_run.rs index 4f80050b10..504d3e0f2d 100644 --- a/cranelift/filetests/src/test_run.rs +++ b/cranelift/filetests/src/test_run.rs @@ -50,6 +50,16 @@ impl SubTest for TestRun { return Ok(()); } + // Disable runtests with pinned reg enabled. + // We've had some abi issues that the trampoline isn't quite ready for. + if context.flags.enable_pinned_reg() { + return Err(anyhow::anyhow!([ + "Cannot run runtests with pinned_reg enabled.", + "See https://github.com/bytecodealliance/wasmtime/issues/4376 for more info" + ] + .join("\n"))); + } + let test_env = RuntestEnvironment::parse(&context.details.comments[..])?; let mut compiler = SingleFunctionCompiler::with_host_isa(context.flags.clone())?;