Rollup merge of #114827 - compiler-errors:next-solver-dyn-safe-candidates, r=lcnr

Only consider object candidates for object-safe dyn types in new solver

We apparently allow this per RFC2027 💀

r? lcnr
This commit is contained in:
Guillaume Gomez
2023-08-15 14:29:48 +02:00
committed by GitHub
2 changed files with 7 additions and 0 deletions

View File

@@ -826,6 +826,11 @@ impl<'tcx> EvalCtxt<'_, 'tcx> {
ty::Dynamic(bounds, ..) => bounds,
};
// Do not consider built-in object impls for non-object-safe types.
if bounds.principal_def_id().is_some_and(|def_id| !tcx.check_is_object_safe(def_id)) {
return;
}
// Consider all of the auto-trait and projection bounds, which don't
// need to be recorded as a `BuiltinImplSource::Object` since they don't
// really have a vtable base...