<p dir="ltr">R-B me</p>
<div class="gmail_quote">On Jul 24, 2015 6:52 AM, "Francisco Jerez" <<a href="mailto:currojerez@riseup.net">currojerez@riseup.net</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This hasn't been overallocating space for the header for a long time.<br>
It still leaves the header uninitialized though until the generator<br>
fixes it.<br>
---<br>
 src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 5 +++--<br>
 1 file changed, 3 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp b/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp<br>
index 09beb4d..5977f6d 100644<br>
--- a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp<br>
+++ b/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp<br>
@@ -473,8 +473,9 @@ fs_visitor::emit_texture_gen7(ir_texture_opcode op, fs_reg dst,<br>
    if (op == ir_tg4 || offset_value.file != BAD_FILE ||<br>
        is_high_sampler(devinfo, sampler)) {<br>
       /* For general texture offsets (no txf workaround), we need a header to<br>
-       * put them in.  Note that for SIMD16 we're making space for two actual<br>
-       * hardware registers here, so the emit will have to fix up for this.<br>
+       * put them in.  Note that we're only reserving space for it in the<br>
+       * message payload as it will be initialized implicitly by the<br>
+       * generator.<br>
        *<br>
        * * ir4_tg4 needs to place its channel select in the header,<br>
        * for interaction with ARB_texture_swizzle<br>
--<br>
2.4.3<br>
<br>
</blockquote></div>