commit b990b55ef9c687a226dafd35fa581d4c6aaa308b
author Olli Etuaho <oetuaho@nvidia.com> Thu Oct 27 12:29:17 2016 +0100
committer Commit Bot <commit-bot@chromium.org> Wed Nov 02 11:45:13 2016 +0000
tree 3c1554610cff5babd345ed5ef905cfc1f6583e28
parent 0982a2bf1a1278ff8aaba63af935675237822f52

Clean up temporary variable usage in ScalarizeVecAndMatConstructorArgs

Use common helper functions instead of manually creating temporary
variable nodes. Also clean up the interface provided by the traverser.

BUG=angleproject:1597
Test=WebGL conformance tests

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

src/compiler/translator/Compiler.cpp

diff --git a/src/compiler/translator/Compiler.cpp b/src/compiler/translator/Compiler.cpp
index 6c8a9d6..c1be552 100644
--- a/src/compiler/translator/Compiler.cpp
+++ b/src/compiler/translator/Compiler.cpp
@@ -438,9 +438,8 @@
 
         if (success && (compileOptions & SH_SCALARIZE_VEC_AND_MAT_CONSTRUCTOR_ARGS))
         {
-            ScalarizeVecAndMatConstructorArgs scalarizer(
-                shaderType, fragmentPrecisionHigh);
-            root->traverse(&scalarizer);
+            ScalarizeVecAndMatConstructorArgs(root, shaderType, fragmentPrecisionHigh,
+                                              &mTemporaryIndex);
         }
 
         if (success && (compileOptions & SH_REGENERATE_STRUCT_NAMES))