i965: Allow clip distances to be read back in fragment shaders.
authorPaul Berry <stereotype441@gmail.com>
Sun, 4 Sep 2011 19:18:06 +0000 (12:18 -0700)
committerPaul Berry <stereotype441@gmail.com>
Fri, 23 Sep 2011 22:32:42 +0000 (15:32 -0700)
commit2a6d43204024bc47df9bf5f7803d62d4d9580ab5
tree2c6ee2cb8b513497ae462d1122fac1700d64b377
parent62bad54727690bff5ed42a74272e7822fd36cdb6
i965: Allow clip distances to be read back in fragment shaders.

Previously, we conditionally set up the SF pipline stage with a
urb_entry_read_offset of 2 when clipping was in use, and 1 otherwise,
causing the clip distance VUE slots to be skipped if present.  This
was an extremely minor savings (it saved the SF unit from reading 2
vec4s out of the URB, but it didn't affect any computation, since we
only instruct the SF unit to perform interpolation on VUE slots that
are actually used by the fragment shader).

GLSL 1.30 requires an interpolated version of gl_ClipDistance to be
available for reading in the fragment shader, so we need the SF's
urb_entry_read_offset to be 1 when the fragment shader reads from
gl_ClipDistance.

This patch just unconditionally sets the urb_entry_read_offset to 1 in
all cases; this is sufficient to make gl_ClipDistance available to the
fragment shader when it is needed, and the performance loss should be
negligible when it isn't.

Reviewed-by: Eric Anholt <eric@anholt.net>
src/mesa/drivers/dri/i965/gen6_sf_state.c
src/mesa/drivers/dri/i965/gen7_sf_state.c