[Mesa-dev] [Bug 30261] New: [GLSL 1.20] allowing inconsistent invariant declaration between two vertex shaders

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=30261

           Summary: [GLSL 1.20] allowing inconsistent invariant
                    declaration between two vertex shaders
           Product: Mesa
           Version: git
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: minor
          Priority: medium
         Component: Mesa core
        AssignedTo: mesa-dev at lists.freedesktop.org
        ReportedBy: gordon.jin at intel.com
                CC: idr at freedesktop.org


Created an attachment (id=38793)
 View: https://bugs.freedesktop.org/attachment.cgi?id=38793
 Review: https://bugs.freedesktop.org/review?bug=30261&attachment=38793

new piglit case

GLSL 1.20 section 4.3.6 says:
The type and presence of the invariant qualifiers of varying variables with the
same name declared in linked vertex and fragments shaders must match, otherwise
the link command will fail.

GLSL 1.50 section 4.6.1 says more clearly:
For variables leaving one shader and coming into another shader, the invariant
keyword has to be used in both shaders, or a link error will result.

The mesa glsl compiler fails link correctly for one vertex shader and one
fragment shader.

But it links (incorrectly) successfully for two vertex shaders.

This also exists in the old glsl compiler. So I'm not marking this as release
blocker.

Tested on Piketon (i965) with mesa master
ca92ae2699c4aad21c0811b9a5562b9223816caf.

New piglit case attached.

(This issues also applies for centroid. I'll add a new case for it after this
one gets handled.)

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.