[Mesa-dev] [PATCH v5 00/29] add fp64 support to mesa and glsl compiler
tapani.palli at intel.com
Mon Feb 16 00:30:25 PST 2015
On 02/11/2015 09:13 AM, Tapani Pälli wrote:
> On 02/10/2015 01:58 PM, Ilia Mirkin wrote:
>> Tapani's "glsl: validate output types for shader stages" patch is
>> necessary for the piglit that its description references to pass. If
>> you have a better suggestion of where to put that code, let me know.
> I've taken more look in to this. If the output does not have a
> interpolation qualifier then we incorrectly allow wrong types. My patch
> validates the output type independent of the interpolation qualifier so
> that's why it works. I can add more tests if wanted.
Sorry, this was not correct conclusion, the involved tests made me think
so. The actual reason why patch works is that we validate the output
variable types only if they are "valid varying variables" (see
is_varying_var function), and output from fragment shader is not "valid
varying". So we allow any type of output from fragment shader declared
with 'out', this patch fixes that. It has some extra checks as it
validates also outputs from other stages but to me it seems legit to do
in that place.
More information about the mesa-dev