<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Apr 13, 2015 at 1:11 PM, Matt Turner <span dir="ltr"><<a href="mailto:mattst88@gmail.com" target="_blank">mattst88@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">a * b is equivalent to -a * -b, and the previous code was failing at<br>
that.<br>
<br>
Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=89961" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=89961</a><br>
---<br>
 src/mesa/drivers/dri/i965/brw_fs_cse.cpp | 2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/src/mesa/drivers/dri/i965/brw_fs_cse.cpp b/src/mesa/drivers/dri/i965/brw_fs_cse.cpp<br>
index f2c4098..c1d0616 100644<br>
--- a/src/mesa/drivers/dri/i965/brw_fs_cse.cpp<br>
+++ b/src/mesa/drivers/dri/i965/brw_fs_cse.cpp<br>
@@ -145,7 +145,7 @@ operands_match(const fs_inst *a, const fs_inst *b, bool *negate)<br>
       xs[1].fixed_hw_reg.dw1.f = xs1_imm;<br>
       ys[1].fixed_hw_reg.dw1.f = ys1_imm;<br>
<br>
-      *negate = (xs0_negate + xs1_negate) != (ys0_negate + ys1_negate);<br>
+      *negate = (xs0_negate != xs1_negate) != (ys0_negate != ys1_negate);<br>
       return ret;<br>
    } else if (!a->is_commutative()) {<br>
       bool match = true;<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.0.5<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 class="gmail_extra">Reviewed-by: Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>></div></div>