gallium: clarify SVIEWINFO opcode
authorRoland Scheidegger <sroland@vmware.com>
Wed, 7 Aug 2013 15:03:45 +0000 (17:03 +0200)
committerRoland Scheidegger <sroland@vmware.com>
Thu, 8 Aug 2013 16:55:57 +0000 (18:55 +0200)
This opcode is quite problematic in tgsi, while it tries to mirror
d3d10 resinfo it can't really do what's stated there due to missing
the crazy return type modifiers. Hence specify this is ignored along
with the swizzle.
(Other options would be to have multiple opcodes or specify the ret
type modifier maybe in dst_reg as there's padding bits left there but
it is the only instruction allowing this.)

Reviewed-by: Zack Rusin <zackr@vmware.com>
src/gallium/docs/source/tgsi.rst

index 8506b7e0309504d21188131af8ee87db803350f3..949ad89e929dd0a29fa7934120d9d7374f120765 100644 (file)
@@ -1868,7 +1868,15 @@ after lookup.
                the mipmap level selected by the src_mip_level and
                are in the number of texels.
                For 1d texture array width is in dst.x, array size
-               is in dst.y and dst.zw are always 0.
+               is in dst.y and dst.z is 0. The number of mipmaps
+               is still in dst.w.
+               In contrast to d3d10 resinfo, there's no way in the
+               tgsi instruction encoding to specify the return type
+               (float/rcpfloat/uint), hence always using uint. Also,
+               unlike the SAMPLE instructions, the swizzle on src1
+               resinfo allowing swizzling dst values is ignored (due
+               to the interaction with rcpfloat modifier which requires
+               some swizzle handling in the state tracker anyway).
 
 .. opcode:: SAMPLE_POS - query the position of a given sample.
                dst receives float4 (x, y, 0, 0) indicated where the