fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-05-08 04:48:08 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

intel/fs: Don't continue fixed point iteration just because liveout changes

Browse source 
Unless the change in liveout also causes livein to change, updates to
liveout cannot have any global effect. Changes to livein already flag
additional interation.

I had additional changes in this area that didn't pan out. While working
on those change, I was a little confused about this bit of code. It's
unnecessary, so it's better to delete it.

Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/25091>
This commit is contained in:
Ian Romanick 2023-03-31 18:13:57 -07:00 • committed by Marge Bot
parent 385f01c89b
commit e488b46419
1 changed files with 2 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/intel/compiler/brw_fs_live_variables.cpp
View file

@ -192,17 +192,13 @@ fs_live_variables::compute_live_variables()
BITSET_WORD new_liveout = (child_bd->livein[i] &
~bd->liveout[i]);
new_liveout &= bd->defout[i]; /* Screen off uses with no reaching def */
if (new_liveout) {
if (new_liveout)
bd->liveout[i] |= new_liveout;
cont = true;
}
}
BITSET_WORD new_liveout = (child_bd->flag_livein[0] &
~bd->flag_liveout[0]);
if (new_liveout) {
if (new_liveout)
bd->flag_liveout[0] |= new_liveout;
cont = true;
}
}
/* Update livein */