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