i965: Fix up the handling of point sprite coordinate replacement.
authorEric Anholt <eric@anholt.net>
Mon, 8 Mar 2010 20:12:31 +0000 (12:12 -0800)
committerEric Anholt <eric@anholt.net>
Mon, 8 Mar 2010 22:23:18 +0000 (14:23 -0800)
commitbc632d04370566c1156cbd0345fe303834f0b910
treebd3d39bc412503773b627aae68c0e887371f81db
parent1d84808dc045d7fcf2fade8d1504bc25e7c5041a
i965: Fix up the handling of point sprite coordinate replacement.

The code was walking over the regs of pairs of attributes and checking
whether the attribute with a given reg index had point sprite enabled.
So the point sprite setup code was rarely even getting executed.
Instead, we need to determine which channels of a reg need point
sprite coordinate replacement.  In addition, it was multiplying the
attribute by 1/w, when it's supposed to cover (0, 1) in each direction
regardless of w, and it wasn't filling in the Z and W components of
the texcoord as specified.

Fixes piglit point-sprite and the spriteblast demo.  Bug #24431, #22245.
src/mesa/drivers/dri/i965/brw_sf.c
src/mesa/drivers/dri/i965/brw_sf.h
src/mesa/drivers/dri/i965/brw_sf_emit.c