Misha Brukman | 03a1134 | 2004-02-28 03:33:01 +0000 | [diff] [blame] | 1 | //===- LoopExtractor.cpp - Extract each loop into a new function ----------===// |
Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 2 | // |
Chris Lattner | 692a47a | 2004-03-14 02:34:07 +0000 | [diff] [blame] | 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
Chris Lattner | f3ebc3f | 2007-12-29 20:36:04 +0000 | [diff] [blame] | 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 7 | // |
Chris Lattner | 692a47a | 2004-03-14 02:34:07 +0000 | [diff] [blame] | 8 | //===----------------------------------------------------------------------===// |
Misha Brukman | 03a1134 | 2004-02-28 03:33:01 +0000 | [diff] [blame] | 9 | // |
| 10 | // A pass wrapper around the ExtractLoop() scalar transformation to extract each |
| 11 | // top-level loop into its own new function. If the loop is the ONLY loop in a |
Misha Brukman | f272f9b | 2004-03-02 00:19:09 +0000 | [diff] [blame] | 12 | // given function, it is not touched. This is a pass most useful for debugging |
| 13 | // via bugpoint. |
Misha Brukman | 03a1134 | 2004-02-28 03:33:01 +0000 | [diff] [blame] | 14 | // |
| 15 | //===----------------------------------------------------------------------===// |
| 16 | |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 17 | #include "llvm/ADT/Statistic.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 18 | #include "llvm/Analysis/LoopPass.h" |
Chandler Carruth | 5ad5f15 | 2014-01-13 09:26:24 +0000 | [diff] [blame] | 19 | #include "llvm/IR/Dominators.h" |
Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 20 | #include "llvm/IR/Instructions.h" |
| 21 | #include "llvm/IR/Module.h" |
Misha Brukman | 03a1134 | 2004-02-28 03:33:01 +0000 | [diff] [blame] | 22 | #include "llvm/Pass.h" |
Nick Lewycky | c624302 | 2007-11-14 06:47:06 +0000 | [diff] [blame] | 23 | #include "llvm/Support/CommandLine.h" |
Chandler Carruth | 6bda14b | 2017-06-06 11:49:48 +0000 | [diff] [blame] | 24 | #include "llvm/Transforms/IPO.h" |
Chris Lattner | 6c3e8c7 | 2004-03-14 04:01:06 +0000 | [diff] [blame] | 25 | #include "llvm/Transforms/Scalar.h" |
Bill Wendling | c1da6ea | 2011-09-20 19:10:24 +0000 | [diff] [blame] | 26 | #include "llvm/Transforms/Utils/BasicBlockUtils.h" |
Chandler Carruth | 0fde001 | 2012-05-04 10:18:49 +0000 | [diff] [blame] | 27 | #include "llvm/Transforms/Utils/CodeExtractor.h" |
Nick Lewycky | c624302 | 2007-11-14 06:47:06 +0000 | [diff] [blame] | 28 | #include <fstream> |
| 29 | #include <set> |
Misha Brukman | 03a1134 | 2004-02-28 03:33:01 +0000 | [diff] [blame] | 30 | using namespace llvm; |
| 31 | |
Chandler Carruth | 964daaa | 2014-04-22 02:55:47 +0000 | [diff] [blame] | 32 | #define DEBUG_TYPE "loop-extract" |
| 33 | |
Chris Lattner | 1631bcb | 2006-12-19 22:09:18 +0000 | [diff] [blame] | 34 | STATISTIC(NumExtracted, "Number of loops extracted"); |
Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 35 | |
Chris Lattner | 1631bcb | 2006-12-19 22:09:18 +0000 | [diff] [blame] | 36 | namespace { |
Nick Lewycky | 02d5f77 | 2009-10-25 06:33:48 +0000 | [diff] [blame] | 37 | struct LoopExtractor : public LoopPass { |
Nick Lewycky | e7da2d6 | 2007-05-06 13:37:16 +0000 | [diff] [blame] | 38 | static char ID; // Pass identification, replacement for typeid |
Chris Lattner | a1672c1 | 2004-03-14 20:01:36 +0000 | [diff] [blame] | 39 | unsigned NumLoops; |
| 40 | |
Justin Bogner | e9fb228d | 2016-01-08 19:08:53 +0000 | [diff] [blame] | 41 | explicit LoopExtractor(unsigned numLoops = ~0) |
Owen Anderson | 6c18d1a | 2010-10-19 17:21:58 +0000 | [diff] [blame] | 42 | : LoopPass(ID), NumLoops(numLoops) { |
| 43 | initializeLoopExtractorPass(*PassRegistry::getPassRegistry()); |
| 44 | } |
Chris Lattner | a1672c1 | 2004-03-14 20:01:36 +0000 | [diff] [blame] | 45 | |
Justin Bogner | 883a3ea | 2015-12-16 18:40:20 +0000 | [diff] [blame] | 46 | bool runOnLoop(Loop *L, LPPassManager &) override; |
Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 47 | |
Craig Topper | 3e4c697 | 2014-03-05 09:10:37 +0000 | [diff] [blame] | 48 | void getAnalysisUsage(AnalysisUsage &AU) const override { |
Chris Lattner | 5bce0c8 | 2004-03-18 05:43:18 +0000 | [diff] [blame] | 49 | AU.addRequiredID(BreakCriticalEdgesID); |
| 50 | AU.addRequiredID(LoopSimplifyID); |
Chandler Carruth | 7352302 | 2014-01-13 13:07:17 +0000 | [diff] [blame] | 51 | AU.addRequired<DominatorTreeWrapperPass>(); |
Justin Bogner | 883a3ea | 2015-12-16 18:40:20 +0000 | [diff] [blame] | 52 | AU.addRequired<LoopInfoWrapperPass>(); |
Chris Lattner | 692a47a | 2004-03-14 02:34:07 +0000 | [diff] [blame] | 53 | } |
| 54 | }; |
Alexander Kornienko | f00654e | 2015-06-23 09:49:53 +0000 | [diff] [blame] | 55 | } |
Misha Brukman | 03a1134 | 2004-02-28 03:33:01 +0000 | [diff] [blame] | 56 | |
Dan Gohman | d78c400 | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 57 | char LoopExtractor::ID = 0; |
Owen Anderson | 8ac477f | 2010-10-12 19:48:12 +0000 | [diff] [blame] | 58 | INITIALIZE_PASS_BEGIN(LoopExtractor, "loop-extract", |
Bill Wendling | c1da6ea | 2011-09-20 19:10:24 +0000 | [diff] [blame] | 59 | "Extract loops into new functions", false, false) |
Owen Anderson | 8ac477f | 2010-10-12 19:48:12 +0000 | [diff] [blame] | 60 | INITIALIZE_PASS_DEPENDENCY(BreakCriticalEdges) |
| 61 | INITIALIZE_PASS_DEPENDENCY(LoopSimplify) |
Chandler Carruth | 7352302 | 2014-01-13 13:07:17 +0000 | [diff] [blame] | 62 | INITIALIZE_PASS_DEPENDENCY(DominatorTreeWrapperPass) |
Owen Anderson | 8ac477f | 2010-10-12 19:48:12 +0000 | [diff] [blame] | 63 | INITIALIZE_PASS_END(LoopExtractor, "loop-extract", |
Bill Wendling | c1da6ea | 2011-09-20 19:10:24 +0000 | [diff] [blame] | 64 | "Extract loops into new functions", false, false) |
Chris Lattner | a1672c1 | 2004-03-14 20:01:36 +0000 | [diff] [blame] | 65 | |
Dan Gohman | d78c400 | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 66 | namespace { |
Chris Lattner | a1672c1 | 2004-03-14 20:01:36 +0000 | [diff] [blame] | 67 | /// SingleLoopExtractor - For bugpoint. |
| 68 | struct SingleLoopExtractor : public LoopExtractor { |
Nick Lewycky | e7da2d6 | 2007-05-06 13:37:16 +0000 | [diff] [blame] | 69 | static char ID; // Pass identification, replacement for typeid |
Chris Lattner | a1672c1 | 2004-03-14 20:01:36 +0000 | [diff] [blame] | 70 | SingleLoopExtractor() : LoopExtractor(1) {} |
| 71 | }; |
Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 72 | } // End anonymous namespace |
Misha Brukman | 03a1134 | 2004-02-28 03:33:01 +0000 | [diff] [blame] | 73 | |
Dan Gohman | d78c400 | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 74 | char SingleLoopExtractor::ID = 0; |
Owen Anderson | a57b97e | 2010-07-21 22:09:45 +0000 | [diff] [blame] | 75 | INITIALIZE_PASS(SingleLoopExtractor, "loop-extract-single", |
Owen Anderson | df7a4f2 | 2010-10-07 22:25:06 +0000 | [diff] [blame] | 76 | "Extract at most one loop into a new function", false, false) |
Dan Gohman | d78c400 | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 77 | |
Jeff Cohen | 677babc | 2005-01-08 17:21:40 +0000 | [diff] [blame] | 78 | // createLoopExtractorPass - This pass extracts all natural loops from the |
| 79 | // program into a function if it can. |
| 80 | // |
Dan Gohman | 9a7320c | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 81 | Pass *llvm::createLoopExtractorPass() { return new LoopExtractor(); } |
Jeff Cohen | 677babc | 2005-01-08 17:21:40 +0000 | [diff] [blame] | 82 | |
Sanjoy Das | def1729 | 2017-09-28 02:45:42 +0000 | [diff] [blame^] | 83 | bool LoopExtractor::runOnLoop(Loop *L, LPPassManager &LPM) { |
Andrew Kaylor | aa641a5 | 2016-04-22 22:06:11 +0000 | [diff] [blame] | 84 | if (skipLoop(L)) |
Paul Robinson | af4e64d | 2014-02-06 00:07:05 +0000 | [diff] [blame] | 85 | return false; |
| 86 | |
Dan Gohman | 9a7320c | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 87 | // Only visit top-level loops. |
| 88 | if (L->getParentLoop()) |
Misha Brukman | 03a1134 | 2004-02-28 03:33:01 +0000 | [diff] [blame] | 89 | return false; |
| 90 | |
Dan Gohman | a83ac2d | 2009-11-05 21:11:53 +0000 | [diff] [blame] | 91 | // If LoopSimplify form is not available, stay out of trouble. |
| 92 | if (!L->isLoopSimplifyForm()) |
| 93 | return false; |
| 94 | |
Chandler Carruth | 7352302 | 2014-01-13 13:07:17 +0000 | [diff] [blame] | 95 | DominatorTree &DT = getAnalysis<DominatorTreeWrapperPass>().getDomTree(); |
Justin Bogner | 883a3ea | 2015-12-16 18:40:20 +0000 | [diff] [blame] | 96 | LoopInfo &LI = getAnalysis<LoopInfoWrapperPass>().getLoopInfo(); |
Dan Gohman | 9a7320c | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 97 | bool Changed = false; |
Chris Lattner | e9235d2 | 2004-03-18 03:48:06 +0000 | [diff] [blame] | 98 | |
Chris Lattner | 2f155d8 | 2004-03-15 00:02:02 +0000 | [diff] [blame] | 99 | // If there is more than one top-level loop in this function, extract all of |
Dan Gohman | 9a7320c | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 100 | // the loops. Otherwise there is exactly one top-level loop; in this case if |
| 101 | // this function is more than a minimal wrapper around the loop, extract |
| 102 | // the loop. |
| 103 | bool ShouldExtractLoop = false; |
Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 104 | |
Dan Gohman | 9a7320c | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 105 | // Extract the loop if the entry block doesn't branch to the loop header. |
| 106 | TerminatorInst *EntryTI = |
| 107 | L->getHeader()->getParent()->getEntryBlock().getTerminator(); |
| 108 | if (!isa<BranchInst>(EntryTI) || |
| 109 | !cast<BranchInst>(EntryTI)->isUnconditional() || |
Bill Wendling | 0058520 | 2011-09-20 22:23:09 +0000 | [diff] [blame] | 110 | EntryTI->getSuccessor(0) != L->getHeader()) { |
Dan Gohman | 9a7320c | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 111 | ShouldExtractLoop = true; |
Bill Wendling | 0058520 | 2011-09-20 22:23:09 +0000 | [diff] [blame] | 112 | } else { |
Dan Gohman | 9a7320c | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 113 | // Check to see if any exits from the loop are more than just return |
Bill Wendling | 04289fc | 2011-09-20 22:27:16 +0000 | [diff] [blame] | 114 | // blocks. |
| 115 | SmallVector<BasicBlock*, 8> ExitBlocks; |
| 116 | L->getExitBlocks(ExitBlocks); |
| 117 | for (unsigned i = 0, e = ExitBlocks.size(); i != e; ++i) |
| 118 | if (!isa<ReturnInst>(ExitBlocks[i]->getTerminator())) { |
| 119 | ShouldExtractLoop = true; |
| 120 | break; |
| 121 | } |
| 122 | } |
| 123 | |
| 124 | if (ShouldExtractLoop) { |
David Majnemer | eb518bd | 2015-08-04 08:21:40 +0000 | [diff] [blame] | 125 | // We must omit EH pads. EH pads must accompany the invoke |
Bill Wendling | 04289fc | 2011-09-20 22:27:16 +0000 | [diff] [blame] | 126 | // instruction. But this would result in a loop in the extracted |
Bill Wendling | 0058520 | 2011-09-20 22:23:09 +0000 | [diff] [blame] | 127 | // function. An infinite cycle occurs when it tries to extract that loop as |
| 128 | // well. |
Dan Gohman | 9a7320c | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 129 | SmallVector<BasicBlock*, 8> ExitBlocks; |
| 130 | L->getExitBlocks(ExitBlocks); |
Bill Wendling | 04289fc | 2011-09-20 22:27:16 +0000 | [diff] [blame] | 131 | for (unsigned i = 0, e = ExitBlocks.size(); i != e; ++i) |
David Majnemer | eb518bd | 2015-08-04 08:21:40 +0000 | [diff] [blame] | 132 | if (ExitBlocks[i]->isEHPad()) { |
Bill Wendling | 0058520 | 2011-09-20 22:23:09 +0000 | [diff] [blame] | 133 | ShouldExtractLoop = false; |
Dan Gohman | 9a7320c | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 134 | break; |
Chris Lattner | 2f155d8 | 2004-03-15 00:02:02 +0000 | [diff] [blame] | 135 | } |
Dan Gohman | 9a7320c | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 136 | } |
Bill Wendling | 04289fc | 2011-09-20 22:27:16 +0000 | [diff] [blame] | 137 | |
Dan Gohman | 9a7320c | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 138 | if (ShouldExtractLoop) { |
| 139 | if (NumLoops == 0) return Changed; |
| 140 | --NumLoops; |
Chandler Carruth | 0fde001 | 2012-05-04 10:18:49 +0000 | [diff] [blame] | 141 | CodeExtractor Extractor(DT, *L); |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 142 | if (Extractor.extractCodeRegion() != nullptr) { |
Dan Gohman | 9a7320c | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 143 | Changed = true; |
| 144 | // After extraction, the loop is replaced by a function call, so |
| 145 | // we shouldn't try to run any more loop passes on it. |
Sanjoy Das | def1729 | 2017-09-28 02:45:42 +0000 | [diff] [blame^] | 146 | LPM.markLoopAsDeleted(*L); |
Sanjoy Das | 388b012 | 2017-09-22 01:47:41 +0000 | [diff] [blame] | 147 | LI.erase(L); |
Chris Lattner | 2f155d8 | 2004-03-15 00:02:02 +0000 | [diff] [blame] | 148 | } |
Dan Gohman | 9a7320c | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 149 | ++NumExtracted; |
Chris Lattner | a1672c1 | 2004-03-14 20:01:36 +0000 | [diff] [blame] | 150 | } |
Misha Brukman | 03a1134 | 2004-02-28 03:33:01 +0000 | [diff] [blame] | 151 | |
| 152 | return Changed; |
| 153 | } |
| 154 | |
Chris Lattner | a1672c1 | 2004-03-14 20:01:36 +0000 | [diff] [blame] | 155 | // createSingleLoopExtractorPass - This pass extracts one natural loop from the |
| 156 | // program into a function if it can. This is used by bugpoint. |
| 157 | // |
Dan Gohman | 9a7320c | 2009-09-28 14:37:51 +0000 | [diff] [blame] | 158 | Pass *llvm::createSingleLoopExtractorPass() { |
Chris Lattner | a1672c1 | 2004-03-14 20:01:36 +0000 | [diff] [blame] | 159 | return new SingleLoopExtractor(); |
Misha Brukman | 03a1134 | 2004-02-28 03:33:01 +0000 | [diff] [blame] | 160 | } |
Chris Lattner | 7386e63 | 2004-08-13 03:05:17 +0000 | [diff] [blame] | 161 | |
| 162 | |
Dan Gohman | d78c400 | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 163 | // BlockFile - A file which contains a list of blocks that should not be |
| 164 | // extracted. |
| 165 | static cl::opt<std::string> |
| 166 | BlockFile("extract-blocks-file", cl::value_desc("filename"), |
| 167 | cl::desc("A file containing list of basic blocks to not extract"), |
| 168 | cl::Hidden); |
Nick Lewycky | c624302 | 2007-11-14 06:47:06 +0000 | [diff] [blame] | 169 | |
Dan Gohman | d78c400 | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 170 | namespace { |
Chris Lattner | 7386e63 | 2004-08-13 03:05:17 +0000 | [diff] [blame] | 171 | /// BlockExtractorPass - This pass is used by bugpoint to extract all blocks |
| 172 | /// from the module into their own functions except for those specified by the |
| 173 | /// BlocksToNotExtract list. |
Chris Lattner | 4f2cf03 | 2004-09-20 04:48:05 +0000 | [diff] [blame] | 174 | class BlockExtractorPass : public ModulePass { |
Nick Lewycky | c624302 | 2007-11-14 06:47:06 +0000 | [diff] [blame] | 175 | void LoadFile(const char *Filename); |
Bill Wendling | c1da6ea | 2011-09-20 19:10:24 +0000 | [diff] [blame] | 176 | void SplitLandingPadPreds(Function *F); |
Nick Lewycky | c624302 | 2007-11-14 06:47:06 +0000 | [diff] [blame] | 177 | |
Chris Lattner | 7386e63 | 2004-08-13 03:05:17 +0000 | [diff] [blame] | 178 | std::vector<BasicBlock*> BlocksToNotExtract; |
Nick Lewycky | c624302 | 2007-11-14 06:47:06 +0000 | [diff] [blame] | 179 | std::vector<std::pair<std::string, std::string> > BlocksToNotExtractByName; |
Chris Lattner | 7386e63 | 2004-08-13 03:05:17 +0000 | [diff] [blame] | 180 | public: |
Nick Lewycky | e7da2d6 | 2007-05-06 13:37:16 +0000 | [diff] [blame] | 181 | static char ID; // Pass identification, replacement for typeid |
Owen Anderson | a7aed18 | 2010-08-06 18:33:48 +0000 | [diff] [blame] | 182 | BlockExtractorPass() : ModulePass(ID) { |
Nick Lewycky | c624302 | 2007-11-14 06:47:06 +0000 | [diff] [blame] | 183 | if (!BlockFile.empty()) |
| 184 | LoadFile(BlockFile.c_str()); |
| 185 | } |
Chris Lattner | 7386e63 | 2004-08-13 03:05:17 +0000 | [diff] [blame] | 186 | |
Craig Topper | 3e4c697 | 2014-03-05 09:10:37 +0000 | [diff] [blame] | 187 | bool runOnModule(Module &M) override; |
Chris Lattner | 7386e63 | 2004-08-13 03:05:17 +0000 | [diff] [blame] | 188 | }; |
Alexander Kornienko | f00654e | 2015-06-23 09:49:53 +0000 | [diff] [blame] | 189 | } |
Chris Lattner | 7386e63 | 2004-08-13 03:05:17 +0000 | [diff] [blame] | 190 | |
Dan Gohman | d78c400 | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 191 | char BlockExtractorPass::ID = 0; |
Owen Anderson | a57b97e | 2010-07-21 22:09:45 +0000 | [diff] [blame] | 192 | INITIALIZE_PASS(BlockExtractorPass, "extract-blocks", |
| 193 | "Extract Basic Blocks From Module (for bugpoint use)", |
Owen Anderson | df7a4f2 | 2010-10-07 22:25:06 +0000 | [diff] [blame] | 194 | false, false) |
Dan Gohman | d78c400 | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 195 | |
Chris Lattner | 7386e63 | 2004-08-13 03:05:17 +0000 | [diff] [blame] | 196 | // createBlockExtractorPass - This pass extracts all blocks (except those |
| 197 | // specified in the argument list) from the functions in the module. |
| 198 | // |
Bill Wendling | c1da6ea | 2011-09-20 19:10:24 +0000 | [diff] [blame] | 199 | ModulePass *llvm::createBlockExtractorPass() { |
Rafael Espindola | 40f1883 | 2010-07-31 00:32:17 +0000 | [diff] [blame] | 200 | return new BlockExtractorPass(); |
Chris Lattner | 7386e63 | 2004-08-13 03:05:17 +0000 | [diff] [blame] | 201 | } |
| 202 | |
Nick Lewycky | c624302 | 2007-11-14 06:47:06 +0000 | [diff] [blame] | 203 | void BlockExtractorPass::LoadFile(const char *Filename) { |
| 204 | // Load the BlockFile... |
| 205 | std::ifstream In(Filename); |
| 206 | if (!In.good()) { |
Chris Lattner | 4883d90 | 2009-08-23 07:19:13 +0000 | [diff] [blame] | 207 | errs() << "WARNING: BlockExtractor couldn't load file '" << Filename |
| 208 | << "'!\n"; |
Nick Lewycky | c624302 | 2007-11-14 06:47:06 +0000 | [diff] [blame] | 209 | return; |
| 210 | } |
| 211 | while (In) { |
| 212 | std::string FunctionName, BlockName; |
| 213 | In >> FunctionName; |
| 214 | In >> BlockName; |
| 215 | if (!BlockName.empty()) |
| 216 | BlocksToNotExtractByName.push_back( |
| 217 | std::make_pair(FunctionName, BlockName)); |
| 218 | } |
| 219 | } |
| 220 | |
Bill Wendling | c1da6ea | 2011-09-20 19:10:24 +0000 | [diff] [blame] | 221 | /// SplitLandingPadPreds - The landing pad needs to be extracted with the invoke |
| 222 | /// instruction. The critical edge breaker will refuse to break critical edges |
| 223 | /// to a landing pad. So do them here. After this method runs, all landing pads |
| 224 | /// should have only one predecessor. |
| 225 | void BlockExtractorPass::SplitLandingPadPreds(Function *F) { |
| 226 | for (Function::iterator I = F->begin(), E = F->end(); I != E; ++I) { |
| 227 | InvokeInst *II = dyn_cast<InvokeInst>(I); |
| 228 | if (!II) continue; |
| 229 | BasicBlock *Parent = II->getParent(); |
| 230 | BasicBlock *LPad = II->getUnwindDest(); |
| 231 | |
| 232 | // Look through the landing pad's predecessors. If one of them ends in an |
| 233 | // 'invoke', then we want to split the landing pad. |
| 234 | bool Split = false; |
Duncan P. N. Exon Smith | 6c99015 | 2014-07-21 17:06:51 +0000 | [diff] [blame] | 235 | for (pred_iterator |
| 236 | PI = pred_begin(LPad), PE = pred_end(LPad); PI != PE; ++PI) { |
| 237 | BasicBlock *BB = *PI; |
Bill Wendling | c1da6ea | 2011-09-20 19:10:24 +0000 | [diff] [blame] | 238 | if (BB->isLandingPad() && BB != Parent && |
| 239 | isa<InvokeInst>(Parent->getTerminator())) { |
| 240 | Split = true; |
| 241 | break; |
| 242 | } |
| 243 | } |
| 244 | |
| 245 | if (!Split) continue; |
| 246 | |
| 247 | SmallVector<BasicBlock*, 2> NewBBs; |
Chandler Carruth | 0eae112 | 2015-01-19 03:03:39 +0000 | [diff] [blame] | 248 | SplitLandingPadPredecessors(LPad, Parent, ".1", ".2", NewBBs); |
Bill Wendling | c1da6ea | 2011-09-20 19:10:24 +0000 | [diff] [blame] | 249 | } |
| 250 | } |
| 251 | |
Chris Lattner | 4f2cf03 | 2004-09-20 04:48:05 +0000 | [diff] [blame] | 252 | bool BlockExtractorPass::runOnModule(Module &M) { |
Andrew Kaylor | aa641a5 | 2016-04-22 22:06:11 +0000 | [diff] [blame] | 253 | if (skipModule(M)) |
| 254 | return false; |
| 255 | |
Chris Lattner | 7386e63 | 2004-08-13 03:05:17 +0000 | [diff] [blame] | 256 | std::set<BasicBlock*> TranslatedBlocksToNotExtract; |
| 257 | for (unsigned i = 0, e = BlocksToNotExtract.size(); i != e; ++i) { |
| 258 | BasicBlock *BB = BlocksToNotExtract[i]; |
| 259 | Function *F = BB->getParent(); |
| 260 | |
| 261 | // Map the corresponding function in this module. |
Reid Spencer | 3aaaa0b | 2007-02-05 20:47:22 +0000 | [diff] [blame] | 262 | Function *MF = M.getFunction(F->getName()); |
| 263 | assert(MF->getFunctionType() == F->getFunctionType() && "Wrong function?"); |
Chris Lattner | 7386e63 | 2004-08-13 03:05:17 +0000 | [diff] [blame] | 264 | |
| 265 | // Figure out which index the basic block is in its function. |
| 266 | Function::iterator BBI = MF->begin(); |
| 267 | std::advance(BBI, std::distance(F->begin(), Function::iterator(BB))); |
Duncan P. N. Exon Smith | 1732340 | 2015-10-13 17:51:03 +0000 | [diff] [blame] | 268 | TranslatedBlocksToNotExtract.insert(&*BBI); |
Chris Lattner | 7386e63 | 2004-08-13 03:05:17 +0000 | [diff] [blame] | 269 | } |
| 270 | |
Nick Lewycky | c624302 | 2007-11-14 06:47:06 +0000 | [diff] [blame] | 271 | while (!BlocksToNotExtractByName.empty()) { |
| 272 | // There's no way to find BBs by name without looking at every BB inside |
| 273 | // every Function. Fortunately, this is always empty except when used by |
| 274 | // bugpoint in which case correctness is more important than performance. |
| 275 | |
| 276 | std::string &FuncName = BlocksToNotExtractByName.back().first; |
| 277 | std::string &BlockName = BlocksToNotExtractByName.back().second; |
| 278 | |
Benjamin Kramer | 135f735 | 2016-06-26 12:28:59 +0000 | [diff] [blame] | 279 | for (Function &F : M) { |
Nick Lewycky | c624302 | 2007-11-14 06:47:06 +0000 | [diff] [blame] | 280 | if (F.getName() != FuncName) continue; |
| 281 | |
Benjamin Kramer | 135f735 | 2016-06-26 12:28:59 +0000 | [diff] [blame] | 282 | for (BasicBlock &BB : F) { |
Nick Lewycky | c624302 | 2007-11-14 06:47:06 +0000 | [diff] [blame] | 283 | if (BB.getName() != BlockName) continue; |
| 284 | |
Benjamin Kramer | 135f735 | 2016-06-26 12:28:59 +0000 | [diff] [blame] | 285 | TranslatedBlocksToNotExtract.insert(&BB); |
Nick Lewycky | c624302 | 2007-11-14 06:47:06 +0000 | [diff] [blame] | 286 | } |
| 287 | } |
| 288 | |
| 289 | BlocksToNotExtractByName.pop_back(); |
| 290 | } |
| 291 | |
Chris Lattner | 7386e63 | 2004-08-13 03:05:17 +0000 | [diff] [blame] | 292 | // Now that we know which blocks to not extract, figure out which ones we WANT |
| 293 | // to extract. |
| 294 | std::vector<BasicBlock*> BlocksToExtract; |
Benjamin Kramer | 135f735 | 2016-06-26 12:28:59 +0000 | [diff] [blame] | 295 | for (Function &F : M) { |
| 296 | SplitLandingPadPreds(&F); |
| 297 | for (BasicBlock &BB : F) |
| 298 | if (!TranslatedBlocksToNotExtract.count(&BB)) |
| 299 | BlocksToExtract.push_back(&BB); |
Bill Wendling | c1da6ea | 2011-09-20 19:10:24 +0000 | [diff] [blame] | 300 | } |
Chris Lattner | 7386e63 | 2004-08-13 03:05:17 +0000 | [diff] [blame] | 301 | |
Benjamin Kramer | 135f735 | 2016-06-26 12:28:59 +0000 | [diff] [blame] | 302 | for (BasicBlock *BlockToExtract : BlocksToExtract) { |
Bill Wendling | c1da6ea | 2011-09-20 19:10:24 +0000 | [diff] [blame] | 303 | SmallVector<BasicBlock*, 2> BlocksToExtractVec; |
Benjamin Kramer | 135f735 | 2016-06-26 12:28:59 +0000 | [diff] [blame] | 304 | BlocksToExtractVec.push_back(BlockToExtract); |
Bill Wendling | 3d48f59 | 2011-09-20 20:20:50 +0000 | [diff] [blame] | 305 | if (const InvokeInst *II = |
Benjamin Kramer | 135f735 | 2016-06-26 12:28:59 +0000 | [diff] [blame] | 306 | dyn_cast<InvokeInst>(BlockToExtract->getTerminator())) |
Bill Wendling | c1da6ea | 2011-09-20 19:10:24 +0000 | [diff] [blame] | 307 | BlocksToExtractVec.push_back(II->getUnwindDest()); |
Chandler Carruth | 0fde001 | 2012-05-04 10:18:49 +0000 | [diff] [blame] | 308 | CodeExtractor(BlocksToExtractVec).extractCodeRegion(); |
Bill Wendling | c1da6ea | 2011-09-20 19:10:24 +0000 | [diff] [blame] | 309 | } |
Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 310 | |
Chris Lattner | 7386e63 | 2004-08-13 03:05:17 +0000 | [diff] [blame] | 311 | return !BlocksToExtract.empty(); |
| 312 | } |