-
Notifications
You must be signed in to change notification settings - Fork 360
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
Rustup to rustc 1.28.0-nightly (952f344cd 2018-05-18) #378
Conversation
run-pass/closures is failing:
|
Backtrace:
Doesn't seem to be related to the changes in this PR |
Indeed. I'll see what I can do on the rustc side |
Maybe related : rust-lang/rust#50706 |
src/bin/miri.rs
Outdated
@@ -67,7 +67,7 @@ impl<'a> CompilerCalls<'a> for MiriCompilerCalls { | |||
} | |||
fn late_callback( | |||
&mut self, | |||
trans: &TransCrate, | |||
trans: &CodegenBackend, |
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.
This argument should be renamed codegen_backend
.
There's a bunch of new failures: |
Those failures are inside rustc:
|
Submitted rust-lang/rust#50917 |
Allow `Size` to be any valid `u64` cc rust-lang/miri#378 (comment) The alternative is to make mir::interpret's pointer offsets not be `Size` fixes #50917 r? @eddyb
We need to allow pointers one past the end of an allocation:
|
Nope, if the size is 9 then 8 is the last dereferencable offset and 10 is two past the end. |
rust-lang/rust#50967 required a huge amount of renaming and I still haven't got the majority of the tests passing after several hours :( |
Blocked on: rust-lang/rust#51086 |
Closing in favour of #379 |
No description provided.