-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
do not propagate closure requirements if we can prove them locally #53745
Conversation
Ping from triage @pnkfelix / @rust-lang/compiler: This PR requires your review. |
@bors r+ |
📌 Commit dacd8b393143fa72874a11db889df6eaad828e96 has been approved by |
⌛ Testing commit dacd8b393143fa72874a11db889df6eaad828e96 with merge 25e07048ff42562bf78b1f9eaebb6819be551c19... |
💔 Test failed - status-appveyor |
@bors r- @nikomatsakis the nll errors are not stable :-/ |
@bors p=1 on RC milestone |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
☔ The latest upstream changes (presumably #53327) made this pull request unmergeable. Please resolve the merge conflicts. |
5bc9a45
to
54a0a59
Compare
This allows it to print out the "late-bound regions" from the closure context more easily. Besides, all the state that is being printed it is private to the `UniversalRegions`.
54a0a59
to
bb8c4c0
Compare
@bors r=pnkfelix |
📌 Commit bb8c4c0 has been approved by |
do not propagate closure requirements if we can prove them locally Fixes #53570 cc @mikhail-m1 -- you're the one who last touched this code r? @pnkfelix
☀️ Test successful - status-appveyor, status-travis |
Fixes #53570
cc @mikhail-m1 -- you're the one who last touched this code
r? @pnkfelix