Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 1 | //=- WebAssemblyFixIrreducibleControlFlow.cpp - Fix irreducible control flow -// |
| 2 | // |
Chandler Carruth | 2946cd7 | 2019-01-19 08:50:56 +0000 | [diff] [blame] | 3 | // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. |
| 4 | // See https://llvm.org/LICENSE.txt for license information. |
| 5 | // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 6 | // |
| 7 | //===----------------------------------------------------------------------===// |
| 8 | /// |
| 9 | /// \file |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 10 | /// This file implements a pass that removes irreducible control flow. |
| 11 | /// Irreducible control flow means multiple-entry loops, which this pass |
| 12 | /// transforms to have a single entry. |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 13 | /// |
Dan Gohman | ddfa1a6 | 2016-03-09 04:17:36 +0000 | [diff] [blame] | 14 | /// Note that LLVM has a generic pass that lowers irreducible control flow, but |
| 15 | /// it linearizes control flow, turning diamonds into two triangles, which is |
| 16 | /// both unnecessary and undesirable for WebAssembly. |
| 17 | /// |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 18 | /// The big picture: We recursively process each "region", defined as a group |
| 19 | /// of blocks with a single entry and no branches back to that entry. A region |
| 20 | /// may be the entire function body, or the inner part of a loop, i.e., the |
| 21 | /// loop's body without branches back to the loop entry. In each region we fix |
| 22 | /// up multi-entry loops by adding a new block that can dispatch to each of the |
| 23 | /// loop entries, based on the value of a label "helper" variable, and we |
| 24 | /// replace direct branches to the entries with assignments to the label |
| 25 | /// variable and a branch to the dispatch block. Then the dispatch block is the |
| 26 | /// single entry in the loop containing the previous multiple entries. After |
| 27 | /// ensuring all the loops in a region are reducible, we recurse into them. The |
| 28 | /// total time complexity of this pass is: |
Alon Zakai | e9e01cc | 2019-03-27 20:12:42 +0000 | [diff] [blame] | 29 | /// |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 30 | /// O(NumBlocks * NumNestedLoops * NumIrreducibleLoops + |
| 31 | /// NumLoops * NumLoops) |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 32 | /// |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 33 | /// This pass is similar to what the Relooper [1] does. Both identify looping |
| 34 | /// code that requires multiple entries, and resolve it in a similar way (in |
| 35 | /// Relooper terminology, we implement a Multiple shape in a Loop shape). Note |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 36 | /// also that like the Relooper, we implement a "minimal" intervention: we only |
| 37 | /// use the "label" helper for the blocks we absolutely must and no others. We |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 38 | /// also prioritize code size and do not duplicate code in order to resolve |
| 39 | /// irreducibility. The graph algorithms for finding loops and entries and so |
| 40 | /// forth are also similar to the Relooper. The main differences between this |
| 41 | /// pass and the Relooper are: |
Alon Zakai | e9e01cc | 2019-03-27 20:12:42 +0000 | [diff] [blame] | 42 | /// |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 43 | /// * We just care about irreducibility, so we just look at loops. |
| 44 | /// * The Relooper emits structured control flow (with ifs etc.), while we |
| 45 | /// emit a CFG. |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 46 | /// |
| 47 | /// [1] Alon Zakai. 2011. Emscripten: an LLVM-to-JavaScript compiler. In |
| 48 | /// Proceedings of the ACM international conference companion on Object oriented |
| 49 | /// programming systems languages and applications companion (SPLASH '11). ACM, |
| 50 | /// New York, NY, USA, 301-312. DOI=10.1145/2048147.2048224 |
| 51 | /// http://doi.acm.org/10.1145/2048147.2048224 |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 52 | /// |
| 53 | //===----------------------------------------------------------------------===// |
| 54 | |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 55 | #include "MCTargetDesc/WebAssemblyMCTargetDesc.h" |
Chandler Carruth | 6bda14b | 2017-06-06 11:49:48 +0000 | [diff] [blame] | 56 | #include "WebAssembly.h" |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 57 | #include "WebAssemblySubtarget.h" |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 58 | #include "llvm/CodeGen/MachineInstrBuilder.h" |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 59 | using namespace llvm; |
| 60 | |
| 61 | #define DEBUG_TYPE "wasm-fix-irreducible-control-flow" |
| 62 | |
| 63 | namespace { |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 64 | |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 65 | using BlockVector = SmallVector<MachineBasicBlock *, 4>; |
| 66 | using BlockSet = SmallPtrSet<MachineBasicBlock *, 4>; |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 67 | |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 68 | // Calculates reachability in a region. Ignores branches to blocks outside of |
| 69 | // the region, and ignores branches to the region entry (for the case where |
| 70 | // the region is the inner part of a loop). |
| 71 | class ReachabilityGraph { |
| 72 | public: |
| 73 | ReachabilityGraph(MachineBasicBlock *Entry, const BlockSet &Blocks) |
| 74 | : Entry(Entry), Blocks(Blocks) { |
| 75 | #ifndef NDEBUG |
| 76 | // The region must have a single entry. |
| 77 | for (auto *MBB : Blocks) { |
| 78 | if (MBB != Entry) { |
| 79 | for (auto *Pred : MBB->predecessors()) { |
| 80 | assert(inRegion(Pred)); |
| 81 | } |
| 82 | } |
| 83 | } |
| 84 | #endif |
| 85 | calculate(); |
| 86 | } |
| 87 | |
Heejin Ahn | c60bc94 | 2019-03-19 05:26:33 +0000 | [diff] [blame] | 88 | bool canReach(MachineBasicBlock *From, MachineBasicBlock *To) const { |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 89 | assert(inRegion(From) && inRegion(To)); |
Heejin Ahn | c60bc94 | 2019-03-19 05:26:33 +0000 | [diff] [blame] | 90 | auto I = Reachable.find(From); |
| 91 | if (I == Reachable.end()) |
| 92 | return false; |
| 93 | return I->second.count(To); |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 94 | } |
| 95 | |
| 96 | // "Loopers" are blocks that are in a loop. We detect these by finding blocks |
| 97 | // that can reach themselves. |
Heejin Ahn | c60bc94 | 2019-03-19 05:26:33 +0000 | [diff] [blame] | 98 | const BlockSet &getLoopers() const { return Loopers; } |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 99 | |
| 100 | // Get all blocks that are loop entries. |
Heejin Ahn | c60bc94 | 2019-03-19 05:26:33 +0000 | [diff] [blame] | 101 | const BlockSet &getLoopEntries() const { return LoopEntries; } |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 102 | |
| 103 | // Get all blocks that enter a particular loop from outside. |
Heejin Ahn | c60bc94 | 2019-03-19 05:26:33 +0000 | [diff] [blame] | 104 | const BlockSet &getLoopEnterers(MachineBasicBlock *LoopEntry) const { |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 105 | assert(inRegion(LoopEntry)); |
Heejin Ahn | c60bc94 | 2019-03-19 05:26:33 +0000 | [diff] [blame] | 106 | auto I = LoopEnterers.find(LoopEntry); |
| 107 | assert(I != LoopEnterers.end()); |
| 108 | return I->second; |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 109 | } |
Jacob Gravelle | 4092645 | 2018-03-30 20:36:58 +0000 | [diff] [blame] | 110 | |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 111 | private: |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 112 | MachineBasicBlock *Entry; |
| 113 | const BlockSet &Blocks; |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 114 | |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 115 | BlockSet Loopers, LoopEntries; |
| 116 | DenseMap<MachineBasicBlock *, BlockSet> LoopEnterers; |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 117 | |
Heejin Ahn | c60bc94 | 2019-03-19 05:26:33 +0000 | [diff] [blame] | 118 | bool inRegion(MachineBasicBlock *MBB) const { return Blocks.count(MBB); } |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 119 | |
| 120 | // Maps a block to all the other blocks it can reach. |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 121 | DenseMap<MachineBasicBlock *, BlockSet> Reachable; |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 122 | |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 123 | void calculate() { |
| 124 | // Reachability computation work list. Contains pairs of recent additions |
| 125 | // (A, B) where we just added a link A => B. |
| 126 | using BlockPair = std::pair<MachineBasicBlock *, MachineBasicBlock *>; |
| 127 | SmallVector<BlockPair, 4> WorkList; |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 128 | |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 129 | // Add all relevant direct branches. |
| 130 | for (auto *MBB : Blocks) { |
| 131 | for (auto *Succ : MBB->successors()) { |
| 132 | if (Succ != Entry && inRegion(Succ)) { |
| 133 | Reachable[MBB].insert(Succ); |
| 134 | WorkList.emplace_back(MBB, Succ); |
| 135 | } |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 136 | } |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 137 | } |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 138 | |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 139 | while (!WorkList.empty()) { |
| 140 | MachineBasicBlock *MBB, *Succ; |
| 141 | std::tie(MBB, Succ) = WorkList.pop_back_val(); |
| 142 | assert(inRegion(MBB) && Succ != Entry && inRegion(Succ)); |
| 143 | if (MBB != Entry) { |
| 144 | // We recently added MBB => Succ, and that means we may have enabled |
| 145 | // Pred => MBB => Succ. |
| 146 | for (auto *Pred : MBB->predecessors()) { |
| 147 | if (Reachable[Pred].insert(Succ).second) { |
| 148 | WorkList.emplace_back(Pred, Succ); |
| 149 | } |
| 150 | } |
| 151 | } |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 152 | } |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 153 | |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 154 | // Blocks that can return to themselves are in a loop. |
| 155 | for (auto *MBB : Blocks) { |
| 156 | if (canReach(MBB, MBB)) { |
| 157 | Loopers.insert(MBB); |
| 158 | } |
| 159 | } |
| 160 | assert(!Loopers.count(Entry)); |
| 161 | |
| 162 | // Find the loop entries - loopers reachable from blocks not in that loop - |
| 163 | // and those outside blocks that reach them, the "loop enterers". |
| 164 | for (auto *Looper : Loopers) { |
| 165 | for (auto *Pred : Looper->predecessors()) { |
| 166 | // Pred can reach Looper. If Looper can reach Pred, it is in the loop; |
| 167 | // otherwise, it is a block that enters into the loop. |
| 168 | if (!canReach(Looper, Pred)) { |
| 169 | LoopEntries.insert(Looper); |
| 170 | LoopEnterers[Looper].insert(Pred); |
| 171 | } |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 172 | } |
| 173 | } |
| 174 | } |
| 175 | }; |
| 176 | |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 177 | // Finds the blocks in a single-entry loop, given the loop entry and the |
| 178 | // list of blocks that enter the loop. |
| 179 | class LoopBlocks { |
| 180 | public: |
| 181 | LoopBlocks(MachineBasicBlock *Entry, const BlockSet &Enterers) |
| 182 | : Entry(Entry), Enterers(Enterers) { |
| 183 | calculate(); |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 184 | } |
| 185 | |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 186 | BlockSet &getBlocks() { return Blocks; } |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 187 | |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 188 | private: |
| 189 | MachineBasicBlock *Entry; |
| 190 | const BlockSet &Enterers; |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 191 | |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 192 | BlockSet Blocks; |
| 193 | |
| 194 | void calculate() { |
| 195 | // Going backwards from the loop entry, if we ignore the blocks entering |
| 196 | // from outside, we will traverse all the blocks in the loop. |
| 197 | BlockVector WorkList; |
| 198 | BlockSet AddedToWorkList; |
| 199 | Blocks.insert(Entry); |
| 200 | for (auto *Pred : Entry->predecessors()) { |
| 201 | if (!Enterers.count(Pred)) { |
| 202 | WorkList.push_back(Pred); |
| 203 | AddedToWorkList.insert(Pred); |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 204 | } |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 205 | } |
| 206 | |
| 207 | while (!WorkList.empty()) { |
| 208 | auto *MBB = WorkList.pop_back_val(); |
| 209 | assert(!Enterers.count(MBB)); |
| 210 | if (Blocks.insert(MBB).second) { |
| 211 | for (auto *Pred : MBB->predecessors()) { |
| 212 | if (!AddedToWorkList.count(Pred)) { |
| 213 | WorkList.push_back(Pred); |
| 214 | AddedToWorkList.insert(Pred); |
| 215 | } |
| 216 | } |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 217 | } |
| 218 | } |
| 219 | } |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 220 | }; |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 221 | |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 222 | class WebAssemblyFixIrreducibleControlFlow final : public MachineFunctionPass { |
| 223 | StringRef getPassName() const override { |
| 224 | return "WebAssembly Fix Irreducible Control Flow"; |
| 225 | } |
| 226 | |
| 227 | bool runOnMachineFunction(MachineFunction &MF) override; |
| 228 | |
| 229 | bool processRegion(MachineBasicBlock *Entry, BlockSet &Blocks, |
| 230 | MachineFunction &MF); |
| 231 | |
| 232 | void makeSingleEntryLoop(BlockSet &Entries, BlockSet &Blocks, |
| 233 | MachineFunction &MF); |
| 234 | |
| 235 | public: |
| 236 | static char ID; // Pass identification, replacement for typeid |
| 237 | WebAssemblyFixIrreducibleControlFlow() : MachineFunctionPass(ID) {} |
| 238 | }; |
| 239 | |
| 240 | bool WebAssemblyFixIrreducibleControlFlow::processRegion( |
| 241 | MachineBasicBlock *Entry, BlockSet &Blocks, MachineFunction &MF) { |
| 242 | bool Changed = false; |
| 243 | |
| 244 | // Remove irreducibility before processing child loops, which may take |
| 245 | // multiple iterations. |
| 246 | while (true) { |
| 247 | ReachabilityGraph Graph(Entry, Blocks); |
| 248 | |
| 249 | bool FoundIrreducibility = false; |
| 250 | |
| 251 | for (auto *LoopEntry : Graph.getLoopEntries()) { |
| 252 | // Find mutual entries - all entries which can reach this one, and |
| 253 | // are reached by it (that always includes LoopEntry itself). All mutual |
| 254 | // entries must be in the same loop, so if we have more than one, then we |
| 255 | // have irreducible control flow. |
| 256 | // |
| 257 | // Note that irreducibility may involve inner loops, e.g. imagine A |
| 258 | // starts one loop, and it has B inside it which starts an inner loop. |
| 259 | // If we add a branch from all the way on the outside to B, then in a |
| 260 | // sense B is no longer an "inner" loop, semantically speaking. We will |
| 261 | // fix that irreducibility by adding a block that dispatches to either |
| 262 | // either A or B, so B will no longer be an inner loop in our output. |
| 263 | // (A fancier approach might try to keep it as such.) |
| 264 | // |
| 265 | // Note that we still need to recurse into inner loops later, to handle |
| 266 | // the case where the irreducibility is entirely nested - we would not |
| 267 | // be able to identify that at this point, since the enclosing loop is |
| 268 | // a group of blocks all of whom can reach each other. (We'll see the |
| 269 | // irreducibility after removing branches to the top of that enclosing |
| 270 | // loop.) |
| 271 | BlockSet MutualLoopEntries; |
| 272 | MutualLoopEntries.insert(LoopEntry); |
| 273 | for (auto *OtherLoopEntry : Graph.getLoopEntries()) { |
| 274 | if (OtherLoopEntry != LoopEntry && |
| 275 | Graph.canReach(LoopEntry, OtherLoopEntry) && |
| 276 | Graph.canReach(OtherLoopEntry, LoopEntry)) { |
| 277 | MutualLoopEntries.insert(OtherLoopEntry); |
| 278 | } |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 279 | } |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 280 | |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 281 | if (MutualLoopEntries.size() > 1) { |
| 282 | makeSingleEntryLoop(MutualLoopEntries, Blocks, MF); |
| 283 | FoundIrreducibility = true; |
| 284 | Changed = true; |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 285 | break; |
| 286 | } |
| 287 | } |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 288 | // Only go on to actually process the inner loops when we are done |
| 289 | // removing irreducible control flow and changing the graph. Modifying |
| 290 | // the graph as we go is possible, and that might let us avoid looking at |
| 291 | // the already-fixed loops again if we are careful, but all that is |
| 292 | // complex and bug-prone. Since irreducible loops are rare, just starting |
| 293 | // another iteration is best. |
| 294 | if (FoundIrreducibility) { |
| 295 | continue; |
| 296 | } |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 297 | |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 298 | for (auto *LoopEntry : Graph.getLoopEntries()) { |
| 299 | LoopBlocks InnerBlocks(LoopEntry, Graph.getLoopEnterers(LoopEntry)); |
| 300 | // Each of these calls to processRegion may change the graph, but are |
| 301 | // guaranteed not to interfere with each other. The only changes we make |
| 302 | // to the graph are to add blocks on the way to a loop entry. As the |
| 303 | // loops are disjoint, that means we may only alter branches that exit |
| 304 | // another loop, which are ignored when recursing into that other loop |
| 305 | // anyhow. |
| 306 | if (processRegion(LoopEntry, InnerBlocks.getBlocks(), MF)) { |
| 307 | Changed = true; |
| 308 | } |
| 309 | } |
| 310 | |
| 311 | return Changed; |
| 312 | } |
| 313 | } |
| 314 | |
| 315 | // Given a set of entries to a single loop, create a single entry for that |
| 316 | // loop by creating a dispatch block for them, routing control flow using |
| 317 | // a helper variable. Also updates Blocks with any new blocks created, so |
| 318 | // that we properly track all the blocks in the region. |
| 319 | void WebAssemblyFixIrreducibleControlFlow::makeSingleEntryLoop( |
| 320 | BlockSet &Entries, BlockSet &Blocks, MachineFunction &MF) { |
| 321 | assert(Entries.size() >= 2); |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 322 | |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 323 | // Sort the entries to ensure a deterministic build. |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 324 | BlockVector SortedEntries(Entries.begin(), Entries.end()); |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 325 | llvm::sort(SortedEntries, |
| 326 | [&](const MachineBasicBlock *A, const MachineBasicBlock *B) { |
| 327 | auto ANum = A->getNumber(); |
| 328 | auto BNum = B->getNumber(); |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 329 | return ANum < BNum; |
| 330 | }); |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 331 | |
Richard Trieu | e1fef94 | 2019-01-04 06:49:24 +0000 | [diff] [blame] | 332 | #ifndef NDEBUG |
| 333 | for (auto Block : SortedEntries) |
| 334 | assert(Block->getNumber() != -1); |
| 335 | if (SortedEntries.size() > 1) { |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 336 | for (auto I = SortedEntries.begin(), E = SortedEntries.end() - 1; I != E; |
| 337 | ++I) { |
Richard Trieu | e1fef94 | 2019-01-04 06:49:24 +0000 | [diff] [blame] | 338 | auto ANum = (*I)->getNumber(); |
| 339 | auto BNum = (*(std::next(I)))->getNumber(); |
| 340 | assert(ANum != BNum); |
| 341 | } |
| 342 | } |
| 343 | #endif |
| 344 | |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 345 | // Create a dispatch block which will contain a jump table to the entries. |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 346 | MachineBasicBlock *Dispatch = MF.CreateMachineBasicBlock(); |
| 347 | MF.insert(MF.end(), Dispatch); |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 348 | Blocks.insert(Dispatch); |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 349 | |
| 350 | // Add the jump table. |
| 351 | const auto &TII = *MF.getSubtarget<WebAssemblySubtarget>().getInstrInfo(); |
Heejin Ahn | c60bc94 | 2019-03-19 05:26:33 +0000 | [diff] [blame] | 352 | MachineInstrBuilder MIB = |
| 353 | BuildMI(Dispatch, DebugLoc(), TII.get(WebAssembly::BR_TABLE_I32)); |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 354 | |
| 355 | // Add the register which will be used to tell the jump table which block to |
| 356 | // jump to. |
| 357 | MachineRegisterInfo &MRI = MF.getRegInfo(); |
| 358 | unsigned Reg = MRI.createVirtualRegister(&WebAssembly::I32RegClass); |
| 359 | MIB.addReg(Reg); |
| 360 | |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 361 | // Compute the indices in the superheader, one for each bad block, and |
| 362 | // add them as successors. |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 363 | DenseMap<MachineBasicBlock *, unsigned> Indices; |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 364 | for (auto *Entry : SortedEntries) { |
| 365 | auto Pair = Indices.insert(std::make_pair(Entry, 0)); |
| 366 | assert(Pair.second); |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 367 | |
Dan Gohman | f456290 | 2016-04-26 01:40:56 +0000 | [diff] [blame] | 368 | unsigned Index = MIB.getInstr()->getNumExplicitOperands() - 1; |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 369 | Pair.first->second = Index; |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 370 | |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 371 | MIB.addMBB(Entry); |
| 372 | Dispatch->addSuccessor(Entry); |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 373 | } |
| 374 | |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 375 | // Rewrite the problematic successors for every block that wants to reach |
| 376 | // the bad blocks. For simplicity, we just introduce a new block for every |
| 377 | // edge we need to rewrite. (Fancier things are possible.) |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 378 | |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 379 | BlockVector AllPreds; |
| 380 | for (auto *Entry : SortedEntries) { |
| 381 | for (auto *Pred : Entry->predecessors()) { |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 382 | if (Pred != Dispatch) { |
| 383 | AllPreds.push_back(Pred); |
| 384 | } |
| 385 | } |
| 386 | } |
| 387 | |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 388 | for (MachineBasicBlock *Pred : AllPreds) { |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 389 | DenseMap<MachineBasicBlock *, MachineBasicBlock *> Map; |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 390 | for (auto *Entry : Pred->successors()) { |
| 391 | if (!Entries.count(Entry)) { |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 392 | continue; |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 393 | } |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 394 | |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 395 | // This is a successor we need to rewrite. |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 396 | MachineBasicBlock *Split = MF.CreateMachineBasicBlock(); |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 397 | MF.insert(Pred->isLayoutSuccessor(Entry) |
| 398 | ? MachineFunction::iterator(Entry) |
| 399 | : MF.end(), |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 400 | Split); |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 401 | Blocks.insert(Split); |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 402 | |
| 403 | // Set the jump table's register of the index of the block we wish to |
| 404 | // jump to, and jump to the jump table. |
Heejin Ahn | c60bc94 | 2019-03-19 05:26:33 +0000 | [diff] [blame] | 405 | BuildMI(Split, DebugLoc(), TII.get(WebAssembly::CONST_I32), Reg) |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 406 | .addImm(Indices[Entry]); |
Heejin Ahn | c60bc94 | 2019-03-19 05:26:33 +0000 | [diff] [blame] | 407 | BuildMI(Split, DebugLoc(), TII.get(WebAssembly::BR)).addMBB(Dispatch); |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 408 | Split->addSuccessor(Dispatch); |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 409 | Map[Entry] = Split; |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 410 | } |
| 411 | // Remap the terminator operands and the successor list. |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 412 | for (MachineInstr &Term : Pred->terminators()) |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 413 | for (auto &Op : Term.explicit_uses()) |
| 414 | if (Op.isMBB() && Indices.count(Op.getMBB())) |
| 415 | Op.setMBB(Map[Op.getMBB()]); |
| 416 | for (auto Rewrite : Map) |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 417 | Pred->replaceSuccessor(Rewrite.first, Rewrite.second); |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 418 | } |
| 419 | |
| 420 | // Create a fake default label, because br_table requires one. |
| 421 | MIB.addMBB(MIB.getInstr() |
| 422 | ->getOperand(MIB.getInstr()->getNumExplicitOperands() - 1) |
| 423 | .getMBB()); |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 424 | } |
| 425 | |
Heejin Ahn | 777d01c | 2019-01-03 23:10:11 +0000 | [diff] [blame] | 426 | } // end anonymous namespace |
| 427 | |
| 428 | char WebAssemblyFixIrreducibleControlFlow::ID = 0; |
| 429 | INITIALIZE_PASS(WebAssemblyFixIrreducibleControlFlow, DEBUG_TYPE, |
| 430 | "Removes irreducible control flow", false, false) |
| 431 | |
| 432 | FunctionPass *llvm::createWebAssemblyFixIrreducibleControlFlow() { |
| 433 | return new WebAssemblyFixIrreducibleControlFlow(); |
| 434 | } |
| 435 | |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 436 | bool WebAssemblyFixIrreducibleControlFlow::runOnMachineFunction( |
| 437 | MachineFunction &MF) { |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 438 | LLVM_DEBUG(dbgs() << "********** Fixing Irreducible Control Flow **********\n" |
| 439 | "********** Function: " |
| 440 | << MF.getName() << '\n'); |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 441 | |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 442 | // Start the recursive process on the entire function body. |
| 443 | BlockSet AllBlocks; |
| 444 | for (auto &MBB : MF) { |
| 445 | AllBlocks.insert(&MBB); |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 446 | } |
| 447 | |
Heejin Ahn | a41250c | 2019-03-16 03:00:19 +0000 | [diff] [blame] | 448 | if (LLVM_UNLIKELY(processRegion(&*MF.begin(), AllBlocks, MF))) { |
| 449 | // We rewrote part of the function; recompute relevant things. |
| 450 | MF.getRegInfo().invalidateLiveness(); |
| 451 | MF.RenumberBlocks(); |
| 452 | return true; |
| 453 | } |
| 454 | |
| 455 | return false; |
Dan Gohman | d7a2eea | 2016-03-09 02:01:14 +0000 | [diff] [blame] | 456 | } |