-
Notifications
You must be signed in to change notification settings - Fork 150
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
Added Miri
to CI
#573
Added Miri
to CI
#573
Conversation
@cospectrum as there’s been a burst of activity in this repo recently and your change touches a lot of files, let’s merge this after fixing the current merge conflict rather. |
@theotherphil Conflict fixed, benches moved. |
Note: we want to ignore |
@cospectrum is this good to go or do you have more changes pending for this PR? |
You can merge |
Thanks a lot for all the work you’re doing getting a proper testing suite in place. |
I've separated all the tests and benchmarks where it makes sense.
benches
are ignored.UB detected in
nalgebra
(Svd::try_new
) andrayon
(if feature is enabled), they are ignored.