<div dir="ltr">On 20 December 2013 06:39, Topi Pohjolainen <span dir="ltr"><<a href="mailto:topi.pohjolainen@intel.com" target="_blank">topi.pohjolainen@intel.com</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">diff --git a/src/mesa/drivers/dri/i965/brw_blorp_blit_eu.h b/src/mesa/drivers/dri/i965/brw_blorp_blit_eu.h<br>
index 1ecf076..3f2301c 100644<br>
--- a/src/mesa/drivers/dri/i965/brw_blorp_blit_eu.h<br>
+++ b/src/mesa/drivers/dri/i965/brw_blorp_blit_eu.h<br>
@@ -162,6 +162,24 @@ protected:<br>
       brw_RNDD(&func, dst, src);<br>
    }<br>
<br>
+   inline void emit_if(int op,<br>
+                       const struct brw_reg &x,<br>
+                       const struct brw_reg &y)<br>
+   {<br>
+      brw_CMP(&func, vec16(brw_null_reg()), op, x, y);<br>
+      brw_IF(&func, BRW_EXECUTE_16);<br>
+   }<br></blockquote><div><br></div><div>Can we maybe call tihs "emit_cmp_if()", so that it's clear when looking at the caller that it performs both a CMP and an IF?<br><br></div><div>With that change, this patch is:<br>
<br>Reviewed-by: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>><br></div></div></div></div>