i965: Ignore 'centroid' interpolation qualifier in case of persample shading
authorAnuj Phogat <anuj.phogat@gmail.com>
Wed, 15 Jan 2014 18:23:02 +0000 (10:23 -0800)
committerAnuj Phogat <anuj.phogat@gmail.com>
Tue, 21 Jan 2014 22:42:28 +0000 (14:42 -0800)
commitf5cfb4ae21df8eebfc6b86c0ce858b1c0a9160dd
tree8862c7bf3e9e77e31f467300f50588106ec3da20
parenta92e5f7cf63d496ad7830b5cea4bbab287c25b8e
i965: Ignore 'centroid' interpolation qualifier in case of persample shading

This patch handles the use of 'centroid' qualifier with 'in' variables
in a fragment shader when persample shading is enabled. Per sample
shading for the whole fragment shader can be enabled by:
glEnable(GL_SAMPLE_SHADING) or using {gl_SamplePosition, gl_SampleID}
builtin variables in fragment shader. Explaining it below in more
detail.

/* Enable sample shading using OpenGL API */
glEnable(GL_SAMPLE_SHADING);
glMinSampleShading(1.0);

Example fragment shader:
in vec4 a;
centroid in vec4 b;
main()
{
  ...
}

Variable 'a' will be interpolated at sample location. But, what
interpolation should we use for variable 'b' ?

ARB_sample_shading recommends interpolation at sample position for
all the variables. GLSL 400 (and earlier) spec says that:

"When an interpolation qualifier is used, it overrides settings
established through the OpenGL API."
But, this text got deleted in later versions of GLSL.

NVIDIA's and AMD's proprietary linux drivers (at OpenGL 4.3)
interpolates at sample position. This convinces me to use
the similar approach on intel hardware.

Cc: mesa-stable@lists.freedesktop.org
Signed-off-by: Anuj Phogat <anuj.phogat@gmail.com>
Reviewed-by: Chris Forbes <chrisf@ijw.co.nz>
src/mesa/drivers/dri/i965/brw_fs.cpp