Skip to content

erase lifetimes when translating specialized substs #38249

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

Merged
merged 1 commit into from
Dec 11, 2016

Conversation

arielb1
Copy link
Contributor

@arielb1 arielb1 commented Dec 8, 2016

Projections can generate lifetime variables with equality constraints,
that will not be resolved by resolve_type_vars_if_possible, so substs
need to be lifetime-erased after that.

Fixes #36848.

Projections can generate lifetime variables with equality constraints,
that will not be resolved by `resolve_type_vars_if_possible`, so substs
need to be lifetime-erased after that.

Fixes rust-lang#36848.
@rust-highfive
Copy link
Contributor

r? @nrc

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

@arielb1
Copy link
Contributor Author

arielb1 commented Dec 8, 2016

r? @nikomatsakis

@rust-highfive rust-highfive assigned nikomatsakis and unassigned nrc Dec 8, 2016
@nikomatsakis
Copy link
Contributor

@bors r+

@bors
Copy link
Collaborator

bors commented Dec 10, 2016

📌 Commit 55180d0 has been approved by nikomatsakis

@bors
Copy link
Collaborator

bors commented Dec 11, 2016

⌛ Testing commit 55180d0 with merge e97a979...

@bors
Copy link
Collaborator

bors commented Dec 11, 2016

💔 Test failed - auto-win-msvc-64-cargotest

@alexcrichton
Copy link
Member

alexcrichton commented Dec 11, 2016 via email

@bors
Copy link
Collaborator

bors commented Dec 11, 2016

⌛ Testing commit 55180d0 with merge e0158e0...

bors added a commit that referenced this pull request Dec 11, 2016
erase lifetimes when translating specialized substs

Projections can generate lifetime variables with equality constraints,
that will not be resolved by `resolve_type_vars_if_possible`, so substs
need to be lifetime-erased after that.

Fixes #36848.
@bors bors merged commit 55180d0 into rust-lang:master Dec 11, 2016
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.

ICE: translate_substs returned a struct containing inference types/regions (specialization)
6 participants