Preserve placeholder ambiguity during normalization - #159964
Conversation
|
r? @chenyukang rustbot has assigned @chenyukang. Use Why was this reviewer chosen?The reviewer was selected based on:
|
|
@rustbot reroll |
|
r? types |
|
I immediately get this change, can you provide some comment/PR description for what went wrong and why this is correct r? BoxyUwU |
|
Failed to set assignee to
|
The problem was that the certainty of the nested goals was combined with the placeholder certainty and then dropped in the normalization path. It is fine to ignore ambiguity from nested goals there since those goals are returned to the caller. placeholder ambiguity is different because it is not reflected in the returned goals |
|
r? lcnr makes sense @bors r+ rollup |
…, r=lcnr Preserve placeholder ambiguity during normalization Fixes rust-lang#159896
|
This pull request was unapproved. This PR was contained in a rollup (#160128), which was unapproved. |
Signed-off-by: Amirhossein Akhlaghpour <m9.akhlaghpoor@gmail.com>
Signed-off-by: Amirhossein Akhlaghpour <m9.akhlaghpoor@gmail.com>
ba4e2f6 to
a519b52
Compare
|
This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
|
Requested reviewer is already assigned to this pull request. Please choose another assignee. |
|
Failed to set assignee to
|
Fixes #159896