The global invariant pragma must not leak across compilations.

BUG=chromium:625363 TEST=conformance/glsl/bugs/global-invariant-does-not-leak-across-shaders.html Change-Id: Ie8d38e44837fc87bb4c94e76e779ad41e8b79957 Reviewed-on: https://chromium-review.googlesource.com/358203Reviewed-by: Corentin Wallez <cwallez@chromium.org> Reviewed-by: Jamie Madill <jmadill@chromium.org> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Commit-Queue: Zhenyao Mo <zmo@chromium.org>

The global invariant pragma must not leak across compilations.
ae2e1918 · Kenneth Russell · Zhenyao Mo · 6e4cfceb · ae2e1918 · ae2e1918
Commit ae2e1918 authored Jul 01, 2016 by Kenneth Russell Committed by Zhenyao Mo Jul 05, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 6 deletions

Compiler.cpp src/compiler/translator/Compiler.cpp +1 -4

SymbolTable.h src/compiler/translator/SymbolTable.h +1 -2

No files found.
--- a/src/compiler/translator/Compiler.cpp
+++ b/src/compiler/translator/Compiler.cpp
@@ -246,10 +246,7 @@ TIntermNode *TCompiler::compileTreeImpl(const char *const shaderStrings[],
    if (success)
    {
        mPragma = parseContext.pragma();
-        if (mPragma.stdgl.invariantAll)
-        {
-            symbolTable.setGlobalInvariant();
-        }
+        symbolTable.setGlobalInvariant(mPragma.stdgl.invariantAll);

        root = parseContext.getTreeRoot();
        root = intermediate.postProcess(root);

--- a/src/compiler/translator/SymbolTable.h
+++ b/src/compiler/translator/SymbolTable.h
@@ -469,8 +469,7 @@ class TSymbolTable : angle::NonCopyable
              mInvariantVaryings.count(originalName) > 0);
    }

-    void setGlobalInvariant() { mGlobalInvariant = true; }
-    bool getGlobalInvariant() const { return mGlobalInvariant; }
+    void setGlobalInvariant(bool invariant) { mGlobalInvariant = invariant; }

    static int nextUniqueId()
    {