Skip to content

Conversation

arielb1
Copy link
Contributor

@arielb1 arielb1 commented Aug 29, 2017

This allows caching closure signatures and kinds in the normal selection
and evaluation caches, and fixes the exponential worst-case in
@remram44's example, which is a part of #43787.

This improvement is complenentary to #43999 - they fix different cases.

This is the pre-generators variation of #43938, cloned for beta.

This allows caching closure signatures and kinds in the normal selection
and evaluation caches, and fixes the exponential worst-case in
@remram44's example, which is a part of rust-lang#43787.

This improvement is complenentary to rust-lang#43999 - they fix different cases.
@arielb1
Copy link
Contributor Author

arielb1 commented Aug 29, 2017

r? @nikomatsakis

@rust-highfive
Copy link
Contributor

r? @pnkfelix

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

@rust-highfive
Copy link
Contributor

warning Warning warning

  • Pull requests are usually filed against the master branch for this repo, but this one is against beta. Please double check that you specified the right target!

@nikomatsakis
Copy link
Contributor

@bors r+

@bors
Copy link
Collaborator

bors commented Aug 29, 2017

📌 Commit b2a4661 has been approved by nikomatsakis

@arielb1 arielb1 added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Aug 29, 2017
@remram44
Copy link
Contributor

I tested this PR and can confirm that it fixes the compilation time on my particular example. Thanks!

@bors
Copy link
Collaborator

bors commented Aug 31, 2017

⌛ Testing commit b2a4661 with merge dc6ab22...

bors added a commit that referenced this pull request Aug 31, 2017
[beta] Track closure signatures & kinds in freshened types

This allows caching closure signatures and kinds in the normal selection
and evaluation caches, and fixes the exponential worst-case in
@remram44's example, which is a part of #43787.

This improvement is complenentary to #43999 - they fix different cases.

This is the pre-generators variation of #43938, cloned for beta.
@bors
Copy link
Collaborator

bors commented Aug 31, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: nikomatsakis
Pushing dc6ab22 to beta...

@bors bors merged commit b2a4661 into rust-lang:beta Aug 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants