-
Notifications
You must be signed in to change notification settings - Fork 432
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
add Miri to CI #781
add Miri to CI #781
Conversation
Oh wow, this worked on the first try. :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good other than the hacks. Lets wait until your other PRs are merged then rebase.
Shame so many tests had to be disabled, but I guess that's the way it is.
Yeah... they don't seem to be around any unsafe code though if I didn't miss anything? |
Looks good.
This commit doesn't do what it says any more. But I think the plan is to fix |
Why doesn't it? It makes it so that on Miri it will use the fallback version (formerly
That would be best. Unfortunately, doing that is beyond my knowledge of rand. |
So I we rebase this, fingers crossed it'll work perfectly... |
Looking good. |
We're now seeing the test fail frequently in many PRs:
Is this a case of Miri not being enabled in the nightly build, and if so can we hide the error? |
You are running into rust-lang/miri#713. I will submit a PR. |
This adds a CI job running the test suites in Miri -- at least if Miri is available for the current nightly.
Includes #780.