nak: fix clippy::needless_borrow warnings

Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/27234>

6 files changed, 144 insertions, 146 deletions

diff --git a/src/nouveau/compiler/nak/encode_sm50.rs b/src/nouveau/compiler/nak/encode_sm50.rs
index d97a78255b2..6479b9ebb3f 100644
--- a/src/nouveau/compiler/nak/encode_sm50.rs
+++ b/src/nouveau/compiler/nak/encode_sm50.rs
@@ -296,7 +296,7 @@ impl SM50Instr {
             }
             SrcRef::CBuf(cbuf) => {
                 self.set_opcode(0x4ca0);
-                self.set_src_cb(20..39, &cbuf);
+                self.set_src_cb(20..39, cbuf);
             }
             src => panic!("Unsupported src type for SEL: {src}"),
         }
@@ -2045,64 +2045,64 @@ impl SM50Instr {
         let mut si = SM50Instr::new(sm);
 
         match &instr.op {
-            Op::FAdd(op) => si.encode_fadd(&op),
-            Op::FMnMx(op) => si.encode_fmnmx(&op),
-            Op::FMul(op) => si.encode_fmul(&op),
-            Op::FFma(op) => si.encode_ffma(&op),
-            Op::FSet(op) => si.encode_fset(&op),
-            Op::FSetP(op) => si.encode_fsetp(&op),
-            Op::FSwzAdd(op) => si.encode_fswzadd(&op),
-            Op::Rro(op) => si.encode_rro(&op),
-            Op::MuFu(op) => si.encode_mufu(&op),
-            Op::Flo(op) => si.encode_flo(&op),
-            Op::DAdd(op) => si.encode_dadd(&op),
-            Op::DFma(op) => si.encode_dfma(&op),
-            Op::DMnMx(op) => si.encode_dmnmx(&op),
-            Op::DMul(op) => si.encode_dmul(&op),
-            Op::DSetP(op) => si.encode_dsetp(&op),
-            Op::IAdd2(op) => si.encode_iadd2(&op),
-            Op::Mov(op) => si.encode_mov(&op),
-            Op::Sel(op) => si.encode_sel(&op),
-            Op::Shfl(op) => si.encode_shfl(&op),
-            Op::Vote(op) => si.encode_vote(&op),
-            Op::PSetP(op) => si.encode_psetp(&op),
-            Op::SuSt(op) => si.encode_sust(&op),
-            Op::S2R(op) => si.encode_s2r(&op),
-            Op::PopC(op) => si.encode_popc(&op),
-            Op::Prmt(op) => si.encode_prmt(&op),
-            Op::Ld(op) => si.encode_ld(&op),
-            Op::Ldc(op) => si.encode_ldc(&op),
-            Op::St(op) => si.encode_st(&op),
-            Op::Lop2(op) => si.encode_lop2(&op),
-            Op::Shf(op) => si.encode_shf(&op),
-            Op::Shl(op) => si.encode_shl(&op),
-            Op::Shr(op) => si.encode_shr(&op),
-            Op::F2F(op) => si.encode_f2f(&op),
-            Op::F2I(op) => si.encode_f2i(&op),
-            Op::I2F(op) => si.encode_i2f(&op),
-            Op::I2I(op) => si.encode_i2i(&op),
-            Op::IMad(op) => si.encode_imad(&op),
-            Op::IMul(op) => si.encode_imul(&op),
-            Op::IMnMx(op) => si.encode_imnmx(&op),
-            Op::ISetP(op) => si.encode_isetp(&op),
-            Op::Tex(op) => si.encode_tex(&op),
-            Op::Tld(op) => si.encode_tld(&op),
-            Op::Tld4(op) => si.encode_tld4(&op),
-            Op::Tmml(op) => si.encode_tmml(&op),
-            Op::Txd(op) => si.encode_txd(&op),
-            Op::Txq(op) => si.encode_txq(&op),
-            Op::Ipa(op) => si.encode_ipa(&op),
-            Op::ALd(op) => si.encode_ald(&op),
-            Op::ASt(op) => si.encode_ast(&op),
-            Op::MemBar(op) => si.encode_membar(&op),
-            Op::Atom(op) => si.encode_atom(&op),
-            Op::Bra(op) => si.encode_bra(&op, ip, labels),
-            Op::Exit(op) => si.encode_exit(&op),
-            Op::Bar(op) => si.encode_bar(&op),
-            Op::SuLd(op) => si.encode_suld(&op),
-            Op::SuAtom(op) => si.encode_suatom(&op),
-            Op::Kill(op) => si.encode_kill(&op),
-            Op::CS2R(op) => si.encode_cs2r(&op),
+            Op::FAdd(op) => si.encode_fadd(op),
+            Op::FMnMx(op) => si.encode_fmnmx(op),
+            Op::FMul(op) => si.encode_fmul(op),
+            Op::FFma(op) => si.encode_ffma(op),
+            Op::FSet(op) => si.encode_fset(op),
+            Op::FSetP(op) => si.encode_fsetp(op),
+            Op::FSwzAdd(op) => si.encode_fswzadd(op),
+            Op::Rro(op) => si.encode_rro(op),
+            Op::MuFu(op) => si.encode_mufu(op),
+            Op::Flo(op) => si.encode_flo(op),
+            Op::DAdd(op) => si.encode_dadd(op),
+            Op::DFma(op) => si.encode_dfma(op),
+            Op::DMnMx(op) => si.encode_dmnmx(op),
+            Op::DMul(op) => si.encode_dmul(op),
+            Op::DSetP(op) => si.encode_dsetp(op),
+            Op::IAdd2(op) => si.encode_iadd2(op),
+            Op::Mov(op) => si.encode_mov(op),
+            Op::Sel(op) => si.encode_sel(op),
+            Op::Shfl(op) => si.encode_shfl(op),
+            Op::Vote(op) => si.encode_vote(op),
+            Op::PSetP(op) => si.encode_psetp(op),
+            Op::SuSt(op) => si.encode_sust(op),
+            Op::S2R(op) => si.encode_s2r(op),
+            Op::PopC(op) => si.encode_popc(op),
+            Op::Prmt(op) => si.encode_prmt(op),
+            Op::Ld(op) => si.encode_ld(op),
+            Op::Ldc(op) => si.encode_ldc(op),
+            Op::St(op) => si.encode_st(op),
+            Op::Lop2(op) => si.encode_lop2(op),
+            Op::Shf(op) => si.encode_shf(op),
+            Op::Shl(op) => si.encode_shl(op),
+            Op::Shr(op) => si.encode_shr(op),
+            Op::F2F(op) => si.encode_f2f(op),
+            Op::F2I(op) => si.encode_f2i(op),
+            Op::I2F(op) => si.encode_i2f(op),
+            Op::I2I(op) => si.encode_i2i(op),
+            Op::IMad(op) => si.encode_imad(op),
+            Op::IMul(op) => si.encode_imul(op),
+            Op::IMnMx(op) => si.encode_imnmx(op),
+            Op::ISetP(op) => si.encode_isetp(op),
+            Op::Tex(op) => si.encode_tex(op),
+            Op::Tld(op) => si.encode_tld(op),
+            Op::Tld4(op) => si.encode_tld4(op),
+            Op::Tmml(op) => si.encode_tmml(op),
+            Op::Txd(op) => si.encode_txd(op),
+            Op::Txq(op) => si.encode_txq(op),
+            Op::Ipa(op) => si.encode_ipa(op),
+            Op::ALd(op) => si.encode_ald(op),
+            Op::ASt(op) => si.encode_ast(op),
+            Op::MemBar(op) => si.encode_membar(op),
+            Op::Atom(op) => si.encode_atom(op),
+            Op::Bra(op) => si.encode_bra(op, ip, labels),
+            Op::Exit(op) => si.encode_exit(op),
+            Op::Bar(op) => si.encode_bar(op),
+            Op::SuLd(op) => si.encode_suld(op),
+            Op::SuAtom(op) => si.encode_suatom(op),
+            Op::Kill(op) => si.encode_kill(op),
+            Op::CS2R(op) => si.encode_cs2r(op),
             Op::Nop(_) => si.encode_nop(),
             _ => panic!("Unhandled instruction {}", instr.op),
         }
diff --git a/src/nouveau/compiler/nak/encode_sm70.rs b/src/nouveau/compiler/nak/encode_sm70.rs
index 117c563900b..44b7d657dfa 100644
--- a/src/nouveau/compiler/nak/encode_sm70.rs
+++ b/src/nouveau/compiler/nak/encode_sm70.rs
@@ -343,7 +343,7 @@ impl SM70Instr {
                         6_u8 // form
                     }
                     ALUSrc::Imm32(imm) => {
-                        self.set_src_imm(32..64, &imm);
+                        self.set_src_imm(32..64, imm);
                         4_u8 // form
                     }
                     ALUSrc::CBuf(cb) => {
@@ -358,7 +358,7 @@ impl SM70Instr {
                 7_u8 // form
             }
             ALUSrc::Imm32(imm) => {
-                self.set_src_imm(32..64, &imm);
+                self.set_src_imm(32..64, imm);
                 self.set_alu_reg_src(64..72, 74, 75, &src1);
                 2_u8 // form
             }
@@ -1995,79 +1995,79 @@ impl SM70Instr {
         };
 
         match &instr.op {
-            Op::FAdd(op) => si.encode_fadd(&op),
-            Op::FFma(op) => si.encode_ffma(&op),
-            Op::FMnMx(op) => si.encode_fmnmx(&op),
-            Op::FMul(op) => si.encode_fmul(&op),
-            Op::FSet(op) => si.encode_fset(&op),
-            Op::FSetP(op) => si.encode_fsetp(&op),
-            Op::FSwzAdd(op) => si.encode_fswzadd(&op),
-            Op::DAdd(op) => si.encode_dadd(&op),
-            Op::DFma(op) => si.encode_dfma(&op),
-            Op::DMul(op) => si.encode_dmul(&op),
-            Op::DSetP(op) => si.encode_dsetp(&op),
-            Op::MuFu(op) => si.encode_mufu(&op),
-            Op::BMsk(op) => si.encode_bmsk(&op),
-            Op::BRev(op) => si.encode_brev(&op),
-            Op::Flo(op) => si.encode_flo(&op),
-            Op::IAbs(op) => si.encode_iabs(&op),
-            Op::IAdd3(op) => si.encode_iadd3(&op),
-            Op::IAdd3X(op) => si.encode_iadd3x(&op),
-            Op::IDp4(op) => si.encode_idp4(&op),
-            Op::IMad(op) => si.encode_imad(&op),
-            Op::IMad64(op) => si.encode_imad64(&op),
-            Op::IMnMx(op) => si.encode_imnmx(&op),
-            Op::ISetP(op) => si.encode_isetp(&op),
-            Op::Lop3(op) => si.encode_lop3(&op),
-            Op::PopC(op) => si.encode_popc(&op),
-            Op::Shf(op) => si.encode_shf(&op),
-            Op::F2F(op) => si.encode_f2f(&op),
-            Op::F2I(op) => si.encode_f2i(&op),
-            Op::I2F(op) => si.encode_i2f(&op),
-            Op::FRnd(op) => si.encode_frnd(&op),
-            Op::Mov(op) => si.encode_mov(&op),
-            Op::Prmt(op) => si.encode_prmt(&op),
-            Op::Sel(op) => si.encode_sel(&op),
-            Op::Shfl(op) => si.encode_shfl(&op),
-            Op::PLop3(op) => si.encode_plop3(&op),
-            Op::Tex(op) => si.encode_tex(&op),
-            Op::Tld(op) => si.encode_tld(&op),
-            Op::Tld4(op) => si.encode_tld4(&op),
-            Op::Tmml(op) => si.encode_tmml(&op),
-            Op::Txd(op) => si.encode_txd(&op),
-            Op::Txq(op) => si.encode_txq(&op),
-            Op::SuLd(op) => si.encode_suld(&op),
-            Op::SuSt(op) => si.encode_sust(&op),
-            Op::SuAtom(op) => si.encode_suatom(&op),
-            Op::Ld(op) => si.encode_ld(&op),
-            Op::Ldc(op) => si.encode_ldc(&op),
-            Op::St(op) => si.encode_st(&op),
-            Op::Atom(op) => si.encode_atom(&op),
-            Op::AL2P(op) => si.encode_al2p(&op),
-            Op::ALd(op) => si.encode_ald(&op),
-            Op::ASt(op) => si.encode_ast(&op),
-            Op::Ipa(op) => si.encode_ipa(&op),
-            Op::LdTram(op) => si.encode_ldtram(&op),
-            Op::CCtl(op) => si.encode_cctl(&op),
-            Op::MemBar(op) => si.encode_membar(&op),
-            Op::BClear(op) => si.encode_bclear(&op),
-            Op::BMov(op) => si.encode_bmov(&op),
-            Op::Break(op) => si.encode_break(&op),
-            Op::BSSy(op) => si.encode_bssy(&op, ip, labels),
-            Op::BSync(op) => si.encode_bsync(&op),
-            Op::Bra(op) => si.encode_bra(&op, ip, labels),
-            Op::Exit(op) => si.encode_exit(&op),
-            Op::WarpSync(op) => si.encode_warpsync(&op),
-            Op::Bar(op) => si.encode_bar(&op),
-            Op::CS2R(op) => si.encode_cs2r(&op),
-            Op::Isberd(op) => si.encode_isberd(&op),
-            Op::Kill(op) => si.encode_kill(&op),
-            Op::Nop(op) => si.encode_nop(&op),
-            Op::PixLd(op) => si.encode_pixld(&op),
-            Op::S2R(op) => si.encode_s2r(&op),
-            Op::Out(op) => si.encode_out(&op),
-            Op::OutFinal(op) => si.encode_out_final(&op),
-            Op::Vote(op) => si.encode_vote(&op),
+            Op::FAdd(op) => si.encode_fadd(op),
+            Op::FFma(op) => si.encode_ffma(op),
+            Op::FMnMx(op) => si.encode_fmnmx(op),
+            Op::FMul(op) => si.encode_fmul(op),
+            Op::FSet(op) => si.encode_fset(op),
+            Op::FSetP(op) => si.encode_fsetp(op),
+            Op::FSwzAdd(op) => si.encode_fswzadd(op),
+            Op::DAdd(op) => si.encode_dadd(op),
+            Op::DFma(op) => si.encode_dfma(op),
+            Op::DMul(op) => si.encode_dmul(op),
+            Op::DSetP(op) => si.encode_dsetp(op),
+            Op::MuFu(op) => si.encode_mufu(op),
+            Op::BMsk(op) => si.encode_bmsk(op),
+            Op::BRev(op) => si.encode_brev(op),
+            Op::Flo(op) => si.encode_flo(op),
+            Op::IAbs(op) => si.encode_iabs(op),
+            Op::IAdd3(op) => si.encode_iadd3(op),
+            Op::IAdd3X(op) => si.encode_iadd3x(op),
+            Op::IDp4(op) => si.encode_idp4(op),
+            Op::IMad(op) => si.encode_imad(op),
+            Op::IMad64(op) => si.encode_imad64(op),
+            Op::IMnMx(op) => si.encode_imnmx(op),
+            Op::ISetP(op) => si.encode_isetp(op),
+            Op::Lop3(op) => si.encode_lop3(op),
+            Op::PopC(op) => si.encode_popc(op),
+            Op::Shf(op) => si.encode_shf(op),
+            Op::F2F(op) => si.encode_f2f(op),
+            Op::F2I(op) => si.encode_f2i(op),
+            Op::I2F(op) => si.encode_i2f(op),
+            Op::FRnd(op) => si.encode_frnd(op),
+            Op::Mov(op) => si.encode_mov(op),
+            Op::Prmt(op) => si.encode_prmt(op),
+            Op::Sel(op) => si.encode_sel(op),
+            Op::Shfl(op) => si.encode_shfl(op),
+            Op::PLop3(op) => si.encode_plop3(op),
+            Op::Tex(op) => si.encode_tex(op),
+            Op::Tld(op) => si.encode_tld(op),
+            Op::Tld4(op) => si.encode_tld4(op),
+            Op::Tmml(op) => si.encode_tmml(op),
+            Op::Txd(op) => si.encode_txd(op),
+            Op::Txq(op) => si.encode_txq(op),
+            Op::SuLd(op) => si.encode_suld(op),
+            Op::SuSt(op) => si.encode_sust(op),
+            Op::SuAtom(op) => si.encode_suatom(op),
+            Op::Ld(op) => si.encode_ld(op),
+            Op::Ldc(op) => si.encode_ldc(op),
+            Op::St(op) => si.encode_st(op),
+            Op::Atom(op) => si.encode_atom(op),
+            Op::AL2P(op) => si.encode_al2p(op),
+            Op::ALd(op) => si.encode_ald(op),
+            Op::ASt(op) => si.encode_ast(op),
+            Op::Ipa(op) => si.encode_ipa(op),
+            Op::LdTram(op) => si.encode_ldtram(op),
+            Op::CCtl(op) => si.encode_cctl(op),
+            Op::MemBar(op) => si.encode_membar(op),
+            Op::BClear(op) => si.encode_bclear(op),
+            Op::BMov(op) => si.encode_bmov(op),
+            Op::Break(op) => si.encode_break(op),
+            Op::BSSy(op) => si.encode_bssy(op, ip, labels),
+            Op::BSync(op) => si.encode_bsync(op),
+            Op::Bra(op) => si.encode_bra(op, ip, labels),
+            Op::Exit(op) => si.encode_exit(op),
+            Op::WarpSync(op) => si.encode_warpsync(op),
+            Op::Bar(op) => si.encode_bar(op),
+            Op::CS2R(op) => si.encode_cs2r(op),
+            Op::Isberd(op) => si.encode_isberd(op),
+            Op::Kill(op) => si.encode_kill(op),
+            Op::Nop(op) => si.encode_nop(op),
+            Op::PixLd(op) => si.encode_pixld(op),
+            Op::S2R(op) => si.encode_s2r(op),
+            Op::Out(op) => si.encode_out(op),
+            Op::OutFinal(op) => si.encode_out_final(op),
+            Op::Vote(op) => si.encode_vote(op),
             _ => panic!("Unhandled instruction"),
         }
 
diff --git a/src/nouveau/compiler/nak/from_nir.rs b/src/nouveau/compiler/nak/from_nir.rs
index b8a241c8b41..1bf137e8791 100644
--- a/src/nouveau/compiler/nak/from_nir.rs
+++ b/src/nouveau/compiler/nak/from_nir.rs
@@ -305,9 +305,7 @@ impl<'a> ShaderFromNir<'a> {
     }
 
     fn get_src(&mut self, src: &nir_src) -> Src {
-        SSARef::try_from(self.get_ssa(&src.as_def()))
-            .unwrap()
-            .into()
+        SSARef::try_from(self.get_ssa(src.as_def())).unwrap().into()
     }
 
     fn get_io_addr_offset(
@@ -453,7 +451,7 @@ impl<'a> ShaderFromNir<'a> {
         for (i, alu_src) in alu.srcs_as_slice().iter().enumerate() {
             let bit_size = alu_src.src.bit_size();
             let comps = alu.src_components(i.try_into().unwrap());
-            let ssa = self.get_ssa(&alu_src.src.as_def());
+            let ssa = self.get_ssa(alu_src.src.as_def());
 
             match bit_size {
                 1 => {
@@ -1636,7 +1634,7 @@ impl<'a> ShaderFromNir<'a> {
                 di += 1;
             }
         }
-        self.set_ssa(&tex.def.as_def(), nir_dst);
+        self.set_ssa(tex.def.as_def(), nir_dst);
     }
 
     fn get_atomic_type(&self, intrin: &nir_intrinsic_instr) -> AtomType {
@@ -2022,7 +2020,7 @@ impl<'a> ShaderFromNir<'a> {
                 } else {
                     panic!("OpKill is only available in fragment shaders");
                 }
-                let cond = self.get_ssa(&srcs[0].as_def())[0];
+                let cond = self.get_ssa(srcs[0].as_def())[0];
                 b.predicate(cond.into()).push_op(OpKill {});
 
                 if intrin.intrinsic == nir_intrinsic_terminate_if {
@@ -2908,7 +2906,7 @@ impl<'a> ShaderFromNir<'a> {
                 target: self.get_block_label(ni.first_else_block()),
             });
 
-            let cond = self.get_ssa(&ni.condition.as_def())[0];
+            let cond = self.get_ssa(ni.condition.as_def())[0];
             bra.pred = cond.into();
             // This is the branch to jump to the else
             bra.pred.pred_inv = true;
diff --git a/src/nouveau/compiler/nak/ir.rs b/src/nouveau/compiler/nak/ir.rs
index 7ff57976f07..5ee40eda4b2 100644
--- a/src/nouveau/compiler/nak/ir.rs
+++ b/src/nouveau/compiler/nak/ir.rs
@@ -1313,7 +1313,7 @@ impl Index<usize> for SrcTypeList {
     fn index(&self, idx: usize) -> &SrcType {
         match self {
             SrcTypeList::Array(arr) => &arr[idx],
-            SrcTypeList::Uniform(typ) => &typ,
+            SrcTypeList::Uniform(typ) => typ,
         }
     }
 }
diff --git a/src/nouveau/compiler/nak/legalize.rs b/src/nouveau/compiler/nak/legalize.rs
index 7471e48b4ea..a7fa15e9ccd 100644
--- a/src/nouveau/compiler/nak/legalize.rs
+++ b/src/nouveau/compiler/nak/legalize.rs
@@ -96,7 +96,7 @@ fn copy_alu_src_if_not_reg(
     src: &mut Src,
     src_type: SrcType,
 ) {
-    if !src_is_reg(&src) {
+    if !src_is_reg(src) {
         copy_alu_src(b, src, src_type);
     }
 }
@@ -107,7 +107,7 @@ fn copy_alu_src_if_both_not_reg(
     src2: &mut Src,
     src_type: SrcType,
 ) {
-    if !src_is_reg(&src1) && !src_is_reg(&src2) {
+    if !src_is_reg(src1) && !src_is_reg(src2) {
         copy_alu_src(b, src2, src_type);
     }
 }
@@ -721,7 +721,7 @@ fn legalize_instr(
             // If the same vector shows up twice in one instruction, that's
             // okay. Just make it look the same as the previous source we
             // fixed up.
-            if let Some(new_vec) = vec_src_map.get(&vec) {
+            if let Some(new_vec) = vec_src_map.get(vec) {
                 src.src_ref = (*new_vec).into();
                 continue;
             }
diff --git a/src/nouveau/compiler/nak/opt_copy_prop.rs b/src/nouveau/compiler/nak/opt_copy_prop.rs
index 6ba0b830216..06933934601 100644
--- a/src/nouveau/compiler/nak/opt_copy_prop.rs
+++ b/src/nouveau/compiler/nak/opt_copy_prop.rs
@@ -73,7 +73,7 @@ impl CopyPropPass {
                 _ => return,
             };
 
-            let entry = match self.get_copy(&src_ssa) {
+            let entry = match self.get_copy(src_ssa) {
                 Some(e) => e,
                 None => return,
             };