From bc0179d1d906c3643493774f476969db4ca7cdd5 Mon Sep 17 00:00:00 2001 From: Faith Ekstrand Date: Sat, 15 Jul 2023 00:15:12 -0500 Subject: [PATCH] nv50/ir: Support vector movs nir_opt_mov and nir_op_vecN are only the same if the mov is only a single component. Otherwise the vec loop will try to access src[c] where c > 0 which breaks for nir_op_mov. It's uncommon but scalar back-ends can see vector movs so we need to handle this correctly. Fixes: 6513c675ad31 ("nv50/ir/nir: implement nir_alu_instr handling") Reviewed-by: Karol Herbst Reviewed-by: M Henning Part-of: (cherry picked from commit 259ba104f79f9f653130865b21bccfab62dd4829) --- .pick_status.json | 2 +- src/nouveau/codegen/nv50_ir_from_nir.cpp | 8 +++++++- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/.pick_status.json b/.pick_status.json index 26da28d777c..9d2ae89346a 100644 --- a/.pick_status.json +++ b/.pick_status.json @@ -22,7 +22,7 @@ "description": "nv50/ir: Support vector movs", "nominated": true, "nomination_type": 1, - "resolution": 0, + "resolution": 1, "main_sha": null, "because_sha": "6513c675ad31d192265a8286d712e8ae81c9f305" }, diff --git a/src/nouveau/codegen/nv50_ir_from_nir.cpp b/src/nouveau/codegen/nv50_ir_from_nir.cpp index a40a8e74905..abc9d0e9a82 100644 --- a/src/nouveau/codegen/nv50_ir_from_nir.cpp +++ b/src/nouveau/codegen/nv50_ir_from_nir.cpp @@ -2723,7 +2723,13 @@ Converter::visit(nir_alu_instr *insn) i->sType = sTypes[0]; break; } - case nir_op_mov: + case nir_op_mov: { + LValues &newDefs = convert(&insn->dest); + for (LValues::size_type c = 0u; c < newDefs.size(); ++c) { + mkMov(newDefs[c], getSrc(&insn->src[0], c), dType); + } + break; + } case nir_op_vec2: case nir_op_vec3: case nir_op_vec4: