Don't translate vtable methods with Self: Sized bounds.

[eddyb]

Fixes#34416.

[rust-highfive]

r? @nikomatsakis

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

[michaelwoerister]

This looks good to me but I don't feel competent enough to r+ this on my own.

[michaelwoerister]

cc @rust-lang/compiler

[Aatch]

This looks like it could just be self.object_safety_violation_for_method(...).is_none(), since it's basically identical to the function above it.

[eddyb]

@Aatch The effect is opposite: Self: Sized methods never have object-safety implications, so .is_none() would return true, but you want false because they're not in the vtable.

[Aatch]

@eddyb oh right, I missed that. Seems fine then.

[Aatch]

@bors r+

[bors]

📌 Commit 096ae80 has been approved by Aatch

[bors]

⌛ Testing commit 096ae80 with merge b42884f...

[bors]

☀️ Test successful - auto-linux-32-nopt-t, auto-linux-32-opt, auto-linux-64-cargotest, auto-linux-64-cross-freebsd, auto-linux-64-debug-opt, auto-linux-64-nopt-t, auto-linux-64-opt, auto-linux-64-opt-mir, auto-linux-64-opt-rustbuild, auto-linux-64-x-android-t, auto-linux-cross-opt, auto-linux-musl-64-opt, auto-mac-32-opt, auto-mac-64-opt, auto-mac-64-opt-rustbuild, auto-mac-cross-ios-opt, auto-win-gnu-32-opt, auto-win-gnu-32-opt-rustbuild, auto-win-gnu-64-opt, auto-win-msvc-32-opt, auto-win-msvc-64-cargotest, auto-win-msvc-64-opt, auto-win-msvc-64-opt-mir, auto-win-msvc-64-opt-rustbuild

[alexcrichton]

Discussed with compiler team in email and sentiment was that this is small enough and important enough to backport, accepting for backport.