Chris Lattner | 4186ad5 | 2008-05-14 20:38:44 +0000 | [diff] [blame] | 1 | //===- SimplifyCFGPass.cpp - CFG Simplification Pass ----------------------===// |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 2 | // |
John Criswell | b576c94 | 2003-10-20 19:43:21 +0000 | [diff] [blame] | 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
Chris Lattner | 4ee451d | 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 | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 7 | // |
John Criswell | b576c94 | 2003-10-20 19:43:21 +0000 | [diff] [blame] | 8 | //===----------------------------------------------------------------------===// |
Chris Lattner | 573527b | 2002-05-21 20:49:37 +0000 | [diff] [blame] | 9 | // |
Chris Lattner | 4cf4b69 | 2007-12-03 19:43:18 +0000 | [diff] [blame] | 10 | // This file implements dead code elimination and basic block merging, along |
| 11 | // with a collection of other peephole control flow optimizations. For example: |
Chris Lattner | 573527b | 2002-05-21 20:49:37 +0000 | [diff] [blame] | 12 | // |
Gordon Henriksen | c86b677 | 2007-11-04 16:15:04 +0000 | [diff] [blame] | 13 | // * Removes basic blocks with no predecessors. |
| 14 | // * Merges a basic block into its predecessor if there is only one and the |
Chris Lattner | 573527b | 2002-05-21 20:49:37 +0000 | [diff] [blame] | 15 | // predecessor only has one successor. |
Gordon Henriksen | c86b677 | 2007-11-04 16:15:04 +0000 | [diff] [blame] | 16 | // * Eliminates PHI nodes for basic blocks with a single predecessor. |
| 17 | // * Eliminates a basic block that only contains an unconditional branch. |
Chris Lattner | 4cf4b69 | 2007-12-03 19:43:18 +0000 | [diff] [blame] | 18 | // * Changes invoke instructions to nounwind functions to be calls. |
| 19 | // * Change things like "if (x) if (y)" into "if (x&y)". |
| 20 | // * etc.. |
Chris Lattner | 573527b | 2002-05-21 20:49:37 +0000 | [diff] [blame] | 21 | // |
| 22 | //===----------------------------------------------------------------------===// |
| 23 | |
Chris Lattner | 0e5f499 | 2006-12-19 21:40:18 +0000 | [diff] [blame] | 24 | #define DEBUG_TYPE "simplifycfg" |
Chris Lattner | 573527b | 2002-05-21 20:49:37 +0000 | [diff] [blame] | 25 | #include "llvm/Transforms/Scalar.h" |
Chandler Carruth | d04a8d4 | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 26 | #include "llvm/ADT/SmallPtrSet.h" |
| 27 | #include "llvm/ADT/SmallVector.h" |
| 28 | #include "llvm/ADT/Statistic.h" |
Chandler Carruth | be04929 | 2013-01-07 03:08:10 +0000 | [diff] [blame] | 29 | #include "llvm/Analysis/TargetTransformInfo.h" |
Tom Stellard | 57e6b2d | 2013-07-27 00:01:07 +0000 | [diff] [blame] | 30 | #include "llvm/Analysis/AliasAnalysis.h" |
Chandler Carruth | 0b8c9a8 | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 31 | #include "llvm/IR/Attributes.h" |
| 32 | #include "llvm/IR/Constants.h" |
| 33 | #include "llvm/IR/DataLayout.h" |
| 34 | #include "llvm/IR/Instructions.h" |
| 35 | #include "llvm/IR/IntrinsicInst.h" |
| 36 | #include "llvm/IR/Module.h" |
Chris Lattner | 573527b | 2002-05-21 20:49:37 +0000 | [diff] [blame] | 37 | #include "llvm/Pass.h" |
Chandler Carruth | d04a8d4 | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 38 | #include "llvm/Support/CFG.h" |
Chandler Carruth | d04a8d4 | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 39 | #include "llvm/Transforms/Utils/Local.h" |
Chris Lattner | d745602 | 2004-01-09 06:02:20 +0000 | [diff] [blame] | 40 | using namespace llvm; |
Brian Gaeke | d0fde30 | 2003-11-11 22:41:34 +0000 | [diff] [blame] | 41 | |
Chris Lattner | 0e5f499 | 2006-12-19 21:40:18 +0000 | [diff] [blame] | 42 | STATISTIC(NumSimpl, "Number of blocks simplified"); |
Chris Lattner | a92f696 | 2002-10-01 22:38:41 +0000 | [diff] [blame] | 43 | |
Chris Lattner | 0e5f499 | 2006-12-19 21:40:18 +0000 | [diff] [blame] | 44 | namespace { |
Tom Stellard | 57e6b2d | 2013-07-27 00:01:07 +0000 | [diff] [blame] | 45 | struct CFGSimplifyPass : public FunctionPass { |
| 46 | CFGSimplifyPass(char &ID, bool isTargetAware) |
| 47 | : FunctionPass(ID), IsTargetAware(isTargetAware) {} |
| 48 | virtual bool runOnFunction(Function &F); |
Devang Patel | 794fd75 | 2007-05-01 21:15:47 +0000 | [diff] [blame] | 49 | |
Tom Stellard | 57e6b2d | 2013-07-27 00:01:07 +0000 | [diff] [blame] | 50 | virtual void getAnalysisUsage(AnalysisUsage &AU) const { |
| 51 | AU.addRequired<TargetTransformInfo>(); |
| 52 | } |
| 53 | private: |
| 54 | AliasAnalysis *AA; |
| 55 | bool IsTargetAware; // Should the pass be target-aware? |
| 56 | }; |
Chandler Carruth | 5f46c3c | 2013-01-07 03:53:25 +0000 | [diff] [blame] | 57 | |
Tom Stellard | 57e6b2d | 2013-07-27 00:01:07 +0000 | [diff] [blame] | 58 | // CFGSimplifyPass that does optimizations. |
| 59 | struct CFGOptimize : public CFGSimplifyPass { |
| 60 | static char ID; // Pass identification, replacement for typeid |
| 61 | public: |
| 62 | CFGOptimize() : CFGSimplifyPass(ID, true) { |
| 63 | initializeCFGOptimizePass(*PassRegistry::getPassRegistry()); |
| 64 | } |
| 65 | virtual void getAnalysisUsage(AnalysisUsage &AU) const { |
| 66 | AU.addRequired<TargetTransformInfo>(); |
| 67 | AU.addRequired<AliasAnalysis>(); |
| 68 | } |
| 69 | }; |
| 70 | |
| 71 | // CFGSimplifyPass that does canonicalizations. |
| 72 | struct CFGCanonicalize : public CFGSimplifyPass { |
| 73 | static char ID; // Pass identification, replacement for typeid |
| 74 | public: |
| 75 | CFGCanonicalize() : CFGSimplifyPass(ID, false) { |
| 76 | initializeCFGCanonicalizePass(*PassRegistry::getPassRegistry()); |
| 77 | } |
| 78 | }; |
Chris Lattner | 573527b | 2002-05-21 20:49:37 +0000 | [diff] [blame] | 79 | } |
| 80 | |
Tom Stellard | 57e6b2d | 2013-07-27 00:01:07 +0000 | [diff] [blame] | 81 | char CFGCanonicalize::ID = 0; |
| 82 | char CFGOptimize::ID = 0; |
| 83 | INITIALIZE_PASS_BEGIN(CFGCanonicalize, "simplifycfg", "Simplify the CFG", false, |
| 84 | false) |
Chandler Carruth | 5f46c3c | 2013-01-07 03:53:25 +0000 | [diff] [blame] | 85 | INITIALIZE_AG_DEPENDENCY(TargetTransformInfo) |
Tom Stellard | 57e6b2d | 2013-07-27 00:01:07 +0000 | [diff] [blame] | 86 | INITIALIZE_PASS_END(CFGCanonicalize, "simplifycfg", "Simplify the CFG", false, |
| 87 | false) |
| 88 | INITIALIZE_PASS_BEGIN(CFGOptimize, "optimizecfg", "optimize the CFG", false, |
| 89 | false) |
| 90 | INITIALIZE_AG_DEPENDENCY(TargetTransformInfo) |
| 91 | INITIALIZE_AG_DEPENDENCY(AliasAnalysis) |
| 92 | INITIALIZE_PASS_END(CFGOptimize, "optimizecfg", "Optimize the CFG", false, |
| 93 | false) |
Dan Gohman | 844731a | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 94 | |
Brian Gaeke | d0fde30 | 2003-11-11 22:41:34 +0000 | [diff] [blame] | 95 | // Public interface to the CFGSimplification pass |
Tom Stellard | 57e6b2d | 2013-07-27 00:01:07 +0000 | [diff] [blame] | 96 | FunctionPass *llvm::createCFGSimplificationPass(bool IsTargetAware) { |
| 97 | if (IsTargetAware) |
| 98 | return new CFGOptimize(); |
| 99 | else |
| 100 | return new CFGCanonicalize(); |
Chris Lattner | 573527b | 2002-05-21 20:49:37 +0000 | [diff] [blame] | 101 | } |
| 102 | |
Jim Grosbach | 557a20a | 2012-09-06 00:59:08 +0000 | [diff] [blame] | 103 | /// changeToUnreachable - Insert an unreachable instruction before the specified |
Chris Lattner | 1b12d88 | 2007-11-14 06:19:25 +0000 | [diff] [blame] | 104 | /// instruction, making it and the rest of the code in the block dead. |
Jim Grosbach | 557a20a | 2012-09-06 00:59:08 +0000 | [diff] [blame] | 105 | static void changeToUnreachable(Instruction *I, bool UseLLVMTrap) { |
Chris Lattner | 1b12d88 | 2007-11-14 06:19:25 +0000 | [diff] [blame] | 106 | BasicBlock *BB = I->getParent(); |
| 107 | // Loop over all of the successors, removing BB's entry from any PHI |
| 108 | // nodes. |
| 109 | for (succ_iterator SI = succ_begin(BB), SE = succ_end(BB); SI != SE; ++SI) |
| 110 | (*SI)->removePredecessor(BB); |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 111 | |
Chris Lattner | c7d7e0c | 2010-05-08 22:15:59 +0000 | [diff] [blame] | 112 | // Insert a call to llvm.trap right before this. This turns the undefined |
| 113 | // behavior into a hard fail instead of falling through into random code. |
| 114 | if (UseLLVMTrap) { |
| 115 | Function *TrapFn = |
| 116 | Intrinsic::getDeclaration(BB->getParent()->getParent(), Intrinsic::trap); |
Devang Patel | 7878f37 | 2011-04-27 17:59:27 +0000 | [diff] [blame] | 117 | CallInst *CallTrap = CallInst::Create(TrapFn, "", I); |
| 118 | CallTrap->setDebugLoc(I->getDebugLoc()); |
Chris Lattner | c7d7e0c | 2010-05-08 22:15:59 +0000 | [diff] [blame] | 119 | } |
Owen Anderson | 1d0be15 | 2009-08-13 21:58:54 +0000 | [diff] [blame] | 120 | new UnreachableInst(I->getContext(), I); |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 121 | |
Chris Lattner | 1b12d88 | 2007-11-14 06:19:25 +0000 | [diff] [blame] | 122 | // All instructions after this are dead. |
| 123 | BasicBlock::iterator BBI = I, BBE = BB->end(); |
| 124 | while (BBI != BBE) { |
| 125 | if (!BBI->use_empty()) |
Owen Anderson | 9e9a0d5 | 2009-07-30 23:03:37 +0000 | [diff] [blame] | 126 | BBI->replaceAllUsesWith(UndefValue::get(BBI->getType())); |
Chris Lattner | 1b12d88 | 2007-11-14 06:19:25 +0000 | [diff] [blame] | 127 | BB->getInstList().erase(BBI++); |
| 128 | } |
| 129 | } |
| 130 | |
Jim Grosbach | 557a20a | 2012-09-06 00:59:08 +0000 | [diff] [blame] | 131 | /// changeToCall - Convert the specified invoke into a normal call. |
| 132 | static void changeToCall(InvokeInst *II) { |
Gabor Greif | c9f7500 | 2010-03-24 13:21:49 +0000 | [diff] [blame] | 133 | SmallVector<Value*, 8> Args(II->op_begin(), II->op_end() - 3); |
Jay Foad | a3efbb1 | 2011-07-15 08:37:34 +0000 | [diff] [blame] | 134 | CallInst *NewCall = CallInst::Create(II->getCalledValue(), Args, "", II); |
Duncan Sands | 73d4bb6 | 2007-11-22 22:24:59 +0000 | [diff] [blame] | 135 | NewCall->takeName(II); |
| 136 | NewCall->setCallingConv(II->getCallingConv()); |
Devang Patel | 0598866 | 2008-09-25 21:00:45 +0000 | [diff] [blame] | 137 | NewCall->setAttributes(II->getAttributes()); |
Devang Patel | d633089 | 2011-06-02 22:46:58 +0000 | [diff] [blame] | 138 | NewCall->setDebugLoc(II->getDebugLoc()); |
Duncan Sands | 73d4bb6 | 2007-11-22 22:24:59 +0000 | [diff] [blame] | 139 | II->replaceAllUsesWith(NewCall); |
| 140 | |
| 141 | // Follow the call by a branch to the normal destination. |
Gabor Greif | 051a950 | 2008-04-06 20:25:17 +0000 | [diff] [blame] | 142 | BranchInst::Create(II->getNormalDest(), II); |
Nuno Lopes | 4ccb89c | 2012-07-02 16:14:47 +0000 | [diff] [blame] | 143 | |
| 144 | // Update PHI nodes in the unwind destination |
| 145 | II->getUnwindDest()->removePredecessor(II->getParent()); |
Nuno Lopes | 7f71f5f | 2012-06-28 22:32:27 +0000 | [diff] [blame] | 146 | II->eraseFromParent(); |
Duncan Sands | 73d4bb6 | 2007-11-22 22:24:59 +0000 | [diff] [blame] | 147 | } |
| 148 | |
Jim Grosbach | 557a20a | 2012-09-06 00:59:08 +0000 | [diff] [blame] | 149 | static bool markAliveBlocks(BasicBlock *BB, |
Nick Lewycky | 2e0316f | 2009-11-23 03:34:29 +0000 | [diff] [blame] | 150 | SmallPtrSet<BasicBlock*, 128> &Reachable) { |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 151 | |
Chris Lattner | b42c8f7 | 2007-11-13 07:32:38 +0000 | [diff] [blame] | 152 | SmallVector<BasicBlock*, 128> Worklist; |
Chris Lattner | 1984a328 | 2007-04-05 01:27:02 +0000 | [diff] [blame] | 153 | Worklist.push_back(BB); |
Evgeniy Stepanov | 1794814 | 2012-12-17 14:28:00 +0000 | [diff] [blame] | 154 | Reachable.insert(BB); |
Chris Lattner | 1984a328 | 2007-04-05 01:27:02 +0000 | [diff] [blame] | 155 | bool Changed = false; |
Dan Gohman | 321a813 | 2010-01-05 16:27:25 +0000 | [diff] [blame] | 156 | do { |
| 157 | BB = Worklist.pop_back_val(); |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 158 | |
Chris Lattner | 1984a328 | 2007-04-05 01:27:02 +0000 | [diff] [blame] | 159 | // Do a quick scan of the basic block, turning any obviously unreachable |
| 160 | // instructions into LLVM unreachable insts. The instruction combining pass |
Duncan Sands | 73d4bb6 | 2007-11-22 22:24:59 +0000 | [diff] [blame] | 161 | // canonicalizes unreachable insts into stores to null or undef. |
Chris Lattner | 1b12d88 | 2007-11-14 06:19:25 +0000 | [diff] [blame] | 162 | for (BasicBlock::iterator BBI = BB->begin(), E = BB->end(); BBI != E;++BBI){ |
| 163 | if (CallInst *CI = dyn_cast<CallInst>(BBI)) { |
Duncan Sands | 2b0e899 | 2007-12-18 09:59:50 +0000 | [diff] [blame] | 164 | if (CI->doesNotReturn()) { |
Chris Lattner | 1b12d88 | 2007-11-14 06:19:25 +0000 | [diff] [blame] | 165 | // If we found a call to a no-return function, insert an unreachable |
| 166 | // instruction after it. Make sure there isn't *already* one there |
| 167 | // though. |
| 168 | ++BBI; |
| 169 | if (!isa<UnreachableInst>(BBI)) { |
Chris Lattner | c7d7e0c | 2010-05-08 22:15:59 +0000 | [diff] [blame] | 170 | // Don't insert a call to llvm.trap right before the unreachable. |
Jim Grosbach | 557a20a | 2012-09-06 00:59:08 +0000 | [diff] [blame] | 171 | changeToUnreachable(BBI, false); |
Chris Lattner | 1b12d88 | 2007-11-14 06:19:25 +0000 | [diff] [blame] | 172 | Changed = true; |
Chris Lattner | 1984a328 | 2007-04-05 01:27:02 +0000 | [diff] [blame] | 173 | } |
| 174 | break; |
Chris Lattner | fc5c1bb0 | 2004-10-18 03:00:50 +0000 | [diff] [blame] | 175 | } |
Chris Lattner | 1b12d88 | 2007-11-14 06:19:25 +0000 | [diff] [blame] | 176 | } |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 177 | |
Chris Lattner | c4775e4 | 2009-11-01 18:42:03 +0000 | [diff] [blame] | 178 | // Store to undef and store to null are undefined and used to signal that |
| 179 | // they should be changed to unreachable by passes that can't modify the |
| 180 | // CFG. |
Chris Lattner | 69074a5 | 2009-06-12 21:01:07 +0000 | [diff] [blame] | 181 | if (StoreInst *SI = dyn_cast<StoreInst>(BBI)) { |
Benjamin Kramer | c05cb8d | 2010-06-13 14:35:54 +0000 | [diff] [blame] | 182 | // Don't touch volatile stores. |
| 183 | if (SI->isVolatile()) continue; |
| 184 | |
Chris Lattner | 69074a5 | 2009-06-12 21:01:07 +0000 | [diff] [blame] | 185 | Value *Ptr = SI->getOperand(1); |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 186 | |
Dale Johannesen | fa196e3 | 2009-07-14 17:48:25 +0000 | [diff] [blame] | 187 | if (isa<UndefValue>(Ptr) || |
| 188 | (isa<ConstantPointerNull>(Ptr) && |
Chris Lattner | 8a67ac5 | 2009-08-30 20:06:40 +0000 | [diff] [blame] | 189 | SI->getPointerAddressSpace() == 0)) { |
Jim Grosbach | 557a20a | 2012-09-06 00:59:08 +0000 | [diff] [blame] | 190 | changeToUnreachable(SI, true); |
Dale Johannesen | fa196e3 | 2009-07-14 17:48:25 +0000 | [diff] [blame] | 191 | Changed = true; |
| 192 | break; |
| 193 | } |
Chris Lattner | 69074a5 | 2009-06-12 21:01:07 +0000 | [diff] [blame] | 194 | } |
Chris Lattner | 1b12d88 | 2007-11-14 06:19:25 +0000 | [diff] [blame] | 195 | } |
Chris Lattner | fc5c1bb0 | 2004-10-18 03:00:50 +0000 | [diff] [blame] | 196 | |
Duncan Sands | 73d4bb6 | 2007-11-22 22:24:59 +0000 | [diff] [blame] | 197 | // Turn invokes that call 'nounwind' functions into ordinary calls. |
Nuno Lopes | 3769fe1 | 2012-06-25 17:11:47 +0000 | [diff] [blame] | 198 | if (InvokeInst *II = dyn_cast<InvokeInst>(BB->getTerminator())) { |
| 199 | Value *Callee = II->getCalledValue(); |
| 200 | if (isa<ConstantPointerNull>(Callee) || isa<UndefValue>(Callee)) { |
Jim Grosbach | 557a20a | 2012-09-06 00:59:08 +0000 | [diff] [blame] | 201 | changeToUnreachable(II, true); |
Nuno Lopes | 3769fe1 | 2012-06-25 17:11:47 +0000 | [diff] [blame] | 202 | Changed = true; |
| 203 | } else if (II->doesNotThrow()) { |
Nuno Lopes | 7f71f5f | 2012-06-28 22:32:27 +0000 | [diff] [blame] | 204 | if (II->use_empty() && II->onlyReadsMemory()) { |
| 205 | // jump to the normal destination branch. |
| 206 | BranchInst::Create(II->getNormalDest(), II); |
Nuno Lopes | d49981a | 2012-07-16 22:49:40 +0000 | [diff] [blame] | 207 | II->getUnwindDest()->removePredecessor(II->getParent()); |
Nuno Lopes | 7f71f5f | 2012-06-28 22:32:27 +0000 | [diff] [blame] | 208 | II->eraseFromParent(); |
| 209 | } else |
Jim Grosbach | 557a20a | 2012-09-06 00:59:08 +0000 | [diff] [blame] | 210 | changeToCall(II); |
Duncan Sands | 73d4bb6 | 2007-11-22 22:24:59 +0000 | [diff] [blame] | 211 | Changed = true; |
| 212 | } |
Nuno Lopes | 3769fe1 | 2012-06-25 17:11:47 +0000 | [diff] [blame] | 213 | } |
Duncan Sands | 73d4bb6 | 2007-11-22 22:24:59 +0000 | [diff] [blame] | 214 | |
Frits van Bommel | 5649ba7 | 2011-05-22 16:24:18 +0000 | [diff] [blame] | 215 | Changed |= ConstantFoldTerminator(BB, true); |
Chris Lattner | 1984a328 | 2007-04-05 01:27:02 +0000 | [diff] [blame] | 216 | for (succ_iterator SI = succ_begin(BB), SE = succ_end(BB); SI != SE; ++SI) |
Evgeniy Stepanov | 1794814 | 2012-12-17 14:28:00 +0000 | [diff] [blame] | 217 | if (Reachable.insert(*SI)) |
| 218 | Worklist.push_back(*SI); |
Dan Gohman | 321a813 | 2010-01-05 16:27:25 +0000 | [diff] [blame] | 219 | } while (!Worklist.empty()); |
Chris Lattner | 573527b | 2002-05-21 20:49:37 +0000 | [diff] [blame] | 220 | return Changed; |
| 221 | } |
| 222 | |
Jim Grosbach | 557a20a | 2012-09-06 00:59:08 +0000 | [diff] [blame] | 223 | /// removeUnreachableBlocksFromFn - Remove blocks that are not reachable, even |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 224 | /// if they are in a dead cycle. Return true if a change was made, false |
Devang Patel | 2379089 | 2008-09-08 22:14:17 +0000 | [diff] [blame] | 225 | /// otherwise. |
Jim Grosbach | 557a20a | 2012-09-06 00:59:08 +0000 | [diff] [blame] | 226 | static bool removeUnreachableBlocksFromFn(Function &F) { |
Chris Lattner | b42c8f7 | 2007-11-13 07:32:38 +0000 | [diff] [blame] | 227 | SmallPtrSet<BasicBlock*, 128> Reachable; |
Jim Grosbach | 557a20a | 2012-09-06 00:59:08 +0000 | [diff] [blame] | 228 | bool Changed = markAliveBlocks(F.begin(), Reachable); |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 229 | |
Chris Lattner | 573527b | 2002-05-21 20:49:37 +0000 | [diff] [blame] | 230 | // If there are unreachable blocks in the CFG... |
Bill Wendling | 381802f | 2008-01-26 06:51:24 +0000 | [diff] [blame] | 231 | if (Reachable.size() == F.size()) |
Chris Lattner | b42c8f7 | 2007-11-13 07:32:38 +0000 | [diff] [blame] | 232 | return Changed; |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 233 | |
Chris Lattner | b42c8f7 | 2007-11-13 07:32:38 +0000 | [diff] [blame] | 234 | assert(Reachable.size() < F.size()); |
| 235 | NumSimpl += F.size()-Reachable.size(); |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 236 | |
Chris Lattner | b42c8f7 | 2007-11-13 07:32:38 +0000 | [diff] [blame] | 237 | // Loop over all of the basic blocks that are not reachable, dropping all of |
| 238 | // their internal references... |
Chris Lattner | 13daadb | 2008-02-14 07:39:01 +0000 | [diff] [blame] | 239 | for (Function::iterator BB = ++F.begin(), E = F.end(); BB != E; ++BB) { |
| 240 | if (Reachable.count(BB)) |
| 241 | continue; |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 242 | |
Chris Lattner | 13daadb | 2008-02-14 07:39:01 +0000 | [diff] [blame] | 243 | for (succ_iterator SI = succ_begin(BB), SE = succ_end(BB); SI != SE; ++SI) |
| 244 | if (Reachable.count(*SI)) |
| 245 | (*SI)->removePredecessor(BB); |
| 246 | BB->dropAllReferences(); |
| 247 | } |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 248 | |
Chris Lattner | b42c8f7 | 2007-11-13 07:32:38 +0000 | [diff] [blame] | 249 | for (Function::iterator I = ++F.begin(); I != F.end();) |
| 250 | if (!Reachable.count(I)) |
| 251 | I = F.getBasicBlockList().erase(I); |
| 252 | else |
| 253 | ++I; |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 254 | |
Chris Lattner | b42c8f7 | 2007-11-13 07:32:38 +0000 | [diff] [blame] | 255 | return true; |
| 256 | } |
Chris Lattner | 573527b | 2002-05-21 20:49:37 +0000 | [diff] [blame] | 257 | |
Jim Grosbach | 557a20a | 2012-09-06 00:59:08 +0000 | [diff] [blame] | 258 | /// mergeEmptyReturnBlocks - If we have more than one empty (other than phi |
Chris Lattner | 1a0e708 | 2009-12-22 06:07:30 +0000 | [diff] [blame] | 259 | /// node) return blocks, merge them together to promote recursive block merging. |
Jim Grosbach | 557a20a | 2012-09-06 00:59:08 +0000 | [diff] [blame] | 260 | static bool mergeEmptyReturnBlocks(Function &F) { |
Chris Lattner | 1a0e708 | 2009-12-22 06:07:30 +0000 | [diff] [blame] | 261 | bool Changed = false; |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 262 | |
Chris Lattner | 1a0e708 | 2009-12-22 06:07:30 +0000 | [diff] [blame] | 263 | BasicBlock *RetBlock = 0; |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 264 | |
Chris Lattner | 1a0e708 | 2009-12-22 06:07:30 +0000 | [diff] [blame] | 265 | // Scan all the blocks in the function, looking for empty return blocks. |
| 266 | for (Function::iterator BBI = F.begin(), E = F.end(); BBI != E; ) { |
| 267 | BasicBlock &BB = *BBI++; |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 268 | |
Chris Lattner | 1a0e708 | 2009-12-22 06:07:30 +0000 | [diff] [blame] | 269 | // Only look at return blocks. |
| 270 | ReturnInst *Ret = dyn_cast<ReturnInst>(BB.getTerminator()); |
| 271 | if (Ret == 0) continue; |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 272 | |
Chris Lattner | 1a0e708 | 2009-12-22 06:07:30 +0000 | [diff] [blame] | 273 | // Only look at the block if it is empty or the only other thing in it is a |
| 274 | // single PHI node that is the operand to the return. |
| 275 | if (Ret != &BB.front()) { |
| 276 | // Check for something else in the block. |
| 277 | BasicBlock::iterator I = Ret; |
| 278 | --I; |
Bill Wendling | ead138b | 2010-03-14 10:40:55 +0000 | [diff] [blame] | 279 | // Skip over debug info. |
| 280 | while (isa<DbgInfoIntrinsic>(I) && I != BB.begin()) |
| 281 | --I; |
| 282 | if (!isa<DbgInfoIntrinsic>(I) && |
| 283 | (!isa<PHINode>(I) || I != BB.begin() || |
| 284 | Ret->getNumOperands() == 0 || |
| 285 | Ret->getOperand(0) != I)) |
Chris Lattner | 1a0e708 | 2009-12-22 06:07:30 +0000 | [diff] [blame] | 286 | continue; |
| 287 | } |
Bill Wendling | ead138b | 2010-03-14 10:40:55 +0000 | [diff] [blame] | 288 | |
Chris Lattner | 1a0e708 | 2009-12-22 06:07:30 +0000 | [diff] [blame] | 289 | // If this is the first returning block, remember it and keep going. |
| 290 | if (RetBlock == 0) { |
| 291 | RetBlock = &BB; |
| 292 | continue; |
| 293 | } |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 294 | |
Chris Lattner | 1a0e708 | 2009-12-22 06:07:30 +0000 | [diff] [blame] | 295 | // Otherwise, we found a duplicate return block. Merge the two. |
| 296 | Changed = true; |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 297 | |
Chris Lattner | 1a0e708 | 2009-12-22 06:07:30 +0000 | [diff] [blame] | 298 | // Case when there is no input to the return or when the returned values |
| 299 | // agree is trivial. Note that they can't agree if there are phis in the |
| 300 | // blocks. |
| 301 | if (Ret->getNumOperands() == 0 || |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 302 | Ret->getOperand(0) == |
Chris Lattner | 1a0e708 | 2009-12-22 06:07:30 +0000 | [diff] [blame] | 303 | cast<ReturnInst>(RetBlock->getTerminator())->getOperand(0)) { |
| 304 | BB.replaceAllUsesWith(RetBlock); |
| 305 | BB.eraseFromParent(); |
| 306 | continue; |
| 307 | } |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 308 | |
Chris Lattner | 1a0e708 | 2009-12-22 06:07:30 +0000 | [diff] [blame] | 309 | // If the canonical return block has no PHI node, create one now. |
| 310 | PHINode *RetBlockPHI = dyn_cast<PHINode>(RetBlock->begin()); |
| 311 | if (RetBlockPHI == 0) { |
Devang Patel | ca70495 | 2010-03-15 19:05:46 +0000 | [diff] [blame] | 312 | Value *InVal = cast<ReturnInst>(RetBlock->getTerminator())->getOperand(0); |
Jay Foad | d8b4fb4 | 2011-03-30 11:19:20 +0000 | [diff] [blame] | 313 | pred_iterator PB = pred_begin(RetBlock), PE = pred_end(RetBlock); |
Jay Foad | 3ecfc86 | 2011-03-30 11:28:46 +0000 | [diff] [blame] | 314 | RetBlockPHI = PHINode::Create(Ret->getOperand(0)->getType(), |
| 315 | std::distance(PB, PE), "merge", |
Chris Lattner | 1a0e708 | 2009-12-22 06:07:30 +0000 | [diff] [blame] | 316 | &RetBlock->front()); |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 317 | |
Jay Foad | d8b4fb4 | 2011-03-30 11:19:20 +0000 | [diff] [blame] | 318 | for (pred_iterator PI = PB; PI != PE; ++PI) |
Chris Lattner | 1a0e708 | 2009-12-22 06:07:30 +0000 | [diff] [blame] | 319 | RetBlockPHI->addIncoming(InVal, *PI); |
| 320 | RetBlock->getTerminator()->setOperand(0, RetBlockPHI); |
| 321 | } |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 322 | |
Chris Lattner | 1a0e708 | 2009-12-22 06:07:30 +0000 | [diff] [blame] | 323 | // Turn BB into a block that just unconditionally branches to the return |
| 324 | // block. This handles the case when the two return blocks have a common |
| 325 | // predecessor but that return different things. |
| 326 | RetBlockPHI->addIncoming(Ret->getOperand(0), &BB); |
| 327 | BB.getTerminator()->eraseFromParent(); |
| 328 | BranchInst::Create(RetBlock, &BB); |
| 329 | } |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 330 | |
Chris Lattner | 1a0e708 | 2009-12-22 06:07:30 +0000 | [diff] [blame] | 331 | return Changed; |
| 332 | } |
| 333 | |
Jim Grosbach | 557a20a | 2012-09-06 00:59:08 +0000 | [diff] [blame] | 334 | /// iterativelySimplifyCFG - Call SimplifyCFG on all the blocks in the function, |
Chris Lattner | b42c8f7 | 2007-11-13 07:32:38 +0000 | [diff] [blame] | 335 | /// iterating until no more changes are made. |
Chandler Carruth | 5f46c3c | 2013-01-07 03:53:25 +0000 | [diff] [blame] | 336 | static bool iterativelySimplifyCFG(Function &F, const TargetTransformInfo &TTI, |
Tom Stellard | 57e6b2d | 2013-07-27 00:01:07 +0000 | [diff] [blame] | 337 | const DataLayout *TD, AliasAnalysis *AA) { |
Chris Lattner | b42c8f7 | 2007-11-13 07:32:38 +0000 | [diff] [blame] | 338 | bool Changed = false; |
Chris Lattner | 573527b | 2002-05-21 20:49:37 +0000 | [diff] [blame] | 339 | bool LocalChange = true; |
| 340 | while (LocalChange) { |
| 341 | LocalChange = false; |
Nadav Rotem | a94d6e8 | 2012-07-24 10:51:42 +0000 | [diff] [blame] | 342 | |
Dan Gohman | e2c6d13 | 2010-08-14 00:29:42 +0000 | [diff] [blame] | 343 | // Loop over all of the basic blocks and remove them if they are unneeded... |
Chris Lattner | 573527b | 2002-05-21 20:49:37 +0000 | [diff] [blame] | 344 | // |
Dan Gohman | e2c6d13 | 2010-08-14 00:29:42 +0000 | [diff] [blame] | 345 | for (Function::iterator BBIt = F.begin(); BBIt != F.end(); ) { |
Tom Stellard | 57e6b2d | 2013-07-27 00:01:07 +0000 | [diff] [blame] | 346 | if (SimplifyCFG(BBIt++, TTI, TD, AA)) { |
Chris Lattner | 573527b | 2002-05-21 20:49:37 +0000 | [diff] [blame] | 347 | LocalChange = true; |
| 348 | ++NumSimpl; |
Chris Lattner | 573527b | 2002-05-21 20:49:37 +0000 | [diff] [blame] | 349 | } |
| 350 | } |
| 351 | Changed |= LocalChange; |
| 352 | } |
Chris Lattner | 573527b | 2002-05-21 20:49:37 +0000 | [diff] [blame] | 353 | return Changed; |
| 354 | } |
Chris Lattner | b42c8f7 | 2007-11-13 07:32:38 +0000 | [diff] [blame] | 355 | |
| 356 | // It is possible that we may require multiple passes over the code to fully |
| 357 | // simplify the CFG. |
| 358 | // |
| 359 | bool CFGSimplifyPass::runOnFunction(Function &F) { |
Tom Stellard | 57e6b2d | 2013-07-27 00:01:07 +0000 | [diff] [blame] | 360 | if (IsTargetAware) |
| 361 | AA = &getAnalysis<AliasAnalysis>(); |
| 362 | else |
| 363 | AA = NULL; |
Chandler Carruth | 5f46c3c | 2013-01-07 03:53:25 +0000 | [diff] [blame] | 364 | const TargetTransformInfo &TTI = getAnalysis<TargetTransformInfo>(); |
Micah Villmow | 3574eca | 2012-10-08 16:38:25 +0000 | [diff] [blame] | 365 | const DataLayout *TD = getAnalysisIfAvailable<DataLayout>(); |
Jim Grosbach | 557a20a | 2012-09-06 00:59:08 +0000 | [diff] [blame] | 366 | bool EverChanged = removeUnreachableBlocksFromFn(F); |
| 367 | EverChanged |= mergeEmptyReturnBlocks(F); |
Tom Stellard | 57e6b2d | 2013-07-27 00:01:07 +0000 | [diff] [blame] | 368 | EverChanged |= iterativelySimplifyCFG(F, TTI, TD, AA); |
Jakob Stoklund Olesen | 58e9ee8 | 2010-02-05 22:03:18 +0000 | [diff] [blame] | 369 | |
Chris Lattner | b42c8f7 | 2007-11-13 07:32:38 +0000 | [diff] [blame] | 370 | // If neither pass changed anything, we're done. |
| 371 | if (!EverChanged) return false; |
| 372 | |
Jim Grosbach | 557a20a | 2012-09-06 00:59:08 +0000 | [diff] [blame] | 373 | // iterativelySimplifyCFG can (rarely) make some loops dead. If this happens, |
| 374 | // removeUnreachableBlocksFromFn is needed to nuke them, which means we should |
Chris Lattner | b42c8f7 | 2007-11-13 07:32:38 +0000 | [diff] [blame] | 375 | // iterate between the two optimizations. We structure the code like this to |
Jim Grosbach | 557a20a | 2012-09-06 00:59:08 +0000 | [diff] [blame] | 376 | // avoid reruning iterativelySimplifyCFG if the second pass of |
| 377 | // removeUnreachableBlocksFromFn doesn't do anything. |
| 378 | if (!removeUnreachableBlocksFromFn(F)) |
Chris Lattner | b42c8f7 | 2007-11-13 07:32:38 +0000 | [diff] [blame] | 379 | return true; |
Jakob Stoklund Olesen | 58e9ee8 | 2010-02-05 22:03:18 +0000 | [diff] [blame] | 380 | |
Chris Lattner | b42c8f7 | 2007-11-13 07:32:38 +0000 | [diff] [blame] | 381 | do { |
Tom Stellard | 57e6b2d | 2013-07-27 00:01:07 +0000 | [diff] [blame] | 382 | EverChanged = iterativelySimplifyCFG(F, TTI, TD, AA); |
Jim Grosbach | 557a20a | 2012-09-06 00:59:08 +0000 | [diff] [blame] | 383 | EverChanged |= removeUnreachableBlocksFromFn(F); |
Chris Lattner | b42c8f7 | 2007-11-13 07:32:38 +0000 | [diff] [blame] | 384 | } while (EverChanged); |
Jakob Stoklund Olesen | 58e9ee8 | 2010-02-05 22:03:18 +0000 | [diff] [blame] | 385 | |
Chris Lattner | b42c8f7 | 2007-11-13 07:32:38 +0000 | [diff] [blame] | 386 | return true; |
| 387 | } |