commit 2a517272f07df229fb5f016ae81046f4fd9c9400
author Zhenyao Mo <zmo@chromium.org> Mon Oct 27 16:09:57 2014 -0700
committer Zhenyao Mo <zmo@chromium.org> Tue Oct 28 20:22:35 2014 +0000
tree fb23e955e91216515539a81221f11dec4e94e83c
parent fe14d455c240b756a66ea2cb20ad3e22f9a12cec

Fix an issue with separate invariant statement.

Basically we end up with an extra "invariant xxx ;;" on GL backend.

This is not handled correctly by Mac drivers and also out shader translator.

BUG=angle:776
TEST=webgl conformance on mac/windows

Change-Id: If32867efba64ab0d3e6e690155790b90d736439f
Reviewed-on: https://chromium-review.googlesource.com/225780
Tested-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>

src/compiler/translator/OutputGLSLBase.cpp

diff --git a/src/compiler/translator/OutputGLSLBase.cpp b/src/compiler/translator/OutputGLSLBase.cpp
index a295aee..772ed03 100644
--- a/src/compiler/translator/OutputGLSLBase.cpp
+++ b/src/compiler/translator/OutputGLSLBase.cpp
@@ -658,7 +658,7 @@
             ASSERT(sequence && sequence->size() == 1);
             const TIntermSymbol *symbol = sequence->front()->getAsSymbolNode();
             ASSERT(symbol);
-            out << "invariant " << hashVariableName(symbol->getSymbol()) << ";";
+            out << "invariant " << hashVariableName(symbol->getSymbol());
         }
         visitChildren = false;
         break;