From: Timur Kristóf <timur.kristof@gmail.com>
Date: Tue, 19 Nov 2019 12:38:34 +0000 (+0100)
Subject: aco: Don't skip combine_instruction when definitions[1] is used.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=1c44129db30db70dc6a0e1906b5a8efa7f8d00cf;p=mesa.git

aco: Don't skip combine_instruction when definitions[1] is used.

Signed-off-by: Timur Kristóf <timur.kristof@gmail.com>
Reviewed-by: Daniel Schürmann <daniel@schuermann.dev>
---

diff --git a/src/amd/compiler/aco_optimizer.cpp b/src/amd/compiler/aco_optimizer.cpp
index bf075b6ffab..84c5e4b78ab 100644
--- a/src/amd/compiler/aco_optimizer.cpp
+++ b/src/amd/compiler/aco_optimizer.cpp
@@ -2150,7 +2150,7 @@ bool apply_omod_clamp(opt_ctx &ctx, Block& block, aco_ptr<Instruction>& instr)
 
 void combine_instruction(opt_ctx &ctx, Block& block, aco_ptr<Instruction>& instr)
 {
-   if (instr->definitions.empty() || !ctx.uses[instr->definitions[0].tempId()])
+   if (instr->definitions.empty() || is_dead(ctx.uses, instr.get()))
       return;
 
    if (instr->isVALU()) {