i965/blorp: retype destination register for texture SEND instruction to UW.
authorPaul Berry <stereotype441@gmail.com>
Tue, 24 Sep 2013 22:18:52 +0000 (15:18 -0700)
committerPaul Berry <stereotype441@gmail.com>
Mon, 30 Sep 2013 22:16:44 +0000 (15:16 -0700)
commit4c4934636cb286e7d7836afc26e9d392e2f0f155
treea569ec4b853a3ed3f6601f0773ae83c24b922ed1
parent1c7f75e45ed68e1d6bddcd01a61709f8f52d4d55
i965/blorp: retype destination register for texture SEND instruction to UW.

From the bspec documentation of the SEND instruction:

    "destination region cannot cross the 256-bit register boundary."

To avoid violating this restriction when executing SIMD16 texturing
operations (such as those used by blorp), we need to ensure that the
destination of the SEND instruction doesn't exceed 256 bits in size.
An easy way to do this is to set the type of the destination register
to UW (unsigned word), since 16 unsigned words can fit inside a
256-bit register.  Fortunately, this has no effect on the sampling
operation, since the sampler always infers the destination data type
from the sampler message rather than from the type of the instruction
operand.

Previously, we did this for texturing operations issued by the vec4
and fs back-ends, but not for blorp.  This patch makes blorp use the
same trick.

I haven't observed any behavioural difference on actual hardware due
to this patch, but it avoids a warning from the simulator so it seems
like the right thing to do.

Reviewed-by: Matt Turner <mattst88@gmail.com>
Reviewed-by: Eric Anholt <eric@anholt.net>
Acked-by: Chad Versace <chad.versace@linux.intel.com>
src/mesa/drivers/dri/i965/brw_blorp_blit.cpp