On 14 June 2018 at 10:12, Matt Turner <matts...@gmail.com> wrote: > On Wed, Jun 13, 2018 at 4:53 PM, Dave Airlie <airl...@gmail.com> wrote: >> From: Dave Airlie <airl...@redhat.com> >> >> GLSL 4.60 offically added this but games and older CTS suites actually >> had shaders that did this, we may as well enable it everywhere. >> --- >> src/compiler/glsl/glsl_parser.yy | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/src/compiler/glsl/glsl_parser.yy >> b/src/compiler/glsl/glsl_parser.yy >> index 91c10ce1a60..432fc874268 100644 >> --- a/src/compiler/glsl/glsl_parser.yy >> +++ b/src/compiler/glsl/glsl_parser.yy >> @@ -2706,6 +2706,7 @@ external_declaration: >> | declaration { $$ = $1; } >> | pragma_statement { $$ = NULL; } >> | layout_defaults { $$ = $1; } >> + | ';' { $$ = NULL; } > > Indentation. > > Also, piglit test?
There is already a piglit test, unfortunately it only runs under glsl 4.60, I suppose I can send a patch to enable it to run from GLSL1.30. > > Reviewed-by: Matt Turner <matts...@gmail.com> Thanks, Dave. _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev