-
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
Where clause Binder(<...>)
was applicable to Obligation(predicate=Binder(TraitPredicate(<...>)),depth=1)
but now is not
#52893
Comments
@akiselev Do you think you could create a reproducer that doesn't depend on external crates? |
@Centril I gave it a shot but I think I'm getting other confounding bugs and I can't get the type checking to pass. It's been over a year since I've looked at this Cthulhu's nest of code so I can't even tell if I'm moving in the right direction. Good news is, it still crashes in its original form on
Cargo.toml
Cargo.lock (from a parent workspace)
|
@rustbot ping ICEBreaker-Cleanup-Crew |
Error: This team ( Please let |
Let's see if we can get some help reducing the bug. @rustbot ping icebreakers-cleanup-crew |
Hey Cleanup Crew ICE-breakers! This bug has been identified as a good cc @AminArria @chrissimpkins @contrun @DutchGhost @elshize @ethanboxx @h-michael @HallerPatrick @hdhoang @hellow554 @imtsuki @jakevossen5 @KarlK90 @LeSeulArtichaut @matheus-consoli @mental32 @nmccarty @Noah-Kennedy @pard68 @PeytonT @pierreN @Redblueflame @RobbieClarken @RobertoSnap @robjtede @SarthakSingh31 @senden9 @shekohex @sinato @spastorino @turboladen @woshilapin @yerke |
Hi! First time I have some time to do some ICE work. I could remove the frunk/frunk_core dependency and some trait implementations + simplify the (other older bigger versions : You could make
Which is I think the intended behavior by the author (once the rustc bug is fixed). Removing |
@pierreN removing all comments is also recommended ;) but good work so far! |
Thanks to the Rust playground provided by @pierreN, I was able to reduce a little bit more the example, removing a bunch more generic constraints, some functions, some useless body functions. |
Issue: rust-lang/rust#52893
Issue: rust-lang/rust#52893
No longer ICEs since #86506 |
add some more testcases resolves rust-lang#52893 resolves rust-lang#68295 resolves rust-lang#87750 resolves rust-lang#88071 All these issues have been fixed according to glacier. Just adding a test so it can be closed. Can anybody tell me why the github keywords do not work? 🤔 Please edit this post if you can fix it.
add some more testcases resolves rust-lang#52893 resolves rust-lang#68295 resolves rust-lang#87750 resolves rust-lang#88071 All these issues have been fixed according to glacier. Just adding a test so it can be closed. Can anybody tell me why the github keywords do not work? 🤔 Please edit this post if you can fix it.
EDITED: MCVE is here: #52893 (comment)
with
From the code (Offending implementation at the top, specifically
<X as Entry>::Data: Fn(Y) -> Class<P, OLIST>
:Changing
F: Fn(Y) -> OLIST
toF: Fn(Y) -> Class<P, OLIST>
does not fix the crash. Changing<X as Entry>::Data: Fn(Y) -> Class<P, OLIST>
to<X as Entry>::Data: Fn(Y) -> OLIST
compiles. The latter is the correct implementation and I discovered this crash in the middle of some refactoring so I did not expect anything specific to happen except a type checking error.The crash also happens in nightly 2018-7-23 and 2018-7-30.
The text was updated successfully, but these errors were encountered: