commit 409078f1e7c4ec79a1cbc58e3d0b3c399831ac5b
author Zhenyao Mo <zmo@chromium.org> Tue Oct 28 13:23:18 2014 -0700
committer Zhenyao Mo <zmo@chromium.org> Wed Oct 29 20:54:39 2014 +0000
tree 27797e626a5092059d0118be648abf3ce01a2ab5
parent 1a8a7e3b47ae18f5bdffaaf0f7c56a83e57f4d6d

Remove ShGetVariableInfo() and related code.

BUG=angle:775
TEST=chromium builds and runs fine

Change-Id: Ic62d3a3c9a8f034880b77a96f7f1c6be2691985e
Reviewed-on: https://chromium-review.googlesource.com/226004
Tested-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Shannon Woods <shannonwoods@chromium.org>

src/compiler/translator/Compiler.cpp

diff --git a/src/compiler/translator/Compiler.cpp b/src/compiler/translator/Compiler.cpp
index 2824064..a93d7e6 100644
--- a/src/compiler/translator/Compiler.cpp
+++ b/src/compiler/translator/Compiler.cpp
@@ -384,7 +384,6 @@
     uniforms.clear();
     expandedUniforms.clear();
     varyings.clear();
-    expandedVaryings.clear();
     interfaceBlocks.clear();
 
     builtInFunctionEmulator.Cleanup();
@@ -518,10 +517,8 @@
                                  symbolTable);
     root->traverse(&collect);
 
-    // For backwards compatiblity with ShGetVariableInfo, expand struct
-    // uniforms and varyings into separate variables for each field.
-    sh::ExpandVariables(uniforms, &expandedUniforms);
-    sh::ExpandVariables(varyings, &expandedVaryings);
+    // This is for enforcePackingRestriction().
+    sh::ExpandUniforms(uniforms, &expandedUniforms);
 }
 
 bool TCompiler::enforcePackingRestrictions()