Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 1 | // |
| 2 | // Copyright (c) 2017 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 | // RemoveUnreferencedVariables.cpp: |
| 7 | // Drop variables that are declared but never referenced in the AST. This avoids adding unnecessary |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 8 | // initialization code for them. Also removes unreferenced struct types. |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 9 | // |
| 10 | |
| 11 | #include "compiler/translator/RemoveUnreferencedVariables.h" |
| 12 | |
| 13 | #include "compiler/translator/IntermTraverse.h" |
| 14 | #include "compiler/translator/SymbolTable.h" |
| 15 | |
| 16 | namespace sh |
| 17 | { |
| 18 | |
| 19 | namespace |
| 20 | { |
| 21 | |
| 22 | class CollectVariableRefCountsTraverser : public TIntermTraverser |
| 23 | { |
| 24 | public: |
| 25 | CollectVariableRefCountsTraverser(); |
| 26 | |
| 27 | using RefCountMap = std::unordered_map<int, unsigned int>; |
| 28 | RefCountMap &getSymbolIdRefCounts() { return mSymbolIdRefCounts; } |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 29 | RefCountMap &getStructIdRefCounts() { return mStructIdRefCounts; } |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 30 | |
| 31 | void visitSymbol(TIntermSymbol *node) override; |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 32 | bool visitAggregate(Visit visit, TIntermAggregate *node) override; |
| 33 | bool visitFunctionPrototype(Visit visit, TIntermFunctionPrototype *node) override; |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 34 | |
| 35 | private: |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 36 | void incrementStructTypeRefCount(const TType &type); |
| 37 | |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 38 | RefCountMap mSymbolIdRefCounts; |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 39 | |
| 40 | // Structure reference counts are counted from symbols, constructors, function calls, function |
| 41 | // return values and from interface block and structure fields. We need to track both function |
| 42 | // calls and function return values since there's a compiler option not to prune unused |
| 43 | // functions. The type of a constant union may also be a struct, but statements that are just a |
| 44 | // constant union are always pruned, and if the constant union is used somehow it will get |
| 45 | // counted by something else. |
| 46 | RefCountMap mStructIdRefCounts; |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 47 | }; |
| 48 | |
| 49 | CollectVariableRefCountsTraverser::CollectVariableRefCountsTraverser() |
| 50 | : TIntermTraverser(true, false, false) |
| 51 | { |
| 52 | } |
| 53 | |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 54 | void CollectVariableRefCountsTraverser::incrementStructTypeRefCount(const TType &type) |
| 55 | { |
| 56 | if (type.isInterfaceBlock()) |
| 57 | { |
| 58 | const auto *block = type.getInterfaceBlock(); |
| 59 | ASSERT(block); |
| 60 | |
| 61 | // We can end up incrementing ref counts of struct types referenced from an interface block |
| 62 | // multiple times for the same block. This doesn't matter, because interface blocks can't be |
| 63 | // pruned so we'll never do the reverse operation. |
| 64 | for (const auto &field : block->fields()) |
| 65 | { |
| 66 | ASSERT(!field->type()->isInterfaceBlock()); |
| 67 | incrementStructTypeRefCount(*field->type()); |
| 68 | } |
| 69 | return; |
| 70 | } |
| 71 | |
| 72 | const auto *structure = type.getStruct(); |
| 73 | if (structure != nullptr) |
| 74 | { |
Olli Etuaho | 97fa855 | 2017-11-28 16:28:42 +0200 | [diff] [blame] | 75 | auto structIter = mStructIdRefCounts.find(structure->uniqueId().get()); |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 76 | if (structIter == mStructIdRefCounts.end()) |
| 77 | { |
Olli Etuaho | 97fa855 | 2017-11-28 16:28:42 +0200 | [diff] [blame] | 78 | mStructIdRefCounts[structure->uniqueId().get()] = 1u; |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 79 | |
| 80 | for (const auto &field : structure->fields()) |
| 81 | { |
| 82 | incrementStructTypeRefCount(*field->type()); |
| 83 | } |
| 84 | |
| 85 | return; |
| 86 | } |
| 87 | ++(structIter->second); |
| 88 | } |
| 89 | } |
| 90 | |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 91 | void CollectVariableRefCountsTraverser::visitSymbol(TIntermSymbol *node) |
| 92 | { |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 93 | incrementStructTypeRefCount(node->getType()); |
| 94 | |
Olli Etuaho | b6af22b | 2017-12-15 14:05:44 +0200 | [diff] [blame] | 95 | auto iter = mSymbolIdRefCounts.find(node->uniqueId().get()); |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 96 | if (iter == mSymbolIdRefCounts.end()) |
| 97 | { |
Olli Etuaho | b6af22b | 2017-12-15 14:05:44 +0200 | [diff] [blame] | 98 | mSymbolIdRefCounts[node->uniqueId().get()] = 1u; |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 99 | return; |
| 100 | } |
| 101 | ++(iter->second); |
| 102 | } |
| 103 | |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 104 | bool CollectVariableRefCountsTraverser::visitAggregate(Visit visit, TIntermAggregate *node) |
| 105 | { |
| 106 | // This tracks struct references in both function calls and constructors. |
| 107 | incrementStructTypeRefCount(node->getType()); |
| 108 | return true; |
| 109 | } |
| 110 | |
| 111 | bool CollectVariableRefCountsTraverser::visitFunctionPrototype(Visit visit, |
| 112 | TIntermFunctionPrototype *node) |
| 113 | { |
| 114 | incrementStructTypeRefCount(node->getType()); |
| 115 | return true; |
| 116 | } |
| 117 | |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 118 | // Traverser that removes all unreferenced variables on one traversal. |
| 119 | class RemoveUnreferencedVariablesTraverser : public TIntermTraverser |
| 120 | { |
| 121 | public: |
| 122 | RemoveUnreferencedVariablesTraverser( |
| 123 | CollectVariableRefCountsTraverser::RefCountMap *symbolIdRefCounts, |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 124 | CollectVariableRefCountsTraverser::RefCountMap *structIdRefCounts, |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 125 | TSymbolTable *symbolTable); |
| 126 | |
| 127 | bool visitDeclaration(Visit visit, TIntermDeclaration *node) override; |
| 128 | void visitSymbol(TIntermSymbol *node) override; |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 129 | bool visitAggregate(Visit visit, TIntermAggregate *node) override; |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 130 | |
| 131 | // Traverse loop and block nodes in reverse order. Note that this traverser does not track |
| 132 | // parent block positions, so insertStatementInParentBlock is unusable! |
| 133 | void traverseBlock(TIntermBlock *block) override; |
| 134 | void traverseLoop(TIntermLoop *loop) override; |
| 135 | |
| 136 | private: |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 137 | void removeVariableDeclaration(TIntermDeclaration *node, TIntermTyped *declarator); |
| 138 | void decrementStructTypeRefCount(const TType &type); |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 139 | |
| 140 | CollectVariableRefCountsTraverser::RefCountMap *mSymbolIdRefCounts; |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 141 | CollectVariableRefCountsTraverser::RefCountMap *mStructIdRefCounts; |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 142 | bool mRemoveReferences; |
| 143 | }; |
| 144 | |
| 145 | RemoveUnreferencedVariablesTraverser::RemoveUnreferencedVariablesTraverser( |
| 146 | CollectVariableRefCountsTraverser::RefCountMap *symbolIdRefCounts, |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 147 | CollectVariableRefCountsTraverser::RefCountMap *structIdRefCounts, |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 148 | TSymbolTable *symbolTable) |
| 149 | : TIntermTraverser(true, false, true, symbolTable), |
| 150 | mSymbolIdRefCounts(symbolIdRefCounts), |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 151 | mStructIdRefCounts(structIdRefCounts), |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 152 | mRemoveReferences(false) |
| 153 | { |
| 154 | } |
| 155 | |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 156 | void RemoveUnreferencedVariablesTraverser::decrementStructTypeRefCount(const TType &type) |
| 157 | { |
| 158 | auto *structure = type.getStruct(); |
| 159 | if (structure != nullptr) |
| 160 | { |
Olli Etuaho | 97fa855 | 2017-11-28 16:28:42 +0200 | [diff] [blame] | 161 | ASSERT(mStructIdRefCounts->find(structure->uniqueId().get()) != mStructIdRefCounts->end()); |
| 162 | unsigned int structRefCount = --(*mStructIdRefCounts)[structure->uniqueId().get()]; |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 163 | |
| 164 | if (structRefCount == 0) |
| 165 | { |
| 166 | for (const auto &field : structure->fields()) |
| 167 | { |
| 168 | decrementStructTypeRefCount(*field->type()); |
| 169 | } |
| 170 | } |
| 171 | } |
| 172 | } |
| 173 | |
| 174 | void RemoveUnreferencedVariablesTraverser::removeVariableDeclaration(TIntermDeclaration *node, |
| 175 | TIntermTyped *declarator) |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 176 | { |
| 177 | if (declarator->getType().isStructSpecifier() && !declarator->getType().isNamelessStruct()) |
| 178 | { |
Olli Etuaho | 97fa855 | 2017-11-28 16:28:42 +0200 | [diff] [blame] | 179 | unsigned int structId = declarator->getType().getStruct()->uniqueId().get(); |
Olli Etuaho | ea78d2b | 2018-01-09 12:55:27 +0200 | [diff] [blame^] | 180 | unsigned int structRefCountInThisDeclarator = 1u; |
| 181 | if (declarator->getAsBinaryNode() && |
| 182 | declarator->getAsBinaryNode()->getRight()->getAsAggregate()) |
| 183 | { |
| 184 | ASSERT(declarator->getAsBinaryNode()->getLeft()->getType().getStruct() == |
| 185 | declarator->getType().getStruct()); |
| 186 | ASSERT(declarator->getAsBinaryNode()->getRight()->getType().getStruct() == |
| 187 | declarator->getType().getStruct()); |
| 188 | structRefCountInThisDeclarator = 2u; |
| 189 | } |
| 190 | if ((*mStructIdRefCounts)[structId] > structRefCountInThisDeclarator) |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 191 | { |
| 192 | // If this declaration declares a named struct type that is used elsewhere, we need to |
| 193 | // keep it. We can still change the declarator though so that it doesn't declare an |
| 194 | // unreferenced variable. |
| 195 | |
| 196 | // Note that since we're not removing the entire declaration, the struct's reference |
| 197 | // count will end up being one less than the correct refcount. But since the struct |
| 198 | // declaration is kept, the incorrect refcount can't cause any other problems. |
| 199 | |
Olli Etuaho | 8b5e8fd | 2017-12-15 14:59:15 +0200 | [diff] [blame] | 200 | if (declarator->getAsSymbolNode() && |
| 201 | declarator->getAsSymbolNode()->variable().symbolType() == SymbolType::Empty) |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 202 | { |
| 203 | // Already an empty declaration - nothing to do. |
| 204 | return; |
| 205 | } |
Olli Etuaho | ae4dbf3 | 2017-12-08 20:49:00 +0100 | [diff] [blame] | 206 | TVariable *emptyVariable = |
| 207 | new TVariable(mSymbolTable, nullptr, declarator->getType(), SymbolType::Empty); |
Olli Etuaho | 195be94 | 2017-12-04 23:40:14 +0200 | [diff] [blame] | 208 | queueReplacementWithParent(node, declarator, new TIntermSymbol(emptyVariable), |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 209 | OriginalNode::IS_DROPPED); |
| 210 | return; |
| 211 | } |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 212 | } |
| 213 | |
| 214 | if (getParentNode()->getAsBlock()) |
| 215 | { |
| 216 | TIntermSequence emptyReplacement; |
| 217 | mMultiReplacements.push_back( |
| 218 | NodeReplaceWithMultipleEntry(getParentNode()->getAsBlock(), node, emptyReplacement)); |
| 219 | } |
| 220 | else |
| 221 | { |
| 222 | ASSERT(getParentNode()->getAsLoopNode()); |
| 223 | queueReplacement(nullptr, OriginalNode::IS_DROPPED); |
| 224 | } |
| 225 | } |
| 226 | |
| 227 | bool RemoveUnreferencedVariablesTraverser::visitDeclaration(Visit visit, TIntermDeclaration *node) |
| 228 | { |
| 229 | if (visit == PreVisit) |
| 230 | { |
| 231 | // SeparateDeclarations should have already been run. |
| 232 | ASSERT(node->getSequence()->size() == 1u); |
| 233 | |
| 234 | TIntermTyped *declarator = node->getSequence()->back()->getAsTyped(); |
| 235 | ASSERT(declarator); |
| 236 | |
| 237 | // We can only remove variables that are not a part of the shader interface. |
| 238 | TQualifier qualifier = declarator->getQualifier(); |
| 239 | if (qualifier != EvqTemporary && qualifier != EvqGlobal) |
| 240 | { |
| 241 | return true; |
| 242 | } |
| 243 | |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 244 | bool canRemoveVariable = false; |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 245 | TIntermSymbol *symbolNode = declarator->getAsSymbolNode(); |
| 246 | if (symbolNode != nullptr) |
| 247 | { |
Olli Etuaho | b6af22b | 2017-12-15 14:05:44 +0200 | [diff] [blame] | 248 | canRemoveVariable = (*mSymbolIdRefCounts)[symbolNode->uniqueId().get()] == 1u || |
Olli Etuaho | 8b5e8fd | 2017-12-15 14:59:15 +0200 | [diff] [blame] | 249 | symbolNode->variable().symbolType() == SymbolType::Empty; |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 250 | } |
| 251 | TIntermBinary *initNode = declarator->getAsBinaryNode(); |
| 252 | if (initNode != nullptr) |
| 253 | { |
| 254 | ASSERT(initNode->getLeft()->getAsSymbolNode()); |
Olli Etuaho | b6af22b | 2017-12-15 14:05:44 +0200 | [diff] [blame] | 255 | int symbolId = initNode->getLeft()->getAsSymbolNode()->uniqueId().get(); |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 256 | canRemoveVariable = |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 257 | (*mSymbolIdRefCounts)[symbolId] == 1u && !initNode->getRight()->hasSideEffects(); |
| 258 | } |
| 259 | |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 260 | if (canRemoveVariable) |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 261 | { |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 262 | removeVariableDeclaration(node, declarator); |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 263 | mRemoveReferences = true; |
| 264 | } |
| 265 | return true; |
| 266 | } |
| 267 | ASSERT(visit == PostVisit); |
| 268 | mRemoveReferences = false; |
| 269 | return true; |
| 270 | } |
| 271 | |
| 272 | void RemoveUnreferencedVariablesTraverser::visitSymbol(TIntermSymbol *node) |
| 273 | { |
| 274 | if (mRemoveReferences) |
| 275 | { |
Olli Etuaho | b6af22b | 2017-12-15 14:05:44 +0200 | [diff] [blame] | 276 | ASSERT(mSymbolIdRefCounts->find(node->uniqueId().get()) != mSymbolIdRefCounts->end()); |
| 277 | --(*mSymbolIdRefCounts)[node->uniqueId().get()]; |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 278 | |
| 279 | decrementStructTypeRefCount(node->getType()); |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 280 | } |
| 281 | } |
| 282 | |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 283 | bool RemoveUnreferencedVariablesTraverser::visitAggregate(Visit visit, TIntermAggregate *node) |
| 284 | { |
Olli Etuaho | ea78d2b | 2018-01-09 12:55:27 +0200 | [diff] [blame^] | 285 | if (visit == PreVisit && mRemoveReferences) |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 286 | { |
| 287 | decrementStructTypeRefCount(node->getType()); |
| 288 | } |
| 289 | return true; |
| 290 | } |
| 291 | |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 292 | void RemoveUnreferencedVariablesTraverser::traverseBlock(TIntermBlock *node) |
| 293 | { |
| 294 | // We traverse blocks in reverse order. This way reference counts can be decremented when |
| 295 | // removing initializers, and variables that become unused when initializers are removed can be |
| 296 | // removed on the same traversal. |
| 297 | |
| 298 | ScopedNodeInTraversalPath addToPath(this, node); |
| 299 | |
| 300 | bool visit = true; |
| 301 | |
| 302 | TIntermSequence *sequence = node->getSequence(); |
| 303 | |
| 304 | if (preVisit) |
| 305 | visit = visitBlock(PreVisit, node); |
| 306 | |
| 307 | if (visit) |
| 308 | { |
| 309 | for (auto iter = sequence->rbegin(); iter != sequence->rend(); ++iter) |
| 310 | { |
| 311 | (*iter)->traverse(this); |
| 312 | if (visit && inVisit) |
| 313 | { |
| 314 | if ((iter + 1) != sequence->rend()) |
| 315 | visit = visitBlock(InVisit, node); |
| 316 | } |
| 317 | } |
| 318 | } |
| 319 | |
| 320 | if (visit && postVisit) |
| 321 | visitBlock(PostVisit, node); |
| 322 | } |
| 323 | |
| 324 | void RemoveUnreferencedVariablesTraverser::traverseLoop(TIntermLoop *node) |
| 325 | { |
| 326 | // We traverse loops in reverse order as well. The loop body gets traversed before the init |
| 327 | // node. |
| 328 | |
| 329 | ScopedNodeInTraversalPath addToPath(this, node); |
| 330 | |
| 331 | bool visit = true; |
| 332 | |
| 333 | if (preVisit) |
| 334 | visit = visitLoop(PreVisit, node); |
| 335 | |
| 336 | if (visit) |
| 337 | { |
| 338 | // We don't need to traverse loop expressions or conditions since they can't be declarations |
| 339 | // in the AST (loops which have a declaration in their condition get transformed in the |
| 340 | // parsing stage). |
| 341 | ASSERT(node->getExpression() == nullptr || |
| 342 | node->getExpression()->getAsDeclarationNode() == nullptr); |
| 343 | ASSERT(node->getCondition() == nullptr || |
| 344 | node->getCondition()->getAsDeclarationNode() == nullptr); |
| 345 | |
| 346 | if (node->getBody()) |
| 347 | node->getBody()->traverse(this); |
| 348 | |
| 349 | if (node->getInit()) |
| 350 | node->getInit()->traverse(this); |
| 351 | } |
| 352 | |
| 353 | if (visit && postVisit) |
| 354 | visitLoop(PostVisit, node); |
| 355 | } |
| 356 | |
| 357 | } // namespace |
| 358 | |
| 359 | void RemoveUnreferencedVariables(TIntermBlock *root, TSymbolTable *symbolTable) |
| 360 | { |
| 361 | CollectVariableRefCountsTraverser collector; |
| 362 | root->traverse(&collector); |
Olli Etuaho | 39f74df | 2017-11-20 16:09:57 +0200 | [diff] [blame] | 363 | RemoveUnreferencedVariablesTraverser traverser(&collector.getSymbolIdRefCounts(), |
| 364 | &collector.getStructIdRefCounts(), symbolTable); |
Olli Etuaho | 3d70ca9 | 2017-11-10 16:53:26 +0200 | [diff] [blame] | 365 | root->traverse(&traverser); |
| 366 | traverser.updateTree(); |
| 367 | } |
| 368 | |
| 369 | } // namespace sh |