abrown opened Issue #2239:
#2226 enabled Cranelift filetests but this functionality seems unavailable from the CLI:
$ cargo run --color=always --features experimental_x64 -p cranelift-tools -- test cranelift/filetests/filetests/isa/x64 Finished dev [unoptimized + debuginfo] target(s) in 0.07s Running `target/debug/clif-util test cranelift/filetests/filetests/isa/x64` skipped "cranelift/filetests/filetests/isa/x64/basic.clif": missing support for experimental_x64 skipped "cranelift/filetests/filetests/isa/x64/run-const.clif": missing support for experimental_x64 skipped "cranelift/filetests/filetests/isa/x64/simd-lane-access-compile.clif": missing support for experimental_x64 skipped "cranelift/filetests/filetests/isa/x64/simd-lane-access-run.clif": missing support for experimental_x64 4 tests
abrown commented on Issue #2239:
cc: @bnjbvr
abrown commented on Issue #2239:
Also, and probably related, the following does not work:
cargo test --color=always --features experimental_x64 -p cranelift-tools --test filetests filetests
bnjbvr commented on Issue #2239:
It seems to work fine on my end, even after a
git pull
and acargo clean
:➜ cargo run --color=always --features experimental_x64 -p cranelift-tools -- test -vd ./filetests/filetests/isa/x64 Finished dev [unoptimized + debuginfo] target(s) in 0.05s Running `/home/ben/code/wasmtime/target/debug/clif-util test -vd ./filetests/filetests/isa/x64` 0.016 ./filetests/filetests/isa/x64/basic.clif 0.005 ./filetests/filetests/isa/x64/run-const.clif 2 tests
I'm happy to help debugging this over chat. Alternatively, you can use the top-level CI script to run filetests, as long as you have the Nightly toolchain installed:
./ci/run-experimental-x64-ci.sh -- filetests
bnjbvr labeled Issue #2239:
#2226 enabled Cranelift filetests but this functionality seems unavailable from the CLI:
$ cargo run --color=always --features experimental_x64 -p cranelift-tools -- test cranelift/filetests/filetests/isa/x64 Finished dev [unoptimized + debuginfo] target(s) in 0.07s Running `target/debug/clif-util test cranelift/filetests/filetests/isa/x64` skipped "cranelift/filetests/filetests/isa/x64/basic.clif": missing support for experimental_x64 skipped "cranelift/filetests/filetests/isa/x64/run-const.clif": missing support for experimental_x64 skipped "cranelift/filetests/filetests/isa/x64/simd-lane-access-compile.clif": missing support for experimental_x64 skipped "cranelift/filetests/filetests/isa/x64/simd-lane-access-run.clif": missing support for experimental_x64 4 tests
github-actions[bot] commented on Issue #2239:
Subscribe to Label Action
cc @bnjbvr
<details>
This issue or pull request has been labeled: "cranelift"Thus the following users have been cc'd because of the following labels:
- bnjbvr: cranelift
To subscribe or unsubscribe from this label, edit the <code>.github/subscribe-to-label.json</code> configuration file.
Learn more.
</details>
abrown commented on Issue #2239:
Something bizarre is happening but I'm not exactly sure what (after a
cargo clean
):cargo run --color=always --features experimental_x64 -p cranelift-tools -- test -v ./cranelift/filetests/filetests/isa/x64 Finished dev [unoptimized + debuginfo] target(s) in 0.10s Running `target/debug/clif-util test -v ./cranelift/filetests/filetests/isa/x64` skipped "./cranelift/filetests/filetests/isa/x64/basic.clif": missing support for experimental_x64 0.000 ./cranelift/filetests/filetests/isa/x64/basic.clif skipped "./cranelift/filetests/filetests/isa/x64/run-const.clif": missing support for experimental_x64 skipped "./cranelift/filetests/filetests/isa/x64/move-elision.clif": missing support for experimental_x64 skipped "./cranelift/filetests/filetests/isa/x64/simd-lane-access-compile.clif": missing support for experimental_x64 0.000 ./cranelift/filetests/filetests/isa/x64/simd-lane-access-compile.clif 0.000 ./cranelift/filetests/filetests/isa/x64/run-const.clif skipped "./cranelift/filetests/filetests/isa/x64/simd-lane-access-run.clif": missing support for experimental_x64 0.002 ./cranelift/filetests/filetests/isa/x64/simd-lane-access-run.clif 0.000 ./cranelift/filetests/filetests/isa/x64/move-elision.clif 5 tests
abrown commented on Issue #2239:
But if I
cd cranelift
, then:$ cargo run --color=always --features experimental_x64 -p cranelift-tools -- test -vd ./filetests/filetests/isa/x64 Finished dev [unoptimized + debuginfo] target(s) in 0.06s Running `/home/abrown/Code/wasmtime/target/debug/clif-util test -vd ./filetests/filetests/isa/x64` 0.013 ./filetests/filetests/isa/x64/basic.clif 0.046 ./filetests/filetests/isa/x64/simd-lane-access-compile.clif 0.003 ./filetests/filetests/isa/x64/run-const.clif 0.034 ./filetests/filetests/isa/x64/simd-lane-access-run.clif 0.012 ./filetests/filetests/isa/x64/move-elision.clif 5 tests
bnjbvr commented on Issue #2239:
Oh oh, i might have run it from the CL directory all the time, and i suspect it's the feature unification issue striking again; if you run from the top-level directory with
cargo +nightly -Zfeatures=all -Zpackage-features
before the rest of the arguments, does it help?
bjorn3 commented on Issue #2239:
If you are in the workspace root, then
--features
applies to wasmtime. Wasmtime has aexperimental_x64
feature, but it doesn't enable it forcranelift-tools
.
abrown commented on Issue #2239:
Ha! The extra arguments make it all work; I'll close this issue. And thanks @bjorn3 for the explanation; I had seen those arguments in the ci script but didn't really want to look into why they were needed :big_smile:.
abrown closed Issue #2239:
#2226 enabled Cranelift filetests but this functionality seems unavailable from the CLI:
$ cargo run --color=always --features experimental_x64 -p cranelift-tools -- test cranelift/filetests/filetests/isa/x64 Finished dev [unoptimized + debuginfo] target(s) in 0.07s Running `target/debug/clif-util test cranelift/filetests/filetests/isa/x64` skipped "cranelift/filetests/filetests/isa/x64/basic.clif": missing support for experimental_x64 skipped "cranelift/filetests/filetests/isa/x64/run-const.clif": missing support for experimental_x64 skipped "cranelift/filetests/filetests/isa/x64/simd-lane-access-compile.clif": missing support for experimental_x64 skipped "cranelift/filetests/filetests/isa/x64/simd-lane-access-run.clif": missing support for experimental_x64 4 tests
Last updated: Dec 23 2024 at 12:05 UTC