-
Notifications
You must be signed in to change notification settings - Fork 88
run Miri on CI (allowed to fail) #255
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
add more simd_reduce intrinsics This makes large parts of the portable-simd test suite work. :D Cc rust-lang/portable-simd#255
I could also add a Miri job to CI? We probably want it to be 'allowed to fail' so that new intrinsics that Miri does not support yet can still easily be added, but it would be nice to somehow track whether the Miri state becomes worse. |
How long does it take to run? Sounds like a good idea to me. |
Not sure, I would guess around 20-30min for the entire test suite -- but I can't say yet because not all of it passes yet. ;) |
c4e8a43
to
302048f
Compare
update recommended CI snippet, add GHA example If this snippet works as expected in rust-lang/portable-simd#255 (which we can test tomorrow), then this adjusts our README to that nicer script.
Ah, now Miri fails since it is still a day old and hence requires |
85e2a69
to
6401dba
Compare
Interesting, that intrinsic is not even declared in https://github.com/rust-lang/portable-simd/blob/master/crates/core_simd/src/intrinsics.rs. 🤔 |
But the tests that are now in there should work. It's a bit annoying to check this since "ignoring failures" means getting green checkmarks everywhere (with GHA there is no way to get a red checkmark for the job but still get a green checkmark for the PR as a whole) -- one has to look at the log to check if it actually failed. But that's the best we can do I guess. |
Yes, that intrinsic is listed with the other "might libcall" floating point intrinsics. portable-simd/crates/std_float/src/lib.rs Lines 24 to 44 in 2d13059
|
This is allowed to fail, but might produce useful results to check on.
With this (and an upcoming Miri PR), a number of tests are actually passing under Miri :D