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
Fix CI: regenerate known-good outputs with latest Rust nightly #78
Conversation
r? @therealprof (rust-highfive has picked a reviewer for you, use r? to override) |
Ah, cool, the objdump format has changed between our 1.51 and stable/nightly. I haven't bisected the exact version where it changed; bumping MSRV to 1.58. |
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.
Ah, great! thank you!
bors r+
78: Fix CI: regenerate known-good outputs with latest Rust nightly r=eldruin a=adamgreig This was a little boring! There aren't any serious changes except to the exceptions app which has changed around a bit; I haven't thoroughly checked the new assembly but don't have any reason to think it's wrong, just the result of much newer LLVM/Rust. Co-authored-by: Adam Greig <adam@adamgreig.com>
Timed out. |
Ah, the bors config needs to be updated due to the MSRV change. |
Oops, thanks. Updated. |
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.
Thank you!
bors r+
Build succeeded: |
This was a little boring! There aren't any serious changes except to the exceptions app which has changed around a bit; I haven't thoroughly checked the new assembly but don't have any reason to think it's wrong, just the result of much newer LLVM/Rust.