Conversation
|
Oh wow, this worked on the first try. :) |
dhardy
left a comment
There was a problem hiding this comment.
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.