commit 4dfe809444b8bec192c74b82cf8da615267b4974
author Olli Etuaho <oetuaho@nvidia.com> Fri Aug 21 17:44:35 2015 +0300
committer Olli Etuaho <oetuaho@nvidia.com> Tue Aug 25 06:01:24 2015 +0000
tree 039c1f086053bc6a1770bf317996daac5c519d25
parent 4001e1da927da8f621df6f9614d6d2994397b285

Always collect variables when generating HLSL

HLSL output needs uniform information generated by the collectVariables()
step to be able to write uniform registers.

Tested manually by compiling a shader with a uniform with
shader_translator.

BUG=angleproject:1132

Change-Id: I91d19b5fa789b7b33cf76a654ffbbd17d279db01
Reviewed-on: https://chromium-review.googlesource.com/294962
Tested-by: Olli Etuaho <oetuaho@nvidia.com>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Zhenyao Mo <zmo@chromium.org>

src/compiler/translator/Compiler.cpp

diff --git a/src/compiler/translator/Compiler.cpp b/src/compiler/translator/Compiler.cpp
index 555d794..b61fb6d 100644
--- a/src/compiler/translator/Compiler.cpp
+++ b/src/compiler/translator/Compiler.cpp
@@ -329,7 +329,7 @@
             RemovePow(root);
         }
 
-        if (success && (compileOptions & SH_VARIABLES))
+        if (success && shouldCollectVariables(compileOptions))
         {
             collectVariables(root);
             if (compileOptions & SH_ENFORCE_PACKING_RESTRICTIONS)