SimplifyLoopConditions: handle empty for expressions

BUG=661558 Change-Id: I7a4af71fafacc31aeae0c52d79641ea5e61d6b24 Reviewed-on: https://chromium-review.googlesource.com/406545 Commit-Queue: Corentin Wallez <cwallez@chromium.org> Reviewed-by: Olli Etuaho <oetuaho@nvidia.com> Reviewed-by: Geoff Lang <geofflang@chromium.org>

SimplifyLoopConditions: handle empty for expressions
71d4c026 · Corentin Wallez · Commit Bot · 32028eb9 · 71d4c026
Commit 71d4c026 authored Nov 02, 2016 by Corentin Wallez Committed by Commit Bot Nov 08, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 1 deletion

SimplifyLoopConditions.cpp src/compiler/translator/SimplifyLoopConditions.cpp +4 -1

No files found.
--- a/src/compiler/translator/SimplifyLoopConditions.cpp
+++ b/src/compiler/translator/SimplifyLoopConditions.cpp
@@ -216,7 +216,10 @@ void SimplifyLoopConditionsTraverser::traverseLoop(TIntermLoop *node)
                {
                    whileLoopBody->getSequence()->push_back(node->getBody());
                }
-                whileLoopBody->getSequence()->push_back(node->getExpression());
+                if (node->getExpression())
+                {
+                    whileLoopBody->getSequence()->push_back(node->getExpression());
+                }
                whileLoopBody->getSequence()->push_back(
                    createTempAssignment(node->getCondition()->deepCopy()));
                TIntermLoop *whileLoop = new TIntermLoop(