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