Skip to content
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

Move the stack to the evaluator. (no-op PR for 70598) #1335

Merged
merged 5 commits into from
Apr 20, 2020

Conversation

vakaras
Copy link
Contributor

@vakaras vakaras commented Apr 15, 2020

The changes to Miri to make it to compile with Rustc PR rust-lang/rust#70598.

src/machine.rs Outdated Show resolved Hide resolved
src/machine.rs Outdated Show resolved Hide resolved
Copy link
Member

@RalfJung RalfJung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot! I was prepared to write this PR myself in order to unblock the rustc PR, this saves me taht work. :)

@RalfJung RalfJung added the S-blocked-on-rust Status: Blocked on landing a Rust PR label Apr 15, 2020
src/machine.rs Outdated Show resolved Hide resolved
@RalfJung
Copy link
Member

The rustc PR landed! I updated the rustc-version file here.
@bors r+

@bors
Copy link
Contributor

bors commented Apr 20, 2020

📌 Commit 5d2c835 has been approved by RalfJung

@bors
Copy link
Contributor

bors commented Apr 20, 2020

⌛ Testing commit 5d2c835 with merge 22cebbe...

bors added a commit that referenced this pull request Apr 20, 2020
Move the stack to the evaluator. (no-op PR for 70598)

The changes to Miri to make it to compile with Rustc PR rust-lang/rust#70598.
@bors
Copy link
Contributor

bors commented Apr 20, 2020

💔 Test failed - status-appveyor

@RalfJung
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Apr 20, 2020

📌 Commit 73772fe has been approved by RalfJung

@bors
Copy link
Contributor

bors commented Apr 20, 2020

⌛ Testing commit 73772fe with merge d474bf8...

bors added a commit that referenced this pull request Apr 20, 2020
Move the stack to the evaluator. (no-op PR for 70598)

The changes to Miri to make it to compile with Rustc PR rust-lang/rust#70598.
@bors
Copy link
Contributor

bors commented Apr 20, 2020

💔 Test failed - status-appveyor

@RalfJung
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Apr 20, 2020

📌 Commit 7406c12 has been approved by RalfJung

@bors
Copy link
Contributor

bors commented Apr 20, 2020

⌛ Testing commit 7406c12 with merge 8d9db57...

@bors
Copy link
Contributor

bors commented Apr 20, 2020

☀️ Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing 8d9db57 to master...

@bors bors merged commit 8d9db57 into rust-lang:master Apr 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-blocked-on-rust Status: Blocked on landing a Rust PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants