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

Add escape-to-raw MIR statement #55716

Merged
merged 6 commits into from
Nov 15, 2018
Merged

Add escape-to-raw MIR statement #55716

merged 6 commits into from
Nov 15, 2018

Conversation

RalfJung
Copy link
Member

@RalfJung RalfJung commented Nov 6, 2018

Add a new MIR "ghost state statement": Escaping a ptr to permit raw accesses.

This includes #55549, click here for just the new commits.

@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 6, 2018
@RalfJung
Copy link
Member Author

RalfJung commented Nov 6, 2018

r? @oli-obk

I can also add that to the previous PR, but I felt I shouldn't grow that any further.

@rust-highfive rust-highfive assigned oli-obk and unassigned nikomatsakis Nov 6, 2018
@RalfJung
Copy link
Member Author

RalfJung commented Nov 6, 2018

I declared array index accesses stable places, and added a test.

@RalfJung
Copy link
Member Author

RalfJung commented Nov 6, 2018

Ah, I need to change a bit how I emit EscapeToRaw so that we operate on the original reference before it gets casted, not on the raw ptr after it got casted.

src/librustc_mir/shim.rs Show resolved Hide resolved
src/librustc_mir/interpret/validity.rs Show resolved Hide resolved
@RalfJung
Copy link
Member Author

RalfJung commented Nov 7, 2018

The base PR landed. I rebased. I take @oli-obk's green checkmark as an "r=me".

@bors r=oli-obk

@bors
Copy link
Contributor

bors commented Nov 7, 2018

📌 Commit b891a81 has been approved by oli-obk

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 7, 2018
@bors
Copy link
Contributor

bors commented Nov 7, 2018

💡 This pull request was already approved, no need to approve it again.

@bors
Copy link
Contributor

bors commented Nov 7, 2018

📌 Commit b891a81 has been approved by oli-obk

@RalfJung
Copy link
Member Author

@bors p=1

This effectively blocks progress on stacked borrows / miri.

@bors
Copy link
Contributor

bors commented Nov 15, 2018

⌛ Testing commit b891a81 with merge 4ec0ba9...

bors added a commit that referenced this pull request Nov 15, 2018
Add escape-to-raw MIR statement

Add a new MIR "ghost state statement": Escaping a ptr to permit raw accesses.

~~This includes #55549, [click here](RalfJung/rust@miri-visitor...RalfJung:escape-to-raw) for just the new commits.~~
@bors
Copy link
Contributor

bors commented Nov 15, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: oli-obk
Pushing 4ec0ba9 to master...

@bors bors merged commit b891a81 into rust-lang:master Nov 15, 2018
pietroalbini added a commit to pietroalbini/rust that referenced this pull request Nov 16, 2018
Make miri value visitor usfeful for mutation

This is based on top of rust-lang#55716, [click here](RalfJung/rust@escape-to-raw...RalfJung:mut-visitor) for just the new commits.

r? @oli-obk
kennytm added a commit to kennytm/rust that referenced this pull request Nov 17, 2018
Make miri value visitor usfeful for mutation

~~This is based on top of rust-lang#55716, [click here](RalfJung/rust@escape-to-raw...RalfJung:mut-visitor) for just the new commits.~~

r? @oli-obk
pietroalbini added a commit to pietroalbini/rust that referenced this pull request Nov 18, 2018
Make miri value visitor usfeful for mutation

~~This is based on top of rust-lang#55716, [click here](RalfJung/rust@escape-to-raw...RalfJung:mut-visitor) for just the new commits.~~

r? @oli-obk
@RalfJung RalfJung deleted the escape-to-raw branch November 30, 2018 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants