diff options
author | Kenneth Graunke <kenneth@whitecape.org> | 2013-08-09 18:27:22 -0700 |
---|---|---|
committer | Kenneth Graunke <kenneth@whitecape.org> | 2013-08-19 11:29:24 -0700 |
commit | a291c59bbae7d9d96487a984f81a298a1fd71389 (patch) | |
tree | e11033e4f77cd3554ea174f73d8fbda636fcaf35 /src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp | |
parent | 597efd2b67d1afb8a95be38145c4f977ed36b672 (diff) | |
download | external_mesa3d-a291c59bbae7d9d96487a984f81a298a1fd71389.zip external_mesa3d-a291c59bbae7d9d96487a984f81a298a1fd71389.tar.gz external_mesa3d-a291c59bbae7d9d96487a984f81a298a1fd71389.tar.bz2 |
i965/fs: Simplify liveout calculation.
Excluding the existing liveout bits is a deviation from the textbook
algorithm. The reason for doing so was to determine if the value
changed, which means the fixed-point algorithm needs to run for another
iteration.
The simpler way to do that is to save the value from step (N-1) and
compare it to the new value at step N.
Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Paul Berry <stereotype441@gmail.com>
Diffstat (limited to 'src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp')
-rw-r--r-- | src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp b/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp index 424d7b5..0078c87 100644 --- a/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp +++ b/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp @@ -186,13 +186,12 @@ fs_copy_prop_dataflow::run() /* Update liveout for all blocks. */ for (int b = 0; b < cfg->num_blocks; b++) { for (int i = 0; i < bitset_words; i++) { - BITSET_WORD new_liveout = (bd[b].livein[i] & - ~bd[b].kill[i] & - ~bd[b].liveout[i]); - if (new_liveout) { - bd[b].liveout[i] |= new_liveout; + const BITSET_WORD old_liveout = bd[b].liveout[i]; + + bd[b].liveout[i] |= bd[b].livein[i] & ~bd[b].kill[i]; + + if (old_liveout != bd[b].liveout[i]) progress = true; - } } } |