summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRhys Perry <[email protected]>2019-10-29 11:19:39 +0000
committerDaniel Schürmann <[email protected]>2019-10-30 18:13:53 +0000
commitd4684a294baf636d1bf71b51537bf229c31ec085 (patch)
treecfeacb0abec919a39798748c6d50efcecdaebe2d
parent12d3b11908e0a033be4f3e8b92ae4a36ce910f36 (diff)
aco: a couple loop handling fixes for GFX10 hazard pass
It was joining from the wrong blocks and block.kind is a bitmask instead of an enum. Reviewed-By: Timur Kristóf <[email protected]>
-rw-r--r--src/amd/compiler/aco_insert_NOPs.cpp6
1 files changed, 3 insertions, 3 deletions
diff --git a/src/amd/compiler/aco_insert_NOPs.cpp b/src/amd/compiler/aco_insert_NOPs.cpp
index 5296f6ac045..2c69407b808 100644
--- a/src/amd/compiler/aco_insert_NOPs.cpp
+++ b/src/amd/compiler/aco_insert_NOPs.cpp
@@ -560,13 +560,13 @@ void mitigate_hazards_gfx10(Program *program)
Block& block = program->blocks[i];
NOP_ctx_gfx10 &ctx = all_ctx[i];
- if (block.kind == block_kind_loop_header) {
+ if (block.kind & block_kind_loop_header) {
loop_header_indices.push(i);
- } else if (block.kind == block_kind_loop_exit) {
+ } else if (block.kind & block_kind_loop_exit) {
/* Go through the whole loop again */
for (unsigned idx = loop_header_indices.top(); idx < i; idx++) {
NOP_ctx_gfx10 loop_block_ctx;
- for (unsigned b : block.linear_preds)
+ for (unsigned b : program->blocks[idx].linear_preds)
loop_block_ctx.join(all_ctx[b]);
handle_block_gfx10(loop_block_ctx, program->blocks[idx]);