Module: Mesa Branch: master Commit: 5c770ba919723bb1c5d168717e814cca4a68e877 URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=5c770ba919723bb1c5d168717e814cca4a68e877
Author: Matt Turner <matts...@gmail.com> Date: Thu Mar 27 11:35:49 2014 -0700 i965/fs: Remove left-over 'removed' variable. I think this was used for coalescing out partly dead large virtual registers, but the patch that enabled that caused regressions and didn't make it upstream. Reviewed-by: Eric Anholt <e...@anholt.net> --- .../drivers/dri/i965/brw_fs_register_coalesce.cpp | 21 ++++++++------------ 1 file changed, 8 insertions(+), 13 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs_register_coalesce.cpp b/src/mesa/drivers/dri/i965/brw_fs_register_coalesce.cpp index c71fd47..6e30d16 100644 --- a/src/mesa/drivers/dri/i965/brw_fs_register_coalesce.cpp +++ b/src/mesa/drivers/dri/i965/brw_fs_register_coalesce.cpp @@ -169,11 +169,8 @@ fs_visitor::register_coalesce() if (!can_coalesce) continue; - bool removed = false; for (int i = 0; i < src_size; i++) { if (mov[i]) { - removed = true; - mov[i]->opcode = BRW_OPCODE_NOP; mov[i]->conditional_mod = BRW_CONDITIONAL_NONE; mov[i]->dst = reg_undef; @@ -206,17 +203,15 @@ fs_visitor::register_coalesce() } } - if (removed) { - for (int i = 0; i < src_size; i++) { - live_intervals->start[var_to[i]] = - MIN2(live_intervals->start[var_to[i]], - live_intervals->start[var_from[i]]); - live_intervals->end[var_to[i]] = - MAX2(live_intervals->end[var_to[i]], - live_intervals->end[var_from[i]]); - reg_from = -1; - } + for (int i = 0; i < src_size; i++) { + live_intervals->start[var_to[i]] = + MIN2(live_intervals->start[var_to[i]], + live_intervals->start[var_from[i]]); + live_intervals->end[var_to[i]] = + MAX2(live_intervals->end[var_to[i]], + live_intervals->end[var_from[i]]); } + reg_from = -1; } foreach_list_safe(node, &this->instructions) { _______________________________________________ mesa-commit mailing list mesa-commit@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-commit