fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-05-09 04:38:03 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

nak: fix clippy::needless_borrow warnings

Browse source 
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/27234>
This commit is contained in:
Karol Herbst 2024-01-24 12:50:14 +01:00 • committed by Marge Bot
parent 3d4bd73fa3
commit 1effb5262a
6 changed files with 144 additions and 146 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

118
src/nouveau/compiler/nak/encode_sm50.rs
View file

@ -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),
}

150
src/nouveau/compiler/nak/encode_sm70.rs
View file

@ -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"),
}

12
src/nouveau/compiler/nak/from_nir.rs
View file

@ -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;

2
src/nouveau/compiler/nak/ir.rs
View file

@ -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,
}
}
}

6
src/nouveau/compiler/nak/legalize.rs
View file

@ -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;
}

2
src/nouveau/compiler/nak/opt_copy_prop.rs
View file

@ -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,
};