Skip to content

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
Contributor

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.

@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
Collaborator

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
Collaborator

bors commented Nov 7, 2018

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

@bors
Copy link
Collaborator

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
Collaborator

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
Collaborator

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