i965/vec4: remove the generator hack for dual instanced GS
authorIago Toral Quiroga <itoral@igalia.com>
Fri, 26 Aug 2016 08:01:59 +0000 (10:01 +0200)
committerIago Toral Quiroga <itoral@igalia.com>
Mon, 29 Aug 2016 06:09:09 +0000 (08:09 +0200)
commit9c9f45b82410646d2f7a8576d03de9916118bf07
treeca401e11d8192e2500ed16cdd48e2abfdf5d0355
parent22cec6dc5e5a3060bc87f4a92871b4f6eef04632
i965/vec4: remove the generator hack for dual instanced GS

This hack was introduced in commit 03ac2c7223f7645e3:
i965/gs: Fix up gl_PointSize input swizzling for DUAL_INSTANCED gs

Specifically to fixup the code we emitted to deal with gl_PointSize inputs
in dual instance mode, where we were emitting a MOV to copy the point
size from .w (where the hardware delivers it) to .x (because code will
expect this to be a float). This meant that we were emitting a MOV
to an ATTR destination that could have a width of 4 (in dual instanced
mode) so it was necessary to fix the execution size and regioning of the
instruction.

Fortunately, Ken fixed this in 67c5d00273ca2:
i965/vec4/gs: Stop munging the ATTR containing gl_PointSize.

by using a WWWW swizzle instead of a MOV, and as the commit log in that
patch states, we no longer emit instructions with ATTR destinations, so
that makes the fixup code in the generator unnecessary.

Reviewed-by: Francisco Jerez <currojerez@riseup.net>
src/mesa/drivers/dri/i965/brw_vec4_generator.cpp