From 618fc13db78b4b8a270daf8fe719910906e49922 Mon Sep 17 00:00:00 2001 From: Gert Wollny Date: Mon, 31 Jul 2023 09:36:18 +0200 Subject: [PATCH] r600/sfn: Don't try to propagate to vec4 with more than one use Otherwise we end up with weird dependencies that make the scheduler choke. Fixes: 18a8d148d862d4a0f5b83f4385f00cf8a4a71b6d r600/sfn: Cleanup copy-prop into vec4 source values Signed-off-by: Gert Wollny Part-of: --- src/gallium/drivers/r600/sfn/sfn_optimizer.cpp | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/gallium/drivers/r600/sfn/sfn_optimizer.cpp b/src/gallium/drivers/r600/sfn/sfn_optimizer.cpp index c2f6183..4cd969c 100644 --- a/src/gallium/drivers/r600/sfn/sfn_optimizer.cpp +++ b/src/gallium/drivers/r600/sfn/sfn_optimizer.cpp @@ -510,6 +510,9 @@ CopyPropFwdVisitor::propagate_to(RegisterVec4& value, Instr *instr) if (value[i]->parents().empty()) return; + if (value[i]->uses().size() > 1) + return; + assert(value[i]->parents().size() == 1); parents[i] = (*value[i]->parents().begin())->as_alu(); @@ -518,6 +521,7 @@ CopyPropFwdVisitor::propagate_to(RegisterVec4& value, Instr *instr) if (!parents[i]) return; + if ((parents[i]->opcode() != op1_mov) || parents[i]->has_source_mod(0, AluInstr::mod_neg) || parents[i]->has_source_mod(0, AluInstr::mod_abs) || -- 2.7.4