Fix not rewriting else-if blocks.

We would miss expanding the else-if clauses in a chain of selection statements. BUG=346463 Change-Id: I8d16b9f25c97b8f6e7e73ad11bf7dae8e8c57ed7 Reviewed-on: https://chromium-review.googlesource.com/187844Reviewed-by: Nicolas Capens <nicolascapens@chromium.org> Tested-by: Jamie Madill <jmadill@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/189080Reviewed-by: Shannon Woods <shannonwoods@chromium.org>

Fix not rewriting else-if blocks.
117186e5 · Jamie Madill · 92db5082 · 117186e5
Commit 117186e5 authored Feb 26, 2014 by Jamie Madill
Hide whitespace changes
Inline Side-by-side

Showing with 8 additions and 0 deletions

RewriteElseBlocks.cpp src/compiler/translator/RewriteElseBlocks.cpp +8 -0

No files found.
--- a/src/compiler/translator/RewriteElseBlocks.cpp
+++ b/src/compiler/translator/RewriteElseBlocks.cpp
@@ -48,6 +48,14 @@ bool ElseBlockRewriter::visitAggregate(Visit visit, TIntermAggregate *node)
                TIntermSelection *selection = statement->getAsSelectionNode();
                if (selection && selection->getFalseBlock() != NULL)
                {
+                    // Check for if / else if
+                    TIntermSelection *elseIfBranch = selection->getFalseBlock()->getAsSelectionNode();
+                    if (elseIfBranch)
+                    {
+                        selection->replaceChildNode(elseIfBranch, rewriteSelection(elseIfBranch));
+                        delete elseIfBranch;
+                    }
+
                    node->getSequence()[statementIndex] = rewriteSelection(selection);
                    delete selection;
                }