Skip to content

Issue 72408 nested closures exponential#72412

Merged
bors merged 4 commits intorust-lang:masterfrom
VFLashM:issue-72408-nested-closures-exponential
Sep 18, 2020
Merged

Issue 72408 nested closures exponential#72412
bors merged 4 commits intorust-lang:masterfrom
VFLashM:issue-72408-nested-closures-exponential

Conversation

@VFLashM
Copy link
Contributor

@VFLashM VFLashM commented May 21, 2020

This fixes #72408.

Nested closures were resulting in exponential compilation time.

This PR is enhancing asymptotic complexity, but also increasing the constant, so I would love to see perf run results.

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

Labels

beta-accepted Accepted for backporting to the compiler in the beta channel. I-compiletime Issue: Problems and improvements with respect to compile times. merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Nested closures result in exponential compilation time increase