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>
diff --git a/src/compiler/translator/RewriteDoWhile.cpp b/src/compiler/translator/RewriteDoWhile.cpp
index 7999cbf..e10ec74 100644
--- a/src/compiler/translator/RewriteDoWhile.cpp
+++ b/src/compiler/translator/RewriteDoWhile.cpp
@@ -138,7 +138,7 @@
node->replaceChildNodeWithMultiple(loop, replacement);
- nextTemporaryIndex();
+ nextTemporaryId();
}
return true;
}
@@ -146,12 +146,12 @@
} // anonymous namespace
-void RewriteDoWhile(TIntermNode *root, unsigned int *temporaryIndex)
+void RewriteDoWhile(TIntermNode *root, TSymbolUniqueId *temporaryId)
{
- ASSERT(temporaryIndex != 0);
+ ASSERT(temporaryId != 0);
DoWhileRewriter rewriter;
- rewriter.useTemporaryIndex(temporaryIndex);
+ rewriter.useTemporaryId(temporaryId);
root->traverse(&rewriter);
}