<p dir="ltr"><br>
On Dec 2, 2014 9:19 PM, "Matt Turner" <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>> wrote:<br>
><br>
> Half gives you the second half of a SIMD16 register, but if the register<br>
> is a uniform it would incorrectly give you the next register.</p>
<p dir="ltr">I'm curious as to where this came up. To my knowledge, the only time that's used, we know it's not a uniform.  Am I missing something?</p>
<p dir="ltr">In any case, that seems like the right way to handle uniforms. We could also do that for immediates.</p>
<p dir="ltr">Assuming it has a point,<br>
Reviewed-by: Jason Ekstrand <<a href="mailto:jason.ekstrand@intel.com">jason.ekstrand@intel.com</a>></p>
<p dir="ltr">> ---<br>
>  src/mesa/drivers/dri/i965/brw_fs.h | 4 ++++<br>
>  1 file changed, 4 insertions(+)<br>
><br>
> diff --git a/src/mesa/drivers/dri/i965/brw_fs.h b/src/mesa/drivers/dri/i965/brw_fs.h<br>
> index 6a7fbe4..71b82ae 100644<br>
> --- a/src/mesa/drivers/dri/i965/brw_fs.h<br>
> +++ b/src/mesa/drivers/dri/i965/brw_fs.h<br>
> @@ -210,6 +210,10 @@ static inline fs_reg<br>
>  half(fs_reg reg, unsigned idx)<br>
>  {<br>
>     assert(idx < 2);<br>
> +<br>
> +   if (reg.file == UNIFORM)<br>
> +      return reg;<br>
> +<br>
>     assert(idx == 0 || (reg.file != HW_REG && reg.file != IMM));<br>
>     assert(reg.width == 16);<br>
>     reg.width = 8;<br>
> --<br>
> 2.0.4<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">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</p>