Skip to content

Dont bail in error predicate unless self ty is error in new solver#150453

Open
ChayimFriedman2 wants to merge 1 commit intorust-lang:mainfrom
ChayimFriedman2:less-err-in-solver
Open

Dont bail in error predicate unless self ty is error in new solver#150453
ChayimFriedman2 wants to merge 1 commit intorust-lang:mainfrom
ChayimFriedman2:less-err-in-solver

Conversation

@ChayimFriedman2
Copy link
Copy Markdown
Contributor

Continuation of #146602.

r? lcnr

@lcnr #146602 (comment)

I think we should move this out of assemble_builtin_impl_candidate to avoid ambiguity between e.g. blanket impls and the builtin impl candidate 🤔

Where do you want to put it?

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. WG-trait-system-refactor The Rustc Trait System Refactor Initiative (-Znext-solver) labels Dec 28, 2025
@ChayimFriedman2 ChayimFriedman2 changed the title Dont bail in error predicate unless self ty is error Dont bail in error predicate unless self ty is error in new solver Dec 28, 2025
@rust-log-analyzer

This comment has been minimized.

@apiraino
Copy link
Copy Markdown
Contributor

I think there's a pending comment, therefore switching to waiting on author. Feel free to request a review with @rustbot ready, thanks!

@rustbot author

@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 12, 2026
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Mar 12, 2026

Reminder, once the PR becomes ready for a review, use @rustbot ready.

@rust-bors

This comment has been minimized.

@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Apr 15, 2026

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.

@rust-log-analyzer

This comment has been minimized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. WG-trait-system-refactor The Rustc Trait System Refactor Initiative (-Znext-solver)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants