From aae67ab678adcb839dfd1e0c9c7a2dd86ebce38b Mon Sep 17 00:00:00 2001 From: Mel Henning Date: Fri, 30 May 2025 20:29:04 -0400 Subject: [PATCH] nak: Don't swap f2fp sources in legalize The order of these is important. Fixes: e19871bd6ad ("nak: Use F2FP for nir_op_pack_half_2x16_split on SM86+") Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/12717 Part-of: --- src/nouveau/compiler/nak/sm70_encode.rs | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/nouveau/compiler/nak/sm70_encode.rs b/src/nouveau/compiler/nak/sm70_encode.rs index 32b743a121f..bf850179035 100644 --- a/src/nouveau/compiler/nak/sm70_encode.rs +++ b/src/nouveau/compiler/nak/sm70_encode.rs @@ -1915,9 +1915,7 @@ impl SM70Op for OpF2F { impl SM70Op for OpF2FP { fn legalize(&mut self, b: &mut LegalizeBuilder) { let gpr = op_gpr(self); - let [src0, src1] = &mut self.srcs; - swap_srcs_if_not_reg(src0, src1, gpr); - + let [src0, _src1] = &mut self.srcs; b.copy_alu_src_if_not_reg(src0, gpr, SrcType::ALU); }