AlexIndustrial/mesa
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

brw: Move defs analysis back to its place in saturate propagation

Browse Source 
The premise of the change was wrong: the case where the defs analysis
was required was rare and requiring the analysis inside just the
case we care was being used for another analysis too.  So for now,
the change doesn't really helps.  I'll revisit this whole pass later on.

This backs out commit 6e19215810.

Acked-by: Kenneth Graunke <kenneth@whitecape.org>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/34039>
This commit is contained in:
Caio Oliveira
2025-03-13 00:26:43 -07:00
committed by Marge Bot
parent 6a7f174fb3
commit ce71f2badd
1 changed files with 3 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/intel/compiler/brw_opt_saturate_propagation.cpp
+3 -6
View File
@@ -86,9 +86,7 @@ propagate_sat(brw_inst *inst, brw_inst *scan_inst)
}
static bool
opt_saturate_propagation_local(brw_shader &s,
const brw_def_analysis &defs,
bblock_t *block)
opt_saturate_propagation_local(brw_shader &s, bblock_t *block)
{
bool progress = false;
int ip = block->end_ip + 1;
@@ -104,6 +102,7 @@ opt_saturate_propagation_local(brw_shader &s,
inst->src[0].abs)
continue;
const brw_def_analysis &defs = s.def_analysis.require();
brw_inst *def = defs.get(inst->src[0]);
if (def != NULL) {
@@ -193,10 +192,8 @@ brw_opt_saturate_propagation(brw_shader &s)
{
bool progress = false;
const brw_def_analysis &defs = s.def_analysis.require();
foreach_block (block, s.cfg) {
progress = opt_saturate_propagation_local(s, defs, block) || progress;
progress = opt_saturate_propagation_local(s, block) || progress;
}
if (progress)