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

Rename check_in_alloc to check_inbounds_alloc #908

Closed
wants to merge 1 commit into from

Conversation

tesuji
Copy link
Contributor

@tesuji tesuji commented Aug 15, 2019

@RalfJung
Copy link
Member

RalfJung commented Aug 15, 2019

Sorry, but I already prepared a PR for this quite some time ago: #863.

Thanks a lot for helping to keep Miri green!

@RalfJung RalfJung closed this Aug 15, 2019
@tesuji tesuji deleted the rustup-63075 branch August 15, 2019 09:06
@RalfJung
Copy link
Member

RalfJung commented Aug 15, 2019

I wonder how we could avoid this in the future... I guess you could check the open Miri PRs; I have something like "Miri side of X" in their description when they are matching some breaking change on the rustc side.

Btw, for PRs like this, you will also have to update the rust-version file. That file documents which version of Rust is supposed to work, and it is the version tested by CI.

@tesuji
Copy link
Contributor Author

tesuji commented Aug 15, 2019

Thanks. That's my bad. Feel free to close redundant PRs.

@RalfJung
Copy link
Member

Ah and please run ./miri test before submitting a PR. This one would not have passed the test suite, even after adjusting rust-version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants