From 8d9155e34df49dcf2968fd56bd88ef5bb2331fe8 Mon Sep 17 00:00:00 2001 From: Caio Oliveira Date: Thu, 13 Mar 2025 15:48:37 -0700 Subject: [PATCH] brw: Clean up saturate propagation after non-defs version removal Remove now unused analysis and no need to walk blocks in reverse after the non-defs version of the pass was removed. Reviewed-by: Ian Romanick Part-of: --- src/intel/compiler/brw_opt_saturate_propagation.cpp | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/src/intel/compiler/brw_opt_saturate_propagation.cpp b/src/intel/compiler/brw_opt_saturate_propagation.cpp index 69959c8f69c..27cc763d8d1 100644 --- a/src/intel/compiler/brw_opt_saturate_propagation.cpp +++ b/src/intel/compiler/brw_opt_saturate_propagation.cpp @@ -87,12 +87,11 @@ propagate_sat(brw_inst *inst, brw_inst *scan_inst) static bool opt_saturate_propagation_local(brw_shader &s, - const brw_ip_ranges &ips, bblock_t *block) { bool progress = false; - foreach_inst_in_block_reverse(brw_inst, inst, block) { + foreach_inst_in_block(brw_inst, inst, block) { if (inst->opcode != BRW_OPCODE_MOV || !inst->saturate || inst->dst.file != VGRF || @@ -133,10 +132,8 @@ brw_opt_saturate_propagation(brw_shader &s) { bool progress = false; - const brw_ip_ranges &ips = s.ip_ranges_analysis.require(); - foreach_block (block, s.cfg) { - progress = opt_saturate_propagation_local(s, ips, block) || progress; + progress = opt_saturate_propagation_local(s, block) || progress; } if (progress)