<div dir="ltr">On 12 August 2013 13:11, Kenneth Graunke <span dir="ltr"><<a href="mailto:kenneth@whitecape.org" target="_blank">kenneth@whitecape.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Since we start with an overestimation of livein (0xffffffff), successive<br>
steps may should actually take away values.  This means we can't simply<br></blockquote><div><br></div><div>Is "may should" a Southernism?<br><br></div><div>In any case,<br><br>Reviewed-by: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>><br>
</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
OR in new liveout values; we need to recompute it from scratch at each<br>
iteration of the fixed-point algorithm.<br>
<br>
Signed-off-by: Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>><br>
---<br>
 src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp | 2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp b/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp<br>
index f5c8e4a..9522649 100644<br>
--- a/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp<br>
+++ b/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp<br>
@@ -219,7 +219,7 @@ fs_copy_prop_dataflow::run()<br>
          for (int i = 0; i < bitset_words; i++) {<br>
             const BITSET_WORD old_liveout = bd[b].liveout[i];<br>
<br>
-            bd[b].liveout[i] |=<br>
+            bd[b].liveout[i] =<br>
                bd[b].copy[i] | (bd[b].livein[i] & ~bd[b].kill[i]);<br>
<br>
             if (old_liveout != bd[b].liveout[i])<br>
<span class="HOEnZb"><font color="#888888">--<br>
1.8.3.4<br>
<br>
_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</font></span></blockquote></div><br></div></div>