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