<div dir="ltr"><div>That's a bit gross that we have to do that.... Oh, well.</div><div><br></div><div>Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Apr 29, 2019 at 6:01 PM Matt Turner <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">In the FS IR we pretend that the instruction is predicated with (+f0.1)<br>
just for flag dependency tracking purposes. Since the instruction<br>
doesn't support predication before Haswell, we unset the predicate so we<br>
should also unset the flag register so that we can round-trip the<br>
disassembly.<br>
---<br>
 src/intel/compiler/brw_fs_generator.cpp | 1 +<br>
 1 file changed, 1 insertion(+)<br>
<br>
diff --git a/src/intel/compiler/brw_fs_generator.cpp b/src/intel/compiler/brw_fs_generator.cpp<br>
index af8350aed6c..84909f83fec 100644<br>
--- a/src/intel/compiler/brw_fs_generator.cpp<br>
+++ b/src/intel/compiler/brw_fs_generator.cpp<br>
@@ -363,6 +363,7 @@ fs_generator::generate_fb_write(fs_inst *inst, struct brw_reg payload)<br>
 {<br>
    if (devinfo->gen < 8 && !devinfo->is_haswell) {<br>
       brw_set_default_predicate_control(p, BRW_PREDICATE_NONE);<br>
+      brw_set_default_flag_reg(p, 0, 0);<br>
    }<br>
<br>
    const struct brw_reg implied_header =<br>
-- <br>
2.21.0<br>
<br>
_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a></blockquote></div>