i965: When gl_PointSize is unwritten, default to 1.0 on Gen4-5.
authorKenneth Graunke <kenneth@whitecape.org>
Wed, 10 May 2017 09:07:46 +0000 (02:07 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Wed, 14 Jun 2017 22:56:21 +0000 (15:56 -0700)
commit6d4e031d9ad7e34d71c8d72afa50d1253515a670
tree896e2786061a548c5c6ff3c6ba9540ab39a712cd
parent3d34e275224689c4782f0aa3eada90146b3406a0
i965: When gl_PointSize is unwritten, default to 1.0 on Gen4-5.

Modern GL specifications say that the point size should be 1.0 when
gl_PointSize is unwritten and the last enabled stage is a geometry
or tessellation shader.  If it's a vertex shader, though, both the
GL specs and ES 3.0 spec say that it's undefined - so since Gen4-5
only support vertex shaders, there's no actual requirement to do this.

Since there is a cost associated (an extra dirty bit, which may cause
SF_STATE to be emitted more often), it may not be a good idea.

The real benefit is that it makes all generations behave identically.
And that seems somewhat nice...

Reviewed-by: Rafael Antognolli <rafael.antognolli@intel.com>
src/mesa/drivers/dri/i965/brw_sf_state.c