-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
[beta] backports #80417
[beta] backports #80417
Conversation
This change breaks some code and doesn't appear to enable any new code.
Thanks to Vadim Petrochenkov who [told me what the fix was][z]! [z]: https://rust-lang.zulipchat.com/#narrow/stream/182449-t-compiler.2Fhelp/topic/finding.20which.20macro.20rule.20to.20use/near/220240422
This avoid the hang/oom from rust-lang#79714
|
@bors r+ rollup=never p=5 |
📌 Commit 3bd08b9992aa605a9343180c2db701e3c19ee5e7 has been approved by |
The job Click to see the possible cause of the failure (guessed by this bot)
|
When normalizing a projection which results in a cycle, we would cache the result of `project_type` without the nested obligations (because they're not needed for inference). This would result in the nested obligations only being handled once in fulfill, which would avoid the cycle error. Fixes rust-lang#79714, a regresion from rust-lang#79305 caused by the removal of `get_paranoid_cache_value_obligation`.
3bd08b9
to
2f925da
Compare
@bors r+ |
📌 Commit 2f925da has been approved by |
⌛ Testing commit 2f925da with merge 6ee62c0425c319949c4411b531fbc51eb5c213bc... |
The job Click to see the possible cause of the failure (guessed by this bot)
|
💔 Test failed - checks-actions |
@bors retry |
☀️ Test successful - checks-actions |
This backports the following to 1.49:
const
to begin a nonterminal Don't allowconst
to begin a nonterminal #80135r? @Mark-Simulacrum