Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 1 | // |
| 2 | // Copyright (c) 2014 The ANGLE Project Authors. All rights reserved. |
| 3 | // Use of this source code is governed by a BSD-style license that can be |
| 4 | // found in the LICENSE file. |
| 5 | // |
| 6 | // RewriteElseBlocks.cpp: Implementation for tree transform to change |
| 7 | // all if-else blocks to if-if blocks. |
| 8 | // |
| 9 | |
| 10 | #include "compiler/translator/RewriteElseBlocks.h" |
| 11 | #include "compiler/translator/NodeSearch.h" |
| 12 | #include "compiler/translator/SymbolTable.h" |
| 13 | |
| 14 | namespace sh |
| 15 | { |
| 16 | |
Jamie Madill | 4836d22 | 2014-07-24 06:55:51 -0400 | [diff] [blame] | 17 | namespace |
| 18 | { |
| 19 | |
| 20 | class ElseBlockRewriter : public TIntermTraverser |
| 21 | { |
| 22 | public: |
| 23 | ElseBlockRewriter(); |
| 24 | |
| 25 | protected: |
Corentin Wallez | 2d58f7a | 2015-09-28 10:44:55 -0700 | [diff] [blame^] | 26 | bool visitAggregate(Visit visit, TIntermAggregate *aggregate) override; |
Jamie Madill | 4836d22 | 2014-07-24 06:55:51 -0400 | [diff] [blame] | 27 | |
| 28 | private: |
Jamie Madill | 4836d22 | 2014-07-24 06:55:51 -0400 | [diff] [blame] | 29 | const TType *mFunctionType; |
| 30 | |
| 31 | TIntermNode *rewriteSelection(TIntermSelection *selection); |
| 32 | }; |
| 33 | |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 34 | TIntermUnary *MakeNewUnary(TOperator op, TIntermTyped *operand) |
| 35 | { |
| 36 | TIntermUnary *unary = new TIntermUnary(op, operand->getType()); |
| 37 | unary->setOperand(operand); |
| 38 | return unary; |
| 39 | } |
| 40 | |
Jamie Madill | 787fc03 | 2014-07-07 12:49:45 -0400 | [diff] [blame] | 41 | ElseBlockRewriter::ElseBlockRewriter() |
Olli Etuaho | 64f0be9 | 2015-06-03 17:38:34 +0300 | [diff] [blame] | 42 | : TIntermTraverser(true, false, true), |
Jamie Madill | 787fc03 | 2014-07-07 12:49:45 -0400 | [diff] [blame] | 43 | mFunctionType(NULL) |
| 44 | {} |
| 45 | |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 46 | bool ElseBlockRewriter::visitAggregate(Visit visit, TIntermAggregate *node) |
| 47 | { |
| 48 | switch (node->getOp()) |
| 49 | { |
| 50 | case EOpSequence: |
Jamie Madill | 787fc03 | 2014-07-07 12:49:45 -0400 | [diff] [blame] | 51 | if (visit == PostVisit) |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 52 | { |
Zhenyao Mo | e40d1e9 | 2014-07-16 17:40:36 -0700 | [diff] [blame] | 53 | for (size_t statementIndex = 0; statementIndex != node->getSequence()->size(); statementIndex++) |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 54 | { |
Zhenyao Mo | e40d1e9 | 2014-07-16 17:40:36 -0700 | [diff] [blame] | 55 | TIntermNode *statement = (*node->getSequence())[statementIndex]; |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 56 | TIntermSelection *selection = statement->getAsSelectionNode(); |
Olli Etuaho | 4d59f3c | 2015-05-28 17:33:53 +0300 | [diff] [blame] | 57 | if (selection && selection->getFalseBlock() != nullptr) |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 58 | { |
Jamie Madill | ed3eef1 | 2014-02-26 09:47:11 -0500 | [diff] [blame] | 59 | // Check for if / else if |
| 60 | TIntermSelection *elseIfBranch = selection->getFalseBlock()->getAsSelectionNode(); |
| 61 | if (elseIfBranch) |
| 62 | { |
| 63 | selection->replaceChildNode(elseIfBranch, rewriteSelection(elseIfBranch)); |
| 64 | delete elseIfBranch; |
| 65 | } |
| 66 | |
Zhenyao Mo | e40d1e9 | 2014-07-16 17:40:36 -0700 | [diff] [blame] | 67 | (*node->getSequence())[statementIndex] = rewriteSelection(selection); |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 68 | delete selection; |
| 69 | } |
| 70 | } |
| 71 | } |
| 72 | break; |
| 73 | |
Jamie Madill | 787fc03 | 2014-07-07 12:49:45 -0400 | [diff] [blame] | 74 | case EOpFunction: |
| 75 | // Store the current function context (see comment below) |
| 76 | mFunctionType = ((visit == PreVisit) ? &node->getType() : NULL); |
| 77 | break; |
| 78 | |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 79 | default: break; |
| 80 | } |
| 81 | |
| 82 | return true; |
| 83 | } |
| 84 | |
| 85 | TIntermNode *ElseBlockRewriter::rewriteSelection(TIntermSelection *selection) |
| 86 | { |
Olli Etuaho | 4d59f3c | 2015-05-28 17:33:53 +0300 | [diff] [blame] | 87 | ASSERT(selection != nullptr); |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 88 | |
Olli Etuaho | 4d59f3c | 2015-05-28 17:33:53 +0300 | [diff] [blame] | 89 | nextTemporaryIndex(); |
| 90 | |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 91 | TIntermTyped *typedCondition = selection->getCondition()->getAsTyped(); |
Olli Etuaho | 4d59f3c | 2015-05-28 17:33:53 +0300 | [diff] [blame] | 92 | TIntermAggregate *storeCondition = createTempInitDeclaration(typedCondition); |
Jamie Madill | 787fc03 | 2014-07-07 12:49:45 -0400 | [diff] [blame] | 93 | |
Olli Etuaho | 4d59f3c | 2015-05-28 17:33:53 +0300 | [diff] [blame] | 94 | TIntermSelection *falseBlock = nullptr; |
| 95 | |
| 96 | TType boolType(EbtBool, EbpUndefined, EvqTemporary); |
Jamie Madill | 4836d22 | 2014-07-24 06:55:51 -0400 | [diff] [blame] | 97 | |
| 98 | if (selection->getFalseBlock()) |
Jamie Madill | 787fc03 | 2014-07-07 12:49:45 -0400 | [diff] [blame] | 99 | { |
Olli Etuaho | 4d59f3c | 2015-05-28 17:33:53 +0300 | [diff] [blame] | 100 | TIntermAggregate *negatedElse = nullptr; |
Jamie Madill | 4836d22 | 2014-07-24 06:55:51 -0400 | [diff] [blame] | 101 | // crbug.com/346463 |
| 102 | // D3D generates error messages claiming a function has no return value, when rewriting |
| 103 | // an if-else clause that returns something non-void in a function. By appending dummy |
| 104 | // returns (that are unreachable) we can silence this compile error. |
| 105 | if (mFunctionType && mFunctionType->getBasicType() != EbtVoid) |
| 106 | { |
| 107 | TString typeString = mFunctionType->getStruct() ? mFunctionType->getStruct()->name() : |
| 108 | mFunctionType->getBasicString(); |
| 109 | TString rawText = "return (" + typeString + ")0"; |
Olli Etuaho | 4d59f3c | 2015-05-28 17:33:53 +0300 | [diff] [blame] | 110 | TIntermRaw *returnNode = new TIntermRaw(*mFunctionType, rawText); |
| 111 | negatedElse = new TIntermAggregate(EOpSequence); |
| 112 | negatedElse->getSequence()->push_back(returnNode); |
Jamie Madill | 4836d22 | 2014-07-24 06:55:51 -0400 | [diff] [blame] | 113 | } |
| 114 | |
Olli Etuaho | 4d59f3c | 2015-05-28 17:33:53 +0300 | [diff] [blame] | 115 | TIntermSymbol *conditionSymbolElse = createTempSymbol(boolType); |
Jamie Madill | 4836d22 | 2014-07-24 06:55:51 -0400 | [diff] [blame] | 116 | TIntermUnary *negatedCondition = MakeNewUnary(EOpLogicalNot, conditionSymbolElse); |
| 117 | falseBlock = new TIntermSelection(negatedCondition, |
| 118 | selection->getFalseBlock(), negatedElse); |
Jamie Madill | 787fc03 | 2014-07-07 12:49:45 -0400 | [diff] [blame] | 119 | } |
| 120 | |
Olli Etuaho | 4d59f3c | 2015-05-28 17:33:53 +0300 | [diff] [blame] | 121 | TIntermSymbol *conditionSymbolSel = createTempSymbol(boolType); |
| 122 | TIntermSelection *newSelection = new TIntermSelection(conditionSymbolSel, selection->getTrueBlock(), falseBlock); |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 123 | |
| 124 | TIntermAggregate *block = new TIntermAggregate(EOpSequence); |
Olli Etuaho | 4d59f3c | 2015-05-28 17:33:53 +0300 | [diff] [blame] | 125 | block->getSequence()->push_back(storeCondition); |
Jamie Madill | 4836d22 | 2014-07-24 06:55:51 -0400 | [diff] [blame] | 126 | block->getSequence()->push_back(newSelection); |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 127 | |
| 128 | return block; |
| 129 | } |
| 130 | |
Jamie Madill | 4836d22 | 2014-07-24 06:55:51 -0400 | [diff] [blame] | 131 | } |
| 132 | |
Olli Etuaho | 4d59f3c | 2015-05-28 17:33:53 +0300 | [diff] [blame] | 133 | void RewriteElseBlocks(TIntermNode *node, unsigned int *temporaryIndex) |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 134 | { |
| 135 | ElseBlockRewriter rewriter; |
Olli Etuaho | 4d59f3c | 2015-05-28 17:33:53 +0300 | [diff] [blame] | 136 | rewriter.useTemporaryIndex(temporaryIndex); |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 137 | node->traverse(&rewriter); |
| 138 | } |
| 139 | |
| 140 | } |