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" |
Olli Etuaho | 32db19b | 2016-10-04 14:43:16 +0100 | [diff] [blame] | 11 | |
| 12 | #include "compiler/translator/Intermediate.h" |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 13 | #include "compiler/translator/NodeSearch.h" |
| 14 | #include "compiler/translator/SymbolTable.h" |
| 15 | |
| 16 | namespace sh |
| 17 | { |
| 18 | |
Jamie Madill | 4836d22 | 2014-07-24 06:55:51 -0400 | [diff] [blame] | 19 | namespace |
| 20 | { |
| 21 | |
| 22 | class ElseBlockRewriter : public TIntermTraverser |
| 23 | { |
| 24 | public: |
| 25 | ElseBlockRewriter(); |
| 26 | |
| 27 | protected: |
Corentin Wallez | 2d58f7a | 2015-09-28 10:44:55 -0700 | [diff] [blame] | 28 | bool visitAggregate(Visit visit, TIntermAggregate *aggregate) override; |
Olli Etuaho | 6d40bbd | 2016-09-30 13:49:38 +0100 | [diff] [blame^] | 29 | bool visitBlock(Visit visit, TIntermBlock *block) override; |
Jamie Madill | 4836d22 | 2014-07-24 06:55:51 -0400 | [diff] [blame] | 30 | |
| 31 | private: |
Jamie Madill | 4836d22 | 2014-07-24 06:55:51 -0400 | [diff] [blame] | 32 | const TType *mFunctionType; |
| 33 | |
Olli Etuaho | 5796127 | 2016-09-14 13:57:46 +0300 | [diff] [blame] | 34 | TIntermNode *rewriteIfElse(TIntermIfElse *ifElse); |
Jamie Madill | 4836d22 | 2014-07-24 06:55:51 -0400 | [diff] [blame] | 35 | }; |
| 36 | |
Jamie Madill | 787fc03 | 2014-07-07 12:49:45 -0400 | [diff] [blame] | 37 | ElseBlockRewriter::ElseBlockRewriter() |
Olli Etuaho | 64f0be9 | 2015-06-03 17:38:34 +0300 | [diff] [blame] | 38 | : TIntermTraverser(true, false, true), |
Jamie Madill | 787fc03 | 2014-07-07 12:49:45 -0400 | [diff] [blame] | 39 | mFunctionType(NULL) |
| 40 | {} |
| 41 | |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 42 | bool ElseBlockRewriter::visitAggregate(Visit visit, TIntermAggregate *node) |
| 43 | { |
Olli Etuaho | 6d40bbd | 2016-09-30 13:49:38 +0100 | [diff] [blame^] | 44 | if (node->getOp() == EOpFunction) |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 45 | { |
Olli Etuaho | 6d40bbd | 2016-09-30 13:49:38 +0100 | [diff] [blame^] | 46 | // Store the current function context (see comment below) |
| 47 | mFunctionType = ((visit == PreVisit) ? &node->getType() : nullptr); |
| 48 | } |
| 49 | return true; |
| 50 | } |
| 51 | |
| 52 | bool ElseBlockRewriter::visitBlock(Visit visit, TIntermBlock *node) |
| 53 | { |
| 54 | if (visit == PostVisit) |
| 55 | { |
| 56 | for (size_t statementIndex = 0; statementIndex != node->getSequence()->size(); |
| 57 | statementIndex++) |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 58 | { |
Olli Etuaho | 6d40bbd | 2016-09-30 13:49:38 +0100 | [diff] [blame^] | 59 | TIntermNode *statement = (*node->getSequence())[statementIndex]; |
| 60 | TIntermIfElse *ifElse = statement->getAsIfElseNode(); |
| 61 | if (ifElse && ifElse->getFalseBlock() != nullptr) |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 62 | { |
Olli Etuaho | 6d40bbd | 2016-09-30 13:49:38 +0100 | [diff] [blame^] | 63 | (*node->getSequence())[statementIndex] = rewriteIfElse(ifElse); |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 64 | } |
| 65 | } |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 66 | } |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 67 | return true; |
| 68 | } |
| 69 | |
Olli Etuaho | 5796127 | 2016-09-14 13:57:46 +0300 | [diff] [blame] | 70 | TIntermNode *ElseBlockRewriter::rewriteIfElse(TIntermIfElse *ifElse) |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 71 | { |
Olli Etuaho | 5796127 | 2016-09-14 13:57:46 +0300 | [diff] [blame] | 72 | ASSERT(ifElse != nullptr); |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 73 | |
Olli Etuaho | 4d59f3c | 2015-05-28 17:33:53 +0300 | [diff] [blame] | 74 | nextTemporaryIndex(); |
| 75 | |
Olli Etuaho | 5796127 | 2016-09-14 13:57:46 +0300 | [diff] [blame] | 76 | TIntermTyped *typedCondition = ifElse->getCondition()->getAsTyped(); |
Olli Etuaho | 4d59f3c | 2015-05-28 17:33:53 +0300 | [diff] [blame] | 77 | TIntermAggregate *storeCondition = createTempInitDeclaration(typedCondition); |
Jamie Madill | 787fc03 | 2014-07-07 12:49:45 -0400 | [diff] [blame] | 78 | |
Olli Etuaho | 6d40bbd | 2016-09-30 13:49:38 +0100 | [diff] [blame^] | 79 | TIntermBlock *falseBlock = nullptr; |
Olli Etuaho | 4d59f3c | 2015-05-28 17:33:53 +0300 | [diff] [blame] | 80 | |
| 81 | TType boolType(EbtBool, EbpUndefined, EvqTemporary); |
Jamie Madill | 4836d22 | 2014-07-24 06:55:51 -0400 | [diff] [blame] | 82 | |
Olli Etuaho | 5796127 | 2016-09-14 13:57:46 +0300 | [diff] [blame] | 83 | if (ifElse->getFalseBlock()) |
Jamie Madill | 787fc03 | 2014-07-07 12:49:45 -0400 | [diff] [blame] | 84 | { |
Olli Etuaho | 6d40bbd | 2016-09-30 13:49:38 +0100 | [diff] [blame^] | 85 | TIntermBlock *negatedElse = nullptr; |
Jamie Madill | 4836d22 | 2014-07-24 06:55:51 -0400 | [diff] [blame] | 86 | // crbug.com/346463 |
| 87 | // D3D generates error messages claiming a function has no return value, when rewriting |
| 88 | // an if-else clause that returns something non-void in a function. By appending dummy |
| 89 | // returns (that are unreachable) we can silence this compile error. |
| 90 | if (mFunctionType && mFunctionType->getBasicType() != EbtVoid) |
| 91 | { |
| 92 | TString typeString = mFunctionType->getStruct() ? mFunctionType->getStruct()->name() : |
| 93 | mFunctionType->getBasicString(); |
| 94 | TString rawText = "return (" + typeString + ")0"; |
Olli Etuaho | 4d59f3c | 2015-05-28 17:33:53 +0300 | [diff] [blame] | 95 | TIntermRaw *returnNode = new TIntermRaw(*mFunctionType, rawText); |
Olli Etuaho | 6d40bbd | 2016-09-30 13:49:38 +0100 | [diff] [blame^] | 96 | negatedElse = new TIntermBlock(); |
Olli Etuaho | 4d59f3c | 2015-05-28 17:33:53 +0300 | [diff] [blame] | 97 | negatedElse->getSequence()->push_back(returnNode); |
Jamie Madill | 4836d22 | 2014-07-24 06:55:51 -0400 | [diff] [blame] | 98 | } |
| 99 | |
Olli Etuaho | 4d59f3c | 2015-05-28 17:33:53 +0300 | [diff] [blame] | 100 | TIntermSymbol *conditionSymbolElse = createTempSymbol(boolType); |
Olli Etuaho | a223430 | 2016-08-31 12:05:39 +0300 | [diff] [blame] | 101 | TIntermUnary *negatedCondition = new TIntermUnary(EOpLogicalNot, conditionSymbolElse); |
Olli Etuaho | 32db19b | 2016-10-04 14:43:16 +0100 | [diff] [blame] | 102 | TIntermIfElse *falseIfElse = |
| 103 | new TIntermIfElse(negatedCondition, ifElse->getFalseBlock(), negatedElse); |
Olli Etuaho | 6d40bbd | 2016-09-30 13:49:38 +0100 | [diff] [blame^] | 104 | falseBlock = TIntermediate::EnsureBlock(falseIfElse); |
Jamie Madill | 787fc03 | 2014-07-07 12:49:45 -0400 | [diff] [blame] | 105 | } |
| 106 | |
Olli Etuaho | 4d59f3c | 2015-05-28 17:33:53 +0300 | [diff] [blame] | 107 | TIntermSymbol *conditionSymbolSel = createTempSymbol(boolType); |
Olli Etuaho | 5796127 | 2016-09-14 13:57:46 +0300 | [diff] [blame] | 108 | TIntermIfElse *newIfElse = |
| 109 | new TIntermIfElse(conditionSymbolSel, ifElse->getTrueBlock(), falseBlock); |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 110 | |
Olli Etuaho | 6d40bbd | 2016-09-30 13:49:38 +0100 | [diff] [blame^] | 111 | TIntermBlock *block = new TIntermBlock(); |
Olli Etuaho | 4d59f3c | 2015-05-28 17:33:53 +0300 | [diff] [blame] | 112 | block->getSequence()->push_back(storeCondition); |
Olli Etuaho | 5796127 | 2016-09-14 13:57:46 +0300 | [diff] [blame] | 113 | block->getSequence()->push_back(newIfElse); |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 114 | |
| 115 | return block; |
| 116 | } |
| 117 | |
Jamie Madill | 4836d22 | 2014-07-24 06:55:51 -0400 | [diff] [blame] | 118 | } |
| 119 | |
Olli Etuaho | 4d59f3c | 2015-05-28 17:33:53 +0300 | [diff] [blame] | 120 | void RewriteElseBlocks(TIntermNode *node, unsigned int *temporaryIndex) |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 121 | { |
| 122 | ElseBlockRewriter rewriter; |
Olli Etuaho | 4d59f3c | 2015-05-28 17:33:53 +0300 | [diff] [blame] | 123 | rewriter.useTemporaryIndex(temporaryIndex); |
Jamie Madill | e53c98b | 2014-02-03 11:57:13 -0500 | [diff] [blame] | 124 | node->traverse(&rewriter); |
| 125 | } |
| 126 | |
| 127 | } |