commit b60d30f71a672e10b38999ae26c6267ba6393739
author Olli Etuaho <oetuaho@nvidia.com> Tue Jan 16 12:31:06 2018 +0200
committer Commit Bot <commit-bot@chromium.org> Fri Jan 19 10:50:42 2018 +0000
tree 7f4d339b50f1f6118eb78c437d4ead309d2ab077
parent fa8869752a63e1818b0b8e53c2d18e7e0c6fe9e8

Make TVariable type immutable

This enables using constexpr types for built-in variables and some of
the variables created in AST transformations.

BUG=angleproject:2267
TEST=angle_unittests

Change-Id: Ie85b3c9872a071a7c023ced013b14ad91cff7cee
Reviewed-on: https://chromium-review.googlesource.com/868134
Commit-Queue: Olli Etuaho <oetuaho@nvidia.com>
Reviewed-by: Jamie Madill <jmadill@chromium.org>

src/compiler/translator/RemoveUnreferencedVariables.cpp

diff --git a/src/compiler/translator/RemoveUnreferencedVariables.cpp b/src/compiler/translator/RemoveUnreferencedVariables.cpp
index a1e3338..ad14648 100644
--- a/src/compiler/translator/RemoveUnreferencedVariables.cpp
+++ b/src/compiler/translator/RemoveUnreferencedVariables.cpp
@@ -203,8 +203,8 @@
                 // Already an empty declaration - nothing to do.
                 return;
             }
-            TVariable *emptyVariable =
-                new TVariable(mSymbolTable, nullptr, declarator->getType(), SymbolType::Empty);
+            TVariable *emptyVariable = new TVariable(
+                mSymbolTable, nullptr, new TType(declarator->getType()), SymbolType::Empty);
             queueReplacementWithParent(node, declarator, new TIntermSymbol(emptyVariable),
                                        OriginalNode::IS_DROPPED);
             return;