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

erase late bound regions in get_vtable_methods() #39887

Merged
merged 1 commit into from
Feb 18, 2017

Conversation

nikomatsakis
Copy link
Contributor

Higher-ranked object types can otherwise cause late-bound regions to
sneak into the substs, leading to the false conclusion that some method
is unreachable.

r? @arielb1, who wrote the heart of this patch anyhow

Fixes #39292

Higher-ranked object types can otherwise cause late-bound regions to
sneak into the substs, leading to the false conclusion that some method
is unreachable.  The heart of this patch is from @arielb1.
@arielb1
Copy link
Contributor

arielb1 commented Feb 16, 2017

@bors r+

@bors
Copy link
Contributor

bors commented Feb 16, 2017

📌 Commit f2d8a00 has been approved by arielb1

@bors
Copy link
Contributor

bors commented Feb 18, 2017

⌛ Testing commit f2d8a00 with merge 8f2fc9d...

bors added a commit that referenced this pull request Feb 18, 2017
erase late bound regions in `get_vtable_methods()`

Higher-ranked object types can otherwise cause late-bound regions to
sneak into the substs, leading to the false conclusion that some method
is unreachable.

r? @arielb1, who wrote the heart of this patch anyhow

Fixes #39292
@bors
Copy link
Contributor

bors commented Feb 18, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: arielb1
Pushing 8f2fc9d to master...

@bors bors merged commit f2d8a00 into rust-lang:master Feb 18, 2017
@nikomatsakis nikomatsakis deleted the issue-39292 branch April 14, 2017 10:13
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.

3 participants