Always reborrow mutable reference receiver in methods

Author: HKalbasi

crates/hir-ty/src/method_resolution.rs

@@ -896,6 +896,10 @@ pub fn iterate_method_candidates_dyn(
             let deref_chain = autoderef_method_receiver(&mut table, ty);
 
             let result = deref_chain.into_iter().try_for_each(|(receiver_ty, adj)| {
+                if matches!(receiver_ty.value.kind(Interner), TyKind::Ref(Mutability::Mut, _, _)) {
+                    // We should prefer reborrow of mutable references instead of moving them.
+                    return ControlFlow::Continue(());
+                }
                 iterate_method_candidates_with_autoref(
                     &receiver_ty,
                     adj,

crates/ide/src/inlay_hints/adjustment.rs

@@ -322,6 +322,8 @@ fn main() {
    //^^^^^^^^^^^&
    //^^^^^^^^^^^*
     (&mut Struct).by_ref_mut();
+   //^^^^^^^^^^^&mut $
+   //^^^^^^^^^^^*
 
     // Check that block-like expressions don't duplicate hints
     let _: &mut [u32] = (&mut []);
@@ -425,6 +427,10 @@ fn main() {
    //^^^^^^^^^^^.*
    //^^^^^^^^^^^.&
     (&mut Struct).by_ref_mut();
+   //^^^^^^^^^^^(
+   //^^^^^^^^^^^)
+   //^^^^^^^^^^^.*
+   //^^^^^^^^^^^.&mut
 
     // Check that block-like expressions don't duplicate hints
     let _: &mut [u32] = (&mut []);