commit 4dd06d5d443668a569abd03bef65aa40408d8e70
author Olli Etuaho <oetuaho@nvidia.com> Wed Jul 05 12:41:06 2017 +0300
committer Commit Bot <commit-bot@chromium.org> Thu Jul 06 17:43:44 2017 +0000
tree e988cf9b3509108da0feb2f3d1d7de7b9c488cdf
parent 69df242c28e048067672b2d6c3d091ae039fba49

Set proper symbol ids on temporary symbol nodes

Temporary symbols used to all have symbol id 0. Now they get assigned
unique symbol ids. This makes it possible to keep track of them
according to the symbol id instead of their name, paving way to more
robust AST handling in the future.

BUG=angleproject:1490
TEST=angle_unittests

Change-Id: I292e2e483cc39173524fd30a30b48c4c808442e5
Reviewed-on: https://chromium-review.googlesource.com/559335
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Commit-Queue: Olli Etuaho <oetuaho@nvidia.com>

src/compiler/translator/IntermTraverse.cpp

diff --git a/src/compiler/translator/IntermTraverse.cpp b/src/compiler/translator/IntermTraverse.cpp
index 1819ab8..2b0e4b2 100644
--- a/src/compiler/translator/IntermTraverse.cpp
+++ b/src/compiler/translator/IntermTraverse.cpp
@@ -108,7 +108,7 @@
       mDepth(-1),
       mMaxDepth(0),
       mInGlobalScope(true),
-      mTemporaryIndex(nullptr)
+      mTemporaryId(nullptr)
 {
 }
 
@@ -167,11 +167,11 @@
     // Each traversal uses at most one temporary variable, so the index stays the same within a
     // single traversal.
     TInfoSinkBase symbolNameOut;
-    ASSERT(mTemporaryIndex != nullptr);
-    symbolNameOut << "s" << (*mTemporaryIndex);
+    ASSERT(mTemporaryId != nullptr);
+    symbolNameOut << "s" << (mTemporaryId->get());
     TString symbolName = symbolNameOut.c_str();
 
-    TIntermSymbol *node = new TIntermSymbol(0, symbolName, type);
+    TIntermSymbol *node = new TIntermSymbol(mTemporaryId->get(), symbolName, type);
     node->setInternal(true);
 
     ASSERT(qualifier == EvqTemporary || qualifier == EvqConst || qualifier == EvqGlobal);
@@ -217,15 +217,15 @@
     return assignment;
 }
 
-void TIntermTraverser::useTemporaryIndex(unsigned int *temporaryIndex)
+void TIntermTraverser::useTemporaryId(TSymbolUniqueId *temporaryId)
 {
-    mTemporaryIndex = temporaryIndex;
+    mTemporaryId = temporaryId;
 }
 
-void TIntermTraverser::nextTemporaryIndex()
+void TIntermTraverser::nextTemporaryId()
 {
-    ASSERT(mTemporaryIndex != nullptr);
-    ++(*mTemporaryIndex);
+    ASSERT(mTemporaryId != nullptr);
+    *mTemporaryId = TSymbolUniqueId();
 }
 
 void TLValueTrackingTraverser::addToFunctionMap(const TSymbolUniqueId &id,