svga: allow non-contiguous VS input declarations
authorBrian Paul <brianp@vmware.com>
Thu, 18 Feb 2016 15:41:16 +0000 (08:41 -0700)
committerBrian Paul <brianp@vmware.com>
Thu, 18 Feb 2016 22:46:17 +0000 (15:46 -0700)
commit2f3d06d9f95982b1dd9733260562e9b6484fc661
tree552ce4e2f55a644103e4a774fecaa65338a5e72f
parenta3e3c3e621a457866e141a18aba094e6e694bd45
svga: allow non-contiguous VS input declarations

This fixes a glDrawPixels regression since b63fe0552b5f.  The new
quad-drawing utility code uses 3 vertex attributes (xyz, rgba, st).
For glDrawPixels path we don't use the rgba attribute so there's a
gap in the TGSI VS input declarations (INPUT[0] = pos, INPUT[2] =
texcoord).  The TGSI->VGPU10 translations code did not handle this
correctly.  I missed this because my VM was configured for HWv11
while testing.

Another way to fix this would be to change the tgsi_scan.c code so
that the tgsi_shader_info::num_inputs (and num_outputs) included
the unused inputs/outputs.  These counts would then actually be
"max input register index + 1" rather than "number of used inputs".
But that change could impact all drivers so put it off for now.

No regressions found with piglit or typical GL apps.

v2: also update alloc_system_value_index() to use info.file_max[]

Reviewed-by: Charmaine Lee <charmainel@vmware.com>
src/gallium/drivers/svga/svga_tgsi_vgpu10.c