Module: Mesa Branch: main Commit: b23b94fbc9201368398823ccdc633cf1fa60127c URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=b23b94fbc9201368398823ccdc633cf1fa60127c
Author: Karol Herbst <[email protected]> Date: Thu Sep 1 21:45:20 2022 +0200 nv50/ir: fix OP_UNION resolving when used for vector values When an OP_UNION def takes part in a vector source e.g. for a tex instruction we failed to clean up the OP_UNION instruction as rep() points towards the coalesced value instead. This fixes a regression on nv50 moving to NIR, but also potentially issues with nvc0. The main reason this is common in nv50 is, that we lower OP_SLCT to a set, predicated movs and a union. Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/6406 Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/7117 Cc: mesa-stable Signed-off-by: Karol Herbst <[email protected]> Reviewed-by: M Henning <[email protected]> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/18377> --- src/nouveau/codegen/nv50_ir_peephole.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/nouveau/codegen/nv50_ir_peephole.cpp b/src/nouveau/codegen/nv50_ir_peephole.cpp index 4e2b3e6f52c..9088e614f7e 100644 --- a/src/nouveau/codegen/nv50_ir_peephole.cpp +++ b/src/nouveau/codegen/nv50_ir_peephole.cpp @@ -53,7 +53,7 @@ Instruction::isNop() const if (!getDef(0)->equals(getSrc(0))) return false; if (op == OP_UNION) - if (!def(0).rep()->equals(getSrc(1))) + if (!getDef(0)->equals(getSrc(1))) return false; return true; }
