Skip to content

refactor: encapsulate TyLoweringContext usage in hir-ty#21622

Closed
raushan728 wants to merge 1 commit intorust-lang:masterfrom
raushan728:fix-dependent-generic-defaults
Closed

refactor: encapsulate TyLoweringContext usage in hir-ty#21622
raushan728 wants to merge 1 commit intorust-lang:masterfrom
raushan728:fix-dependent-generic-defaults

Conversation

@raushan728
Copy link
Contributor

@raushan728 raushan728 commented Feb 11, 2026

Waiting

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 11, 2026
@ShoyuVanilla
Copy link
Member

ShoyuVanilla commented Feb 11, 2026

That test is just sensitive to query graph changes, not flaky. You could just update the expectation to the actual output

@raushan728
Copy link
Contributor Author

thanks for the correction @ShoyuVanilla . I've updated the test expectations. I realized my previous local run was mistaken—the test is indeed sensitive to the query changes from this fix.

@ChayimFriedman2
Copy link
Contributor

I'm really not pleased with the solution here, and given that we need to get rid of the distinction between TyLoweringContext and InferenceContext anyway (as I said on Zulip), I would much prefer to solve this by that. That is larger work, of course.

@raushan728
Copy link
Contributor Author

Thanks @ChayimFriedman2. I just saw your messages on Zulip, let's discuss it further there."

@raushan728 raushan728 force-pushed the fix-dependent-generic-defaults branch from 56b9ec5 to 9f9d242 Compare February 12, 2026 12:23
@raushan728 raushan728 changed the title fix: resolve dependent generic defaults during type inference refactor: encapsulate TyLoweringContext usage in hir-ty Feb 12, 2026
@rustbot
Copy link
Collaborator

rustbot commented Mar 12, 2026

☔ The latest upstream changes (possibly #21804) made this pull request unmergeable. Please resolve the merge conflicts.

@raushan728 raushan728 closed this Mar 15, 2026
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants