Bug 32910 - Keywords 'in' and 'out' not handled properly for GLSL 1.20 shaders
Keywords 'in' and 'out' not handled properly for GLSL 1.20 shaders
Product: Mesa
Classification: Unclassified
Component: glsl-compiler
All All
: medium normal
Assigned To: Ian Romanick
Depends on:
  Show dependency treegraph
Reported: 2011-01-07 16:37 UTC by Ian Romanick
Modified: 2011-01-25 17:39 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Note You need to log in before you can comment on or make changes to this bug.
Description Ian Romanick 2011-01-07 16:37:54 UTC
In GLSL 1.20 mode, the keywords 'in' and 'out' are (incorrectly) accepted on global variables.  This was discovered by code inspection and verified with piglit tests spec/glsl-1.20/compiler/qualifiers/in-01.vert and spec/glsl-1.20/compiler/qualifiers/out-01.vert.
Comment 1 Ian Romanick 2011-01-10 17:50:14 UTC
Fixed in master by the following patch series.  I'm leaving the bug open, but I'm reassigning it to version 7.10.

commit 469ea695bbbc984ebec26b2413ab70d450a283ca
Author: Ian Romanick <ian.d.romanick@intel.com>
Date:   Fri Jan 7 16:05:59 2011 -0800

    glsl: Disallow 'in' and 'out' on globals in GLSL 1.20
    Fixes piglit tests glsl-1.20/compiler/qualifiers/in-01.vert and
    glsl-1.20/compiler/qualifiers/out-01.vert and bugzilla #32910.
    NOTE: This is a candidate for the 7.9 and 7.10 branches.  This patch
    also depends on the previous two commits.

commit a0c2ec8e2dcf90709dd2e147b06a0a173611b0a1
Author: Ian Romanick <ian.d.romanick@intel.com>
Date:   Fri Jan 7 16:04:30 2011 -0800

    glsl: Refresh autogenerated parser file.
    For the previous commit.

commit eebdfdfbcf2721349d2c2cafe2d81fd83cade9b9
Author: Ian Romanick <ian.d.romanick@intel.com>
Date:   Fri Jan 7 16:03:48 2011 -0800

    glsl: Add version_string containing properly formatted GLSL version
Comment 2 Ian Romanick 2011-01-25 17:39:04 UTC
In 7.9, fixed by:


In 7.10 fixed by: