commit bbd9d4c61e96d3736654ddd7624e68755a8ac218
author Olli Etuaho <oetuaho@nvidia.com> Thu Dec 21 12:02:00 2017 +0200
committer Commit Bot <commit-bot@chromium.org> Thu Jan 04 15:08:21 2018 +0000
tree 28db957ae7cb3131569b16a3dafcf973896b81ee
parent 47c8ea3fb265682b6dd040bf04488747f0292da1

Use TVariable instead of TIntermSymbol for variables

This removes unnecessary indirection. It's easier to just create
TVariables in createSamplerSymbols, and to track referenced variables
using TVariable pointers instead of TIntermSymbol pointers.

BUG=angleproject:2267
TEST=angle_unittests, angle_end2end_tests

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

src/compiler/translator/Symbol.cpp

diff --git a/src/compiler/translator/Symbol.cpp b/src/compiler/translator/Symbol.cpp
index c1009cb..4301a5b 100644
--- a/src/compiler/translator/Symbol.cpp
+++ b/src/compiler/translator/Symbol.cpp
@@ -76,8 +76,8 @@
 
 void TStructure::createSamplerSymbols(const TString &namePrefix,
                                       const TString &apiNamePrefix,
-                                      TVector<TIntermSymbol *> *outputSymbols,
-                                      TMap<TIntermSymbol *, TString> *outputSymbolsToAPINames,
+                                      TVector<const TVariable *> *outputSymbols,
+                                      TMap<const TVariable *, TString> *outputSymbolsToAPINames,
                                       TSymbolTable *symbolTable) const
 {
     ASSERT(containsSamplers());