[Mesa-dev] [PATCH 1/3] glsl: Disallow `pragma STDGL invariant(all)` in fragment shaders

[Chris Forbes]

Will do.

On Mon, Dec 1, 2014 at 11:38 AM, Jordan Justen
<jordan.l.justen at intel.com> wrote:
> Could you add spec references in comments or the commit messages?
>
> On 2014-11-29 12:54:58, Chris Forbes wrote:
>> Fixes the piglit test: spec/glsl-es-3.00/compiler/invariant_all.frag
>>
>> Signed-off-by: Chris Forbes <chrisf at ijw.co.nz>
>> ---
>>  src/glsl/glsl_parser.yy | 4 ++++
>>  1 file changed, 4 insertions(+)
>>
>> diff --git a/src/glsl/glsl_parser.yy b/src/glsl/glsl_parser.yy
>> index dbab815..6592a84 100644
>> --- a/src/glsl/glsl_parser.yy
>> +++ b/src/glsl/glsl_parser.yy
>> @@ -337,6 +337,10 @@ pragma_statement:
>>                              "pragma `invariant(all)' not supported in %s "
>>                              "(GLSL ES 1.00 or GLSL 1.20 required)",
>>                              state->get_version_string());
>> +      } else if (state->stage == MESA_SHADER_FRAGMENT) {
>> +         _mesa_glsl_error(& @1, state,
>> +                          "pragma `invariant(all)' may not be used in a "
>> +                          "fragment shader.");
>>        } else {
>>           state->all_invariant = true;
>>        }
>> --
>> 2.1.3
>>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/mesa-dev