-
Notifications
You must be signed in to change notification settings - Fork 181
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
Make reproducing test(s) for failing to generalize program clauses #614
Comments
Pass canonical goals to program clauses code in SLG solver It seemed like we were leaning towards this solution in the [design meeting](https://rust-lang.zulipchat.com/#narrow/stream/144729-wg-traits/topic/meeting.202020-09-22), and it turned out (assuming these changes are right) to be pretty straightforward. Closes #568. This should stop seeing issues related to failing to generalize program clauses pop up, but I'm only going to cc #614 since I'd like some tests maybe.
Minimal example:
Trying to fit this in a test is tricky though, as the problem triggers with this goal:
The syntax has changed slightly since (old debug formatting here), but this is a I don't think there's currently a way to trigger closure substitutions in chalk tests. My attempts so far:
Similar goals like |
Yes, I've had similar problems reproducing problems with closures from RA. Maybe the closure support in the test harness needs to be more flexible somehow. I've usually been able to reproduce the problems by just modeling the closure as a normal struct. |
If program clauses are not properly generalized it can cause issues in the recursive solver. This was the cause of the recent breakage in rust-analyzer. We should have a test that covers this to prevent future issues.
Discussed briefly here.
The text was updated successfully, but these errors were encountered: