-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
ICE when returning generic abitrary self type in defining function for existential associated type #57700
Comments
This doesn't panic on f001287 or nightly for me. Has it been fixed? |
Can't confirm currently, because my toolchain is stuck on nightly 2019-01-18, because of the toolstate changes in that PR. On that toolchain, it ICEs. Will try to reproduce on new nightly once available. |
@dlrobertson If you run it on the playground, you see that it still ICEs |
@MSleepyPanda Aha! Running EDIT: After some research I realized that in order to reproduce the ICE |
This is the same as #53598 I believe, since the |
@Arnavion good catch. I agree. |
NLL triage. P-high, assigning to self for initial investigation. |
downgrading to P-medium under same reasoning as given in #53598 |
This now produces an error instead of an ICE on the latest nightly. |
Fixing the compile error makes it compile successfully. |
Add tests for issue rust-lang#53598 and rust-lang#57700 Closes rust-lang#53598 and rust-lang#57700
Rollup of 10 pull requests Successful merges: - #62108 (Use sharded maps for queries) - #63297 (Improve pointer offset method docs) - #63306 (Adapt AddRetag for shallow retagging) - #63406 (Suggest using a qualified path in patterns with inconsistent bindings) - #63431 (Revert "Simplify MIR generation for logical ops") - #63449 (resolve: Remove remaining special cases from built-in macros) - #63461 (docs: add stdlib env::var(_os) panic) - #63473 (Regression test for #56870) - #63474 (Add tests for issue #53598 and #57700) - #63480 (Fixes #63477) Failed merges: r? @ghost
Add tests for issue rust-lang#53598 and rust-lang#57700 Closes rust-lang#53598 and rust-lang#57700
Rollup of 9 pull requests Successful merges: - #62108 (Use sharded maps for queries) - #63297 (Improve pointer offset method docs) - #63406 (Suggest using a qualified path in patterns with inconsistent bindings) - #63431 (Revert "Simplify MIR generation for logical ops") - #63449 (resolve: Remove remaining special cases from built-in macros) - #63461 (docs: add stdlib env::var(_os) panic) - #63473 (Regression test for #56870) - #63474 (Add tests for issue #53598 and #57700) - #63480 (Fixes #63477) Failed merges: r? @ghost
Playground
The text was updated successfully, but these errors were encountered: