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

Fix validation and reborrowing of integer pointers #830

Merged
merged 7 commits into from
Jul 11, 2019

Conversation

RalfJung
Copy link
Member

@RalfJung RalfJung commented Jul 6, 2019

@RalfJung RalfJung added the S-blocked-on-rust Status: Blocked on landing a Rust PR label Jul 6, 2019
bors added a commit to rust-lang/rust that referenced this pull request Jul 10, 2019
Miri: Provide pointer forcing methods for MemPlace and Op

These are useful when one wants to to a lot of work with some place or operand and not to int-to-ptr casts all the time. In particular, this is needed to fix some test failures in Miri: we need to normalize before starting a visitor that walks a run-time value, so that we can later be sure (during the visitor walk) that we have a proper `Pointer`.

Also see the Miri side at rust-lang/miri#830.

Cc @eddyb @oli-obk
@RalfJung
Copy link
Member Author

@bors r+

@bors
Copy link
Contributor

bors commented Jul 10, 2019

📌 Commit 11686f4 has been approved by RalfJung

bors added a commit that referenced this pull request Jul 10, 2019
Fix validation and reborrowing of integer pointers

Depends on rust-lang/rust#62441
@bors
Copy link
Contributor

bors commented Jul 10, 2019

⌛ Testing commit 11686f4 with merge 51cffba...

@bors
Copy link
Contributor

bors commented Jul 10, 2019

💔 Test failed - checks-travis

@RalfJung
Copy link
Member Author

Looks like serde needs updating.

@RalfJung
Copy link
Member Author

@bors r+

@bors
Copy link
Contributor

bors commented Jul 11, 2019

📌 Commit 7dd0dd3 has been approved by RalfJung

bors added a commit that referenced this pull request Jul 11, 2019
Fix validation and reborrowing of integer pointers

Depends on rust-lang/rust#62441
@bors
Copy link
Contributor

bors commented Jul 11, 2019

⌛ Testing commit 7dd0dd3 with merge fe9056e...

@bors
Copy link
Contributor

bors commented Jul 11, 2019

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

@bors bors merged commit 7dd0dd3 into rust-lang:master Jul 11, 2019
@RalfJung RalfJung deleted the check-place branch July 23, 2019 06:53
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.

2 participants