-
Notifications
You must be signed in to change notification settings - Fork 356
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
Update panic machinery to match #[track_caller] changes. #1137
Conversation
This looks right, thanks a lot. :) |
This gets miri's tests passing again with rust-lang/rust#67137.
38ebe42
to
0a3f460
Compare
Rebased now that rust-lang/rust#67137 has landed. Anything else I need to do to unbreak miri? |
@anp you need to import |
Thanks! Cherry-picked your commit, hopefully CI is happy. |
Ugh, I didn't test on travis, this should make check suite green: JohnTitor@7edf4f4 |
Awesome. :) @bors r+ |
📌 Commit 3607daf has been approved by |
Update panic machinery to match #[track_caller] changes. This gets miri's tests passing again with rust-lang/rust#67137, I don't think it will pass CI until that has merged.
☀️ Test successful - checks-travis, status-appveyor |
This gets miri's tests passing again with rust-lang/rust#67137, I don't think it will pass CI until that has merged.