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.

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

Invalid Recursive Enum (2+ occurances) causes compiler out-of-memory on beta/nightly #32326

Closed
alex-ozdemir opened this issue Mar 18, 2016 · 2 comments
Closed

Invalid Recursive Enum (2+ occurances) causes compiler out-of-memory on beta/nightly #32326

alex-ozdemir opened this issue Mar 18, 2016 · 2 comments
Assignees
@nikomatsakis
Labels
P-high High priority regression-from-stable-to-beta Performance or correctness regression from stable to beta. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Comments

@alex-ozdemir
Copy link
Contributor

For example, compiling

enum Expr {
    Plus(Expr, Expr),
    Literal(i64),
}

with rustc causes the error message fatal runtime error: out of memory

Single occurrences seem to work fine.
@eddyb eddyb added regression-from-stable-to-beta Performance or correctness regression from stable to beta. I-nominated T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Mar 18, 2016
@nikomatsakis
Copy link
Contributor

Yeah, I can imagine why this is happening. We are exploring out the nested obligations in a breadth-first-fashion, and hence this blows up memory usage dramatically.

@llogiq llogiq mentioned this issue Mar 22, 2016
Closed
@nikomatsakis
Copy link
Contributor

triage: P-high

@rust-highfive rust-highfive added P-high High priority and removed I-nominated labels Mar 24, 2016
@nikomatsakis nikomatsakis mentioned this issue Mar 24, 2016
Closed
This was referenced Mar 26, 2016
Closed
Closed
@nikomatsakis nikomatsakis mentioned this issue Mar 29, 2016
Merged
@nagisa nagisa mentioned this issue Mar 30, 2016
Closed
bors added a commit that referenced this issue Apr 4, 2016
@bors
Auto merge of #32582 - nikomatsakis:issue-32326, r=aturon
8d5e845 
process cycles as soon as they are detected

We used to wait for the recursion limit, but that might well be too
long!

Fixes #32326
r? @aturon
yberreby added a commit to yberreby/rgo that referenced this issue Apr 17, 2016
@yberreby
ast: get it to compile
4de62a8 
rust-lang/rust#32326 is very annoying
@hanna-kruppe hanna-kruppe mentioned this issue May 1, 2016
Closed
@jonas-schievink jonas-schievink mentioned this issue May 6, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nikomatsakis nikomatsakis

Labels
P-high High priority regression-from-stable-to-beta Performance or correctness regression from stable to beta. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@eddyb @nikomatsakis @alex-ozdemir @rust-highfive