commit 8b5e8fdb3c353d46c81243b31889856181ff66ae
author Olli Etuaho <oetuaho@nvidia.com> Fri Dec 15 14:59:15 2017 +0200
committer Commit Bot <commit-bot@chromium.org> Wed Jan 03 18:20:55 2018 +0000
tree 05ed91646e38bf161099dabee12a95e3dee1e65f
parent 93b059db4a5ab007e5239dff61eae3656e8da286

Replace remaining usage of TName with TSymbol

TName used to contain just a subset of the information stored in
TSymbol. It makes more sense to use TSymbol directly instead of
converting it to TName.

This also improves type safety a bit by making some functions only
take in TVariable or TFunction instead of the more generic TName.

BUG=angleproject:2267
TEST=angle_unittests

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

src/compiler/translator/IntermNode.cpp

diff --git a/src/compiler/translator/IntermNode.cpp b/src/compiler/translator/IntermNode.cpp
index 2d3c14e..1f5dfa8 100644
--- a/src/compiler/translator/IntermNode.cpp
+++ b/src/compiler/translator/IntermNode.cpp
@@ -140,16 +140,6 @@
 
 }  // namespace anonymous
 
-TName::TName(const TString &name) : mName(name), mIsInternal(false)
-{
-}
-
-TName::TName(const TSymbol *symbol)
-    : mName(symbol->symbolType() == SymbolType::Empty ? "" : symbol->name()),
-      mIsInternal(symbol->symbolType() == SymbolType::AngleInternal)
-{
-}
-
 ////////////////////////////////////////////////////////////////
 //
 // Member functions of the nodes used for building the tree.
@@ -278,7 +268,7 @@
 }
 
 TIntermSymbol::TIntermSymbol(const TVariable *variable)
-    : TIntermTyped(variable->getType()), mVariable(variable), mSymbol(variable)
+    : TIntermTyped(variable->getType()), mVariable(variable)
 {
 }
 
@@ -287,6 +277,11 @@
     return mVariable->uniqueId();
 }
 
+const TString &TIntermSymbol::getName() const
+{
+    return mVariable->name();
+}
+
 TIntermAggregate *TIntermAggregate::CreateFunctionCall(const TFunction &func,
                                                        TIntermSequence *arguments)
 {