Skip to content

ICE from regions (Cannot encode region variables) #4492

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.

Sign up for GitHub

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

Jump to bottom
Closed
thestinger opened this issue Jan 15, 2013 · 1 comment
Closed

ICE from regions (Cannot encode region variables) #4492

thestinger opened this issue Jan 15, 2013 · 1 comment
Assignees
@nikomatsakis
Labels
A-lifetimes Area: Lifetimes / regions I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️
Milestone
0.6

Comments

@thestinger
Copy link
Contributor

I'll try cutting this down to a small test case later, but for now here are some in my TreeMap implementation (commented out):
@ghost ghost assigned nikomatsakis Feb 6, 2013
@nikomatsakis nikomatsakis mentioned this issue Mar 21, 2013
Merged
nikomatsakis added a commit to nikomatsakis/rust that referenced this issue Mar 25, 2013
@nikomatsakis
Stop writing directly to the final type/method/vtable sidetables from…
1ca6d06 
… astconv

and from typeck, which is verboten.  We are supposed to write inference results
into the FnCtxt and then these get copied over in writeback.  Add assertions
that no inference by-products are added to this table.

Fixes rust-lang#3888
Fixes rust-lang#4036
Fixes rust-lang#4492
nikomatsakis added a commit to nikomatsakis/rust that referenced this issue Mar 26, 2013
@nikomatsakis
Stop writing directly to the final type/method/vtable sidetables from…
3ca7c22 
… astconv

and from typeck, which is verboten.  We are supposed to write inference results
into the FnCtxt and then these get copied over in writeback.  Add assertions
that no inference by-products are added to this table.

Fixes rust-lang#3888
Fixes rust-lang#4036
Fixes rust-lang#4492
bors added a commit that referenced this issue Mar 26, 2013
@bors
auto merge of #5467 : nikomatsakis/rust/issues-3888-4036-4492-cannot-…
3f7c74d 
…encode-region-variables, r=nikomatsakis

The basic problem was that vtables were not being resolved.  The fix uncovered another issue, which was that the syntax extensions were not creating method calls properly and were relying on outdated code in typeck, so I fixed that too.  

Resolves issues #3888, #4036, #4492.

r? @catamorphism
@catamorphism
Copy link
Contributor

Closing as per 3f7c74d -- reopen if I'm wrong!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nikomatsakis nikomatsakis

Labels
A-lifetimes Area: Lifetimes / regions I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️
Projects
None yet
Milestone
0.6
Development

No branches or pull requests
3 participants
@nikomatsakis @catamorphism @thestinger