Don't ICE if method receiver fails to unify with arbitrary_self_types

[compiler-errors]

Consider:

struct Foo(u32);
impl Foo {
    fn get<R: Deref<Target=Self>>(self: R) -> u32 {
        self.0
    }
}

fn main() {
    let mut foo = Foo(1);
    foo.get::<&Foo>();
}

The problem here is that with arbitrary_self_types, we're allowed to have a method receiver that mentions generics from the method itself (fn get<R: Deref<Target=Self>>(self: R)). Since we don't actually take into account the user-written turbofish generics when doing method lookup (nor do we check that method predicates hold), method probing will happily infer R = Foo during the probe. When we later confirm the method, we do use the turbofish'd subst and instead now have that R = &Foo. This doesn't unify with the self type we chose during the probe, causing an ICE.

Getting this to work correctly will be difficult. Specifically, we'll need to actually pass in the turbofish generics for the method being probed for and check that the self type unifies considering those generics. This seems like a lot of work, and I'm not actually familiar with the restrictions originally called out for #![feature(arbitrary_self_types)], but I think we should probably instead just deny having receivers that mention (type/const) generics that come from the method itself.

But I mostly just want to turn this ICE into an error, so I'll leave that up for later PRs.

Fixes #111838

[rustbot]

r? @WaffleLapkin

(rustbot has picked a reviewer for you, use r? to override)

[WaffleLapkin]

as an ICE fix this looks fine

[WaffleLapkin]

@bors r+

[bors]

📌 Commit 05c5caa has been approved by WaffleLapkin

It is now in the queue for this repository.