Mesa (main): aco: fix s_setreg hazards

[GitLab Mirror]

Module: Mesa
Branch: main
Commit: ed3ca5b7812f39ca0072d871d785efe0e34bbb82
URL:    
http://cgit.freedesktop.org/mesa/mesa/commit/?id=ed3ca5b7812f39ca0072d871d785efe0e34bbb82

Author: Rhys Perry <pendingchao...@gmail.com>
Date:   Fri Sep 22 16:45:06 2023 +0100

aco: fix s_setreg hazards

s_setreg doesn't have any definitions.

Signed-off-by: Rhys Perry <pendingchao...@gmail.com>
Reviewed-by: Daniel Schürmann <dan...@schuermann.dev>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/25374>

---

 src/amd/compiler/aco_insert_NOPs.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/amd/compiler/aco_insert_NOPs.cpp 
b/src/amd/compiler/aco_insert_NOPs.cpp
index a03b90000e9..6290ac16745 100644
--- a/src/amd/compiler/aco_insert_NOPs.cpp
+++ b/src/amd/compiler/aco_insert_NOPs.cpp
@@ -681,7 +681,7 @@ handle_instruction_gfx6(State& state, NOP_ctx_gfx6& ctx, 
aco_ptr<Instruction>& i
             }
          }
       }
-   } else if (instr->isSALU() && !instr->definitions.empty()) {
+   } else if (instr->isSALU()) {
       if (!instr->definitions.empty()) {
          /* all other definitions should be SCC */
          Definition def = instr->definitions[0];