Chris Lattner | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 1 | //===-- LICM.cpp - Loop Invariant Code Motion 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 | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 9 | // |
Chris Lattner | 92094b4 | 2003-12-09 17:18:00 +0000 | [diff] [blame] | 10 | // This pass performs loop invariant code motion, attempting to remove as much |
| 11 | // code from the body of a loop as possible. It does this by either hoisting |
| 12 | // code into the preheader block, or by sinking code to the exit blocks if it is |
| 13 | // safe. This pass also promotes must-aliased memory locations in the loop to |
Chris Lattner | e4365b2 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 14 | // live in registers, thus hoisting and sinking "invariant" loads and stores. |
Chris Lattner | 92094b4 | 2003-12-09 17:18:00 +0000 | [diff] [blame] | 15 | // |
| 16 | // This pass uses alias analysis for two purposes: |
Chris Lattner | 2e6e741 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 17 | // |
Chris Lattner | 2741c97 | 2004-05-23 21:20:19 +0000 | [diff] [blame] | 18 | // 1. Moving loop invariant loads and calls out of loops. If we can determine |
| 19 | // that a load or call inside of a loop never aliases anything stored to, |
| 20 | // we can hoist it or sink it like any other instruction. |
Chris Lattner | 2e6e741 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 21 | // 2. Scalar Promotion of Memory - If there is a store instruction inside of |
| 22 | // the loop, we try to move the store to happen AFTER the loop instead of |
| 23 | // inside of the loop. This can only happen if a few conditions are true: |
| 24 | // A. The pointer stored through is loop invariant |
| 25 | // B. There are no stores or loads in the loop which _may_ alias the |
| 26 | // pointer. There are no calls in the loop which mod/ref the pointer. |
| 27 | // If these conditions are true, we can promote the loads and stores in the |
| 28 | // loop of the pointer to use a temporary alloca'd variable. We then use |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 29 | // the SSAUpdater to construct the appropriate SSA form for the value. |
Chris Lattner | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 30 | // |
Chris Lattner | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 31 | //===----------------------------------------------------------------------===// |
| 32 | |
Chris Lattner | 1f309c1 | 2005-03-23 21:00:12 +0000 | [diff] [blame] | 33 | #define DEBUG_TYPE "licm" |
Chris Lattner | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 34 | #include "llvm/Transforms/Scalar.h" |
Chris Lattner | 79066fa | 2007-01-30 23:46:24 +0000 | [diff] [blame] | 35 | #include "llvm/Constants.h" |
Chris Lattner | 2741c97 | 2004-05-23 21:20:19 +0000 | [diff] [blame] | 36 | #include "llvm/DerivedTypes.h" |
Torok Edwin | 9289ae8 | 2009-10-11 19:15:54 +0000 | [diff] [blame] | 37 | #include "llvm/IntrinsicInst.h" |
Chris Lattner | 2741c97 | 2004-05-23 21:20:19 +0000 | [diff] [blame] | 38 | #include "llvm/Instructions.h" |
Dan Gohman | a8702ea | 2010-10-18 20:44:50 +0000 | [diff] [blame] | 39 | #include "llvm/LLVMContext.h" |
Chris Lattner | f5e84aa | 2002-08-22 21:39:55 +0000 | [diff] [blame] | 40 | #include "llvm/Analysis/AliasAnalysis.h" |
Chris Lattner | 0252e49 | 2003-03-03 23:32:45 +0000 | [diff] [blame] | 41 | #include "llvm/Analysis/AliasSetTracker.h" |
Chris Lattner | 2ac6e23 | 2010-08-31 23:00:16 +0000 | [diff] [blame] | 42 | #include "llvm/Analysis/ConstantFolding.h" |
| 43 | #include "llvm/Analysis/LoopInfo.h" |
| 44 | #include "llvm/Analysis/LoopPass.h" |
Chris Lattner | 952eaee | 2002-09-29 21:46:09 +0000 | [diff] [blame] | 45 | #include "llvm/Analysis/Dominators.h" |
Chris Lattner | 0de5cad | 2010-08-29 18:22:25 +0000 | [diff] [blame] | 46 | #include "llvm/Transforms/Utils/Local.h" |
Chris Lattner | a6a36f5 | 2010-08-29 04:55:06 +0000 | [diff] [blame] | 47 | #include "llvm/Transforms/Utils/SSAUpdater.h" |
Reid Spencer | 9133fe2 | 2007-02-05 23:32:05 +0000 | [diff] [blame] | 48 | #include "llvm/Support/CFG.h" |
Reid Spencer | 551ccae | 2004-09-01 22:55:40 +0000 | [diff] [blame] | 49 | #include "llvm/Support/CommandLine.h" |
Daniel Dunbar | ce63ffb | 2009-07-25 00:23:56 +0000 | [diff] [blame] | 50 | #include "llvm/Support/raw_ostream.h" |
Reid Spencer | 551ccae | 2004-09-01 22:55:40 +0000 | [diff] [blame] | 51 | #include "llvm/Support/Debug.h" |
| 52 | #include "llvm/ADT/Statistic.h" |
Chris Lattner | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 53 | #include <algorithm> |
Chris Lattner | 92094b4 | 2003-12-09 17:18:00 +0000 | [diff] [blame] | 54 | using namespace llvm; |
Brian Gaeke | d0fde30 | 2003-11-11 22:41:34 +0000 | [diff] [blame] | 55 | |
Chris Lattner | 0e5f499 | 2006-12-19 21:40:18 +0000 | [diff] [blame] | 56 | STATISTIC(NumSunk , "Number of instructions sunk out of loop"); |
| 57 | STATISTIC(NumHoisted , "Number of instructions hoisted out of loop"); |
| 58 | STATISTIC(NumMovedLoads, "Number of load insts hoisted or sunk"); |
| 59 | STATISTIC(NumMovedCalls, "Number of call insts hoisted or sunk"); |
| 60 | STATISTIC(NumPromoted , "Number of memory locations promoted to registers"); |
| 61 | |
Dan Gohman | 844731a | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 62 | static cl::opt<bool> |
| 63 | DisablePromotion("disable-licm-promotion", cl::Hidden, |
| 64 | cl::desc("Disable memory promotion in LICM pass")); |
Chris Lattner | 2e6e741 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 65 | |
Dan Gohman | 844731a | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 66 | namespace { |
Chris Lattner | 3e8b663 | 2009-09-02 06:11:42 +0000 | [diff] [blame] | 67 | struct LICM : public LoopPass { |
Nick Lewycky | ecd94c8 | 2007-05-06 13:37:16 +0000 | [diff] [blame] | 68 | static char ID; // Pass identification, replacement for typeid |
Owen Anderson | 081c34b | 2010-10-19 17:21:58 +0000 | [diff] [blame] | 69 | LICM() : LoopPass(ID) { |
| 70 | initializeLICMPass(*PassRegistry::getPassRegistry()); |
| 71 | } |
Devang Patel | 794fd75 | 2007-05-01 21:15:47 +0000 | [diff] [blame] | 72 | |
Devang Patel | 54959d6 | 2007-03-07 04:41:30 +0000 | [diff] [blame] | 73 | virtual bool runOnLoop(Loop *L, LPPassManager &LPM); |
Chris Lattner | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 74 | |
Chris Lattner | 9417059 | 2002-09-26 16:52:07 +0000 | [diff] [blame] | 75 | /// This transformation requires natural loop information & requires that |
| 76 | /// loop preheaders be inserted into the CFG... |
| 77 | /// |
Chris Lattner | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 78 | virtual void getAnalysisUsage(AnalysisUsage &AU) const { |
Chris Lattner | cb2610e | 2002-10-21 20:00:28 +0000 | [diff] [blame] | 79 | AU.setPreservesCFG(); |
Owen Anderson | 3a2b58f | 2007-04-24 06:40:39 +0000 | [diff] [blame] | 80 | AU.addRequired<DominatorTree>(); |
Dan Gohman | 1e381fc | 2010-07-16 17:58:45 +0000 | [diff] [blame] | 81 | AU.addRequired<LoopInfo>(); |
| 82 | AU.addRequiredID(LoopSimplifyID); |
Chris Lattner | f5e84aa | 2002-08-22 21:39:55 +0000 | [diff] [blame] | 83 | AU.addRequired<AliasAnalysis>(); |
Chris Lattner | e418ac8 | 2010-08-29 07:02:56 +0000 | [diff] [blame] | 84 | AU.addPreserved<AliasAnalysis>(); |
Dan Gohman | a9b61e7 | 2010-11-17 20:50:07 +0000 | [diff] [blame] | 85 | AU.addPreserved("scalar-evolution"); |
Dan Gohman | 5c89b52 | 2009-09-08 15:45:00 +0000 | [diff] [blame] | 86 | AU.addPreservedID(LoopSimplifyID); |
Chris Lattner | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 87 | } |
| 88 | |
Dan Gohman | 747603e | 2007-04-17 18:21:36 +0000 | [diff] [blame] | 89 | bool doFinalization() { |
Chris Lattner | 4282e32 | 2010-08-29 17:46:00 +0000 | [diff] [blame] | 90 | assert(LoopToAliasSetMap.empty() && "Didn't free loop alias sets"); |
Devang Patel | 54959d6 | 2007-03-07 04:41:30 +0000 | [diff] [blame] | 91 | return false; |
| 92 | } |
| 93 | |
Chris Lattner | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 94 | private: |
Chris Lattner | 2e6e741 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 95 | AliasAnalysis *AA; // Current AliasAnalysis information |
Chris Lattner | 92094b4 | 2003-12-09 17:18:00 +0000 | [diff] [blame] | 96 | LoopInfo *LI; // Current LoopInfo |
Chris Lattner | 44e2bd3 | 2010-08-29 06:49:44 +0000 | [diff] [blame] | 97 | DominatorTree *DT; // Dominator Tree for the current Loop. |
Chris Lattner | 92094b4 | 2003-12-09 17:18:00 +0000 | [diff] [blame] | 98 | |
Chris Lattner | 4282e32 | 2010-08-29 17:46:00 +0000 | [diff] [blame] | 99 | // State that is updated as we process loops. |
Chris Lattner | 2e6e741 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 100 | bool Changed; // Set to true when we change anything. |
| 101 | BasicBlock *Preheader; // The preheader block of the current loop... |
| 102 | Loop *CurLoop; // The current loop we are working on... |
Chris Lattner | 0252e49 | 2003-03-03 23:32:45 +0000 | [diff] [blame] | 103 | AliasSetTracker *CurAST; // AliasSet information for the current loop... |
Chris Lattner | 4282e32 | 2010-08-29 17:46:00 +0000 | [diff] [blame] | 104 | DenseMap<Loop*, AliasSetTracker*> LoopToAliasSetMap; |
Chris Lattner | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 105 | |
Devang Patel | 91d22c8 | 2007-07-31 08:01:41 +0000 | [diff] [blame] | 106 | /// cloneBasicBlockAnalysis - Simple Analysis hook. Clone alias set info. |
| 107 | void cloneBasicBlockAnalysis(BasicBlock *From, BasicBlock *To, Loop *L); |
| 108 | |
| 109 | /// deleteAnalysisValue - Simple Analysis hook. Delete value V from alias |
| 110 | /// set. |
| 111 | void deleteAnalysisValue(Value *V, Loop *L); |
| 112 | |
Chris Lattner | e4365b2 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 113 | /// SinkRegion - Walk the specified region of the CFG (defined by all blocks |
| 114 | /// dominated by the specified block, and that are in the current loop) in |
Owen Anderson | 3a2b58f | 2007-04-24 06:40:39 +0000 | [diff] [blame] | 115 | /// reverse depth first order w.r.t the DominatorTree. This allows us to |
Chris Lattner | e4365b2 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 116 | /// visit uses before definitions, allowing us to sink a loop body in one |
| 117 | /// pass without iteration. |
| 118 | /// |
Devang Patel | 2604242 | 2007-06-04 00:32:22 +0000 | [diff] [blame] | 119 | void SinkRegion(DomTreeNode *N); |
Chris Lattner | e4365b2 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 120 | |
Chris Lattner | 952eaee | 2002-09-29 21:46:09 +0000 | [diff] [blame] | 121 | /// HoistRegion - Walk the specified region of the CFG (defined by all |
| 122 | /// blocks dominated by the specified block, and that are in the current |
Owen Anderson | 3a2b58f | 2007-04-24 06:40:39 +0000 | [diff] [blame] | 123 | /// loop) in depth first order w.r.t the DominatorTree. This allows us to |
Misha Brukman | cf00c4a | 2003-10-10 17:57:28 +0000 | [diff] [blame] | 124 | /// visit definitions before uses, allowing us to hoist a loop body in one |
Chris Lattner | 952eaee | 2002-09-29 21:46:09 +0000 | [diff] [blame] | 125 | /// pass without iteration. |
| 126 | /// |
Devang Patel | 2604242 | 2007-06-04 00:32:22 +0000 | [diff] [blame] | 127 | void HoistRegion(DomTreeNode *N); |
Chris Lattner | 952eaee | 2002-09-29 21:46:09 +0000 | [diff] [blame] | 128 | |
Chris Lattner | b461373 | 2002-09-29 22:26:07 +0000 | [diff] [blame] | 129 | /// inSubLoop - Little predicate that returns true if the specified basic |
| 130 | /// block is in a subloop of the current one, not the current one itself. |
Chris Lattner | 9417059 | 2002-09-26 16:52:07 +0000 | [diff] [blame] | 131 | /// |
Chris Lattner | b461373 | 2002-09-29 22:26:07 +0000 | [diff] [blame] | 132 | bool inSubLoop(BasicBlock *BB) { |
| 133 | assert(CurLoop->contains(BB) && "Only valid if BB is IN the loop"); |
Chris Lattner | 81a866d | 2011-01-02 18:53:08 +0000 | [diff] [blame] | 134 | return LI->getLoopFor(BB) != CurLoop; |
Chris Lattner | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 135 | } |
| 136 | |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 137 | /// sink - When an instruction is found to only be used outside of the loop, |
| 138 | /// this function moves it to the exit blocks and patches up SSA form as |
| 139 | /// needed. |
| 140 | /// |
| 141 | void sink(Instruction &I); |
| 142 | |
Chris Lattner | 9417059 | 2002-09-26 16:52:07 +0000 | [diff] [blame] | 143 | /// hoist - When an instruction is found to only use loop invariant operands |
| 144 | /// that is safe to hoist, this instruction is called to do the dirty work. |
| 145 | /// |
Chris Lattner | 7e70829 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 146 | void hoist(Instruction &I); |
Chris Lattner | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 147 | |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 148 | /// isSafeToExecuteUnconditionally - Only sink or hoist an instruction if it |
| 149 | /// is not a trapping instruction or if it is a trapping instruction and is |
| 150 | /// guaranteed to execute. |
Tanya Lattner | 9966c03 | 2003-08-05 18:45:46 +0000 | [diff] [blame] | 151 | /// |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 152 | bool isSafeToExecuteUnconditionally(Instruction &I); |
Tanya Lattner | 9966c03 | 2003-08-05 18:45:46 +0000 | [diff] [blame] | 153 | |
Eli Friedman | 73bfa4a | 2011-07-20 21:37:47 +0000 | [diff] [blame] | 154 | /// isGuaranteedToExecute - Check that the instruction is guaranteed to |
| 155 | /// execute. |
| 156 | /// |
| 157 | bool isGuaranteedToExecute(Instruction &I); |
| 158 | |
Chris Lattner | 9417059 | 2002-09-26 16:52:07 +0000 | [diff] [blame] | 159 | /// pointerInvalidatedByLoop - Return true if the body of this loop may |
| 160 | /// store into the memory location pointed to by V. |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 161 | /// |
Dan Gohman | 3da848b | 2010-10-19 22:54:46 +0000 | [diff] [blame] | 162 | bool pointerInvalidatedByLoop(Value *V, uint64_t Size, |
Dan Gohman | a8702ea | 2010-10-18 20:44:50 +0000 | [diff] [blame] | 163 | const MDNode *TBAAInfo) { |
Chris Lattner | 0252e49 | 2003-03-03 23:32:45 +0000 | [diff] [blame] | 164 | // Check to see if any of the basic blocks in CurLoop invalidate *V. |
Dan Gohman | a8702ea | 2010-10-18 20:44:50 +0000 | [diff] [blame] | 165 | return CurAST->getAliasSetForPointer(V, Size, TBAAInfo).isMod(); |
Chris Lattner | 2e6e741 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 166 | } |
Chris Lattner | f5e84aa | 2002-08-22 21:39:55 +0000 | [diff] [blame] | 167 | |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 168 | bool canSinkOrHoistInst(Instruction &I); |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 169 | bool isNotUsedInLoop(Instruction &I); |
Chris Lattner | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 170 | |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 171 | void PromoteAliasSet(AliasSet &AS); |
Chris Lattner | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 172 | }; |
| 173 | } |
| 174 | |
Dan Gohman | 844731a | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 175 | char LICM::ID = 0; |
Owen Anderson | 2ab36d3 | 2010-10-12 19:48:12 +0000 | [diff] [blame] | 176 | INITIALIZE_PASS_BEGIN(LICM, "licm", "Loop Invariant Code Motion", false, false) |
| 177 | INITIALIZE_PASS_DEPENDENCY(DominatorTree) |
| 178 | INITIALIZE_PASS_DEPENDENCY(LoopInfo) |
Owen Anderson | 2ab36d3 | 2010-10-12 19:48:12 +0000 | [diff] [blame] | 179 | INITIALIZE_PASS_DEPENDENCY(LoopSimplify) |
| 180 | INITIALIZE_AG_DEPENDENCY(AliasAnalysis) |
| 181 | INITIALIZE_PASS_END(LICM, "licm", "Loop Invariant Code Motion", false, false) |
Dan Gohman | 844731a | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 182 | |
Daniel Dunbar | 394f044 | 2008-10-22 23:32:42 +0000 | [diff] [blame] | 183 | Pass *llvm::createLICMPass() { return new LICM(); } |
Chris Lattner | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 184 | |
Devang Patel | 8d246f0 | 2007-07-31 16:52:25 +0000 | [diff] [blame] | 185 | /// Hoist expressions out of the specified loop. Note, alias info for inner |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 186 | /// loop is not preserved so it is not a good idea to run LICM multiple |
Devang Patel | 8d246f0 | 2007-07-31 16:52:25 +0000 | [diff] [blame] | 187 | /// times on one loop. |
Chris Lattner | 9417059 | 2002-09-26 16:52:07 +0000 | [diff] [blame] | 188 | /// |
Devang Patel | 54959d6 | 2007-03-07 04:41:30 +0000 | [diff] [blame] | 189 | bool LICM::runOnLoop(Loop *L, LPPassManager &LPM) { |
Chris Lattner | 2e6e741 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 190 | Changed = false; |
Chris Lattner | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 191 | |
Chris Lattner | 2e6e741 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 192 | // Get our Loop and Alias Analysis information... |
| 193 | LI = &getAnalysis<LoopInfo>(); |
Chris Lattner | f5e84aa | 2002-08-22 21:39:55 +0000 | [diff] [blame] | 194 | AA = &getAnalysis<AliasAnalysis>(); |
Owen Anderson | 3a2b58f | 2007-04-24 06:40:39 +0000 | [diff] [blame] | 195 | DT = &getAnalysis<DominatorTree>(); |
Chris Lattner | f5e84aa | 2002-08-22 21:39:55 +0000 | [diff] [blame] | 196 | |
Devang Patel | 54959d6 | 2007-03-07 04:41:30 +0000 | [diff] [blame] | 197 | CurAST = new AliasSetTracker(*AA); |
Chris Lattner | 4282e32 | 2010-08-29 17:46:00 +0000 | [diff] [blame] | 198 | // Collect Alias info from subloops. |
Devang Patel | 54959d6 | 2007-03-07 04:41:30 +0000 | [diff] [blame] | 199 | for (Loop::iterator LoopItr = L->begin(), LoopItrE = L->end(); |
| 200 | LoopItr != LoopItrE; ++LoopItr) { |
| 201 | Loop *InnerL = *LoopItr; |
Chris Lattner | 4282e32 | 2010-08-29 17:46:00 +0000 | [diff] [blame] | 202 | AliasSetTracker *InnerAST = LoopToAliasSetMap[InnerL]; |
| 203 | assert(InnerAST && "Where is my AST?"); |
Devang Patel | 54959d6 | 2007-03-07 04:41:30 +0000 | [diff] [blame] | 204 | |
| 205 | // What if InnerLoop was modified by other passes ? |
| 206 | CurAST->add(*InnerAST); |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 207 | |
Chris Lattner | 4282e32 | 2010-08-29 17:46:00 +0000 | [diff] [blame] | 208 | // Once we've incorporated the inner loop's AST into ours, we don't need the |
| 209 | // subloop's anymore. |
| 210 | delete InnerAST; |
| 211 | LoopToAliasSetMap.erase(InnerL); |
Chris Lattner | 2e6e741 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 212 | } |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 213 | |
Chris Lattner | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 214 | CurLoop = L; |
| 215 | |
Chris Lattner | 99a5721 | 2002-09-26 19:40:25 +0000 | [diff] [blame] | 216 | // Get the preheader block to move instructions into... |
| 217 | Preheader = L->getLoopPreheader(); |
Chris Lattner | 99a5721 | 2002-09-26 19:40:25 +0000 | [diff] [blame] | 218 | |
Chris Lattner | 2e6e741 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 219 | // Loop over the body of this loop, looking for calls, invokes, and stores. |
Chris Lattner | 0252e49 | 2003-03-03 23:32:45 +0000 | [diff] [blame] | 220 | // Because subloops have already been incorporated into AST, we skip blocks in |
Chris Lattner | 2e6e741 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 221 | // subloops. |
| 222 | // |
Dan Gohman | 9b78763 | 2008-06-22 20:18:58 +0000 | [diff] [blame] | 223 | for (Loop::block_iterator I = L->block_begin(), E = L->block_end(); |
| 224 | I != E; ++I) { |
| 225 | BasicBlock *BB = *I; |
Chris Lattner | 4282e32 | 2010-08-29 17:46:00 +0000 | [diff] [blame] | 226 | if (LI->getLoopFor(BB) == L) // Ignore blocks in subloops. |
Dan Gohman | 9b78763 | 2008-06-22 20:18:58 +0000 | [diff] [blame] | 227 | CurAST->add(*BB); // Incorporate the specified basic block |
| 228 | } |
Chris Lattner | 2e6e741 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 229 | |
Chris Lattner | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 230 | // We want to visit all of the instructions in this loop... that are not parts |
| 231 | // of our subloops (they have already had their invariants hoisted out of |
| 232 | // their loop, into this loop, so there is no need to process the BODIES of |
| 233 | // the subloops). |
| 234 | // |
Chris Lattner | 952eaee | 2002-09-29 21:46:09 +0000 | [diff] [blame] | 235 | // Traverse the body of the loop in depth first order on the dominator tree so |
| 236 | // that we are guaranteed to see definitions before we see uses. This allows |
Nick Lewycky | af5cbc8 | 2007-08-18 15:08:56 +0000 | [diff] [blame] | 237 | // us to sink instructions in one pass, without iteration. After sinking |
Chris Lattner | e4365b2 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 238 | // instructions, we perform another pass to hoist them out of the loop. |
Chris Lattner | 952eaee | 2002-09-29 21:46:09 +0000 | [diff] [blame] | 239 | // |
Dan Gohman | 03e896b | 2009-11-05 21:11:53 +0000 | [diff] [blame] | 240 | if (L->hasDedicatedExits()) |
| 241 | SinkRegion(DT->getNode(L->getHeader())); |
| 242 | if (Preheader) |
| 243 | HoistRegion(DT->getNode(L->getHeader())); |
Chris Lattner | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 244 | |
Chris Lattner | 2e6e741 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 245 | // Now that all loop invariants have been removed from the loop, promote any |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 246 | // memory references to scalars that we can. |
| 247 | if (!DisablePromotion && Preheader && L->hasDedicatedExits()) { |
| 248 | // Loop over all of the alias sets in the tracker object. |
| 249 | for (AliasSetTracker::iterator I = CurAST->begin(), E = CurAST->end(); |
| 250 | I != E; ++I) |
| 251 | PromoteAliasSet(*I); |
| 252 | } |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 253 | |
Chris Lattner | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 254 | // Clear out loops state information for the next iteration |
| 255 | CurLoop = 0; |
Chris Lattner | 99a5721 | 2002-09-26 19:40:25 +0000 | [diff] [blame] | 256 | Preheader = 0; |
Devang Patel | 54959d6 | 2007-03-07 04:41:30 +0000 | [diff] [blame] | 257 | |
Chris Lattner | 4282e32 | 2010-08-29 17:46:00 +0000 | [diff] [blame] | 258 | // If this loop is nested inside of another one, save the alias information |
| 259 | // for when we process the outer loop. |
| 260 | if (L->getParentLoop()) |
| 261 | LoopToAliasSetMap[L] = CurAST; |
| 262 | else |
| 263 | delete CurAST; |
Devang Patel | 54959d6 | 2007-03-07 04:41:30 +0000 | [diff] [blame] | 264 | return Changed; |
Chris Lattner | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 265 | } |
| 266 | |
Chris Lattner | e4365b2 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 267 | /// SinkRegion - Walk the specified region of the CFG (defined by all blocks |
| 268 | /// dominated by the specified block, and that are in the current loop) in |
Owen Anderson | 3a2b58f | 2007-04-24 06:40:39 +0000 | [diff] [blame] | 269 | /// reverse depth first order w.r.t the DominatorTree. This allows us to visit |
Chris Lattner | e4365b2 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 270 | /// uses before definitions, allowing us to sink a loop body in one pass without |
| 271 | /// iteration. |
| 272 | /// |
Devang Patel | 2604242 | 2007-06-04 00:32:22 +0000 | [diff] [blame] | 273 | void LICM::SinkRegion(DomTreeNode *N) { |
Owen Anderson | 3a2b58f | 2007-04-24 06:40:39 +0000 | [diff] [blame] | 274 | assert(N != 0 && "Null dominator tree node?"); |
| 275 | BasicBlock *BB = N->getBlock(); |
Chris Lattner | e4365b2 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 276 | |
| 277 | // If this subregion is not in the top level loop at all, exit. |
| 278 | if (!CurLoop->contains(BB)) return; |
| 279 | |
Chris Lattner | 0de5cad | 2010-08-29 18:22:25 +0000 | [diff] [blame] | 280 | // We are processing blocks in reverse dfo, so process children first. |
Devang Patel | 2604242 | 2007-06-04 00:32:22 +0000 | [diff] [blame] | 281 | const std::vector<DomTreeNode*> &Children = N->getChildren(); |
Chris Lattner | e4365b2 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 282 | for (unsigned i = 0, e = Children.size(); i != e; ++i) |
| 283 | SinkRegion(Children[i]); |
| 284 | |
| 285 | // Only need to process the contents of this block if it is not part of a |
| 286 | // subloop (which would already have been processed). |
| 287 | if (inSubLoop(BB)) return; |
| 288 | |
Chris Lattner | a3df8a9 | 2003-12-19 08:18:16 +0000 | [diff] [blame] | 289 | for (BasicBlock::iterator II = BB->end(); II != BB->begin(); ) { |
| 290 | Instruction &I = *--II; |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 291 | |
Chris Lattner | 0de5cad | 2010-08-29 18:22:25 +0000 | [diff] [blame] | 292 | // If the instruction is dead, we would try to sink it because it isn't used |
| 293 | // in the loop, instead, just delete it. |
| 294 | if (isInstructionTriviallyDead(&I)) { |
Chris Lattner | cb7f653 | 2010-08-29 18:42:23 +0000 | [diff] [blame] | 295 | DEBUG(dbgs() << "LICM deleting dead inst: " << I << '\n'); |
Chris Lattner | 0de5cad | 2010-08-29 18:22:25 +0000 | [diff] [blame] | 296 | ++II; |
| 297 | CurAST->deleteValue(&I); |
| 298 | I.eraseFromParent(); |
| 299 | Changed = true; |
| 300 | continue; |
| 301 | } |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 302 | |
Chris Lattner | e4365b2 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 303 | // Check to see if we can sink this instruction to the exit blocks |
| 304 | // of the loop. We can do this if the all users of the instruction are |
| 305 | // outside of the loop. In this case, it doesn't even matter if the |
| 306 | // operands of the instruction are loop invariant. |
| 307 | // |
Chris Lattner | 70ac2dc | 2005-03-25 00:22:36 +0000 | [diff] [blame] | 308 | if (isNotUsedInLoop(I) && canSinkOrHoistInst(I)) { |
Chris Lattner | a3df8a9 | 2003-12-19 08:18:16 +0000 | [diff] [blame] | 309 | ++II; |
Chris Lattner | e4365b2 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 310 | sink(I); |
Chris Lattner | a3df8a9 | 2003-12-19 08:18:16 +0000 | [diff] [blame] | 311 | } |
Chris Lattner | e4365b2 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 312 | } |
| 313 | } |
| 314 | |
Chris Lattner | 952eaee | 2002-09-29 21:46:09 +0000 | [diff] [blame] | 315 | /// HoistRegion - Walk the specified region of the CFG (defined by all blocks |
| 316 | /// dominated by the specified block, and that are in the current loop) in depth |
Owen Anderson | 3a2b58f | 2007-04-24 06:40:39 +0000 | [diff] [blame] | 317 | /// first order w.r.t the DominatorTree. This allows us to visit definitions |
Chris Lattner | 952eaee | 2002-09-29 21:46:09 +0000 | [diff] [blame] | 318 | /// before uses, allowing us to hoist a loop body in one pass without iteration. |
| 319 | /// |
Devang Patel | 2604242 | 2007-06-04 00:32:22 +0000 | [diff] [blame] | 320 | void LICM::HoistRegion(DomTreeNode *N) { |
Owen Anderson | 3a2b58f | 2007-04-24 06:40:39 +0000 | [diff] [blame] | 321 | assert(N != 0 && "Null dominator tree node?"); |
| 322 | BasicBlock *BB = N->getBlock(); |
Chris Lattner | 952eaee | 2002-09-29 21:46:09 +0000 | [diff] [blame] | 323 | |
Chris Lattner | b461373 | 2002-09-29 22:26:07 +0000 | [diff] [blame] | 324 | // If this subregion is not in the top level loop at all, exit. |
Chris Lattner | ed6dfc2 | 2003-12-09 19:32:44 +0000 | [diff] [blame] | 325 | if (!CurLoop->contains(BB)) return; |
Chris Lattner | 952eaee | 2002-09-29 21:46:09 +0000 | [diff] [blame] | 326 | |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 327 | // Only need to process the contents of this block if it is not part of a |
| 328 | // subloop (which would already have been processed). |
Chris Lattner | ed6dfc2 | 2003-12-09 19:32:44 +0000 | [diff] [blame] | 329 | if (!inSubLoop(BB)) |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 330 | for (BasicBlock::iterator II = BB->begin(), E = BB->end(); II != E; ) { |
| 331 | Instruction &I = *II++; |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 332 | |
Chris Lattner | 2ac6e23 | 2010-08-31 23:00:16 +0000 | [diff] [blame] | 333 | // Try constant folding this instruction. If all the operands are |
| 334 | // constants, it is technically hoistable, but it would be better to just |
| 335 | // fold it. |
| 336 | if (Constant *C = ConstantFoldInstruction(&I)) { |
| 337 | DEBUG(dbgs() << "LICM folding inst: " << I << " --> " << *C << '\n'); |
| 338 | CurAST->copyValue(&I, C); |
| 339 | CurAST->deleteValue(&I); |
| 340 | I.replaceAllUsesWith(C); |
| 341 | I.eraseFromParent(); |
| 342 | continue; |
| 343 | } |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 344 | |
Chris Lattner | e4365b2 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 345 | // Try hoisting the instruction out to the preheader. We can only do this |
| 346 | // if all of the operands of the instruction are loop invariant and if it |
| 347 | // is safe to hoist the instruction. |
| 348 | // |
Chris Lattner | adc7991 | 2010-09-06 01:05:37 +0000 | [diff] [blame] | 349 | if (CurLoop->hasLoopInvariantOperands(&I) && canSinkOrHoistInst(I) && |
Chris Lattner | e4365b2 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 350 | isSafeToExecuteUnconditionally(I)) |
Chris Lattner | 3c80a51 | 2006-06-26 19:10:05 +0000 | [diff] [blame] | 351 | hoist(I); |
Chris Lattner | 2ac6e23 | 2010-08-31 23:00:16 +0000 | [diff] [blame] | 352 | } |
Chris Lattner | 952eaee | 2002-09-29 21:46:09 +0000 | [diff] [blame] | 353 | |
Devang Patel | 2604242 | 2007-06-04 00:32:22 +0000 | [diff] [blame] | 354 | const std::vector<DomTreeNode*> &Children = N->getChildren(); |
Chris Lattner | 952eaee | 2002-09-29 21:46:09 +0000 | [diff] [blame] | 355 | for (unsigned i = 0, e = Children.size(); i != e; ++i) |
Chris Lattner | 56b7ee2 | 2004-06-19 20:23:35 +0000 | [diff] [blame] | 356 | HoistRegion(Children[i]); |
Chris Lattner | 952eaee | 2002-09-29 21:46:09 +0000 | [diff] [blame] | 357 | } |
| 358 | |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 359 | /// canSinkOrHoistInst - Return true if the hoister and sinker can handle this |
| 360 | /// instruction. |
| 361 | /// |
| 362 | bool LICM::canSinkOrHoistInst(Instruction &I) { |
Chris Lattner | ed6dfc2 | 2003-12-09 19:32:44 +0000 | [diff] [blame] | 363 | // Loads have extra constraints we have to verify before we can hoist them. |
| 364 | if (LoadInst *LI = dyn_cast<LoadInst>(&I)) { |
Eli Friedman | 9767156 | 2011-08-15 20:52:09 +0000 | [diff] [blame] | 365 | if (!LI->isUnordered()) |
| 366 | return false; // Don't hoist volatile/atomic loads! |
Chris Lattner | ed6dfc2 | 2003-12-09 19:32:44 +0000 | [diff] [blame] | 367 | |
Chris Lattner | 967948b | 2008-07-23 05:06:28 +0000 | [diff] [blame] | 368 | // Loads from constant memory are always safe to move, even if they end up |
| 369 | // in the same alias set as something that ends up being modified. |
Dan Gohman | dab249b | 2009-11-19 19:00:10 +0000 | [diff] [blame] | 370 | if (AA->pointsToConstantMemory(LI->getOperand(0))) |
Chris Lattner | 967948b | 2008-07-23 05:06:28 +0000 | [diff] [blame] | 371 | return true; |
Pete Cooper | 2d76a78 | 2011-11-08 19:30:00 +0000 | [diff] [blame] | 372 | if (LI->getMetadata(LI->getContext().getMDKindID("invariant.load"))) |
| 373 | return true; |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 374 | |
Chris Lattner | ed6dfc2 | 2003-12-09 19:32:44 +0000 | [diff] [blame] | 375 | // Don't hoist loads which have may-aliased stores in loop. |
Dan Gohman | 3da848b | 2010-10-19 22:54:46 +0000 | [diff] [blame] | 376 | uint64_t Size = 0; |
Chris Lattner | f3e1d69 | 2004-11-26 21:20:09 +0000 | [diff] [blame] | 377 | if (LI->getType()->isSized()) |
Dan Gohman | fc2a3ed | 2009-07-25 00:48:42 +0000 | [diff] [blame] | 378 | Size = AA->getTypeStoreSize(LI->getType()); |
Dan Gohman | a8702ea | 2010-10-18 20:44:50 +0000 | [diff] [blame] | 379 | return !pointerInvalidatedByLoop(LI->getOperand(0), Size, |
| 380 | LI->getMetadata(LLVMContext::MD_tbaa)); |
Chris Lattner | 118dd0c | 2004-03-15 04:11:30 +0000 | [diff] [blame] | 381 | } else if (CallInst *CI = dyn_cast<CallInst>(&I)) { |
Eli Friedman | 30a121b | 2011-05-27 18:37:52 +0000 | [diff] [blame] | 382 | // Don't sink or hoist dbg info; it's legal, but not useful. |
| 383 | if (isa<DbgInfoIntrinsic>(I)) |
| 384 | return false; |
| 385 | |
| 386 | // Handle simple cases by querying alias analysis. |
Duncan Sands | dff6710 | 2007-12-01 07:51:45 +0000 | [diff] [blame] | 387 | AliasAnalysis::ModRefBehavior Behavior = AA->getModRefBehavior(CI); |
| 388 | if (Behavior == AliasAnalysis::DoesNotAccessMemory) |
| 389 | return true; |
Dan Gohman | cd93f3b | 2010-11-09 19:58:21 +0000 | [diff] [blame] | 390 | if (AliasAnalysis::onlyReadsMemory(Behavior)) { |
Duncan Sands | dff6710 | 2007-12-01 07:51:45 +0000 | [diff] [blame] | 391 | // If this call only reads from memory and there are no writes to memory |
| 392 | // in the loop, we can hoist or sink the call as appropriate. |
| 393 | bool FoundMod = false; |
| 394 | for (AliasSetTracker::iterator I = CurAST->begin(), E = CurAST->end(); |
| 395 | I != E; ++I) { |
| 396 | AliasSet &AS = *I; |
| 397 | if (!AS.isForwardingAliasSet() && AS.isMod()) { |
| 398 | FoundMod = true; |
| 399 | break; |
Chris Lattner | 118dd0c | 2004-03-15 04:11:30 +0000 | [diff] [blame] | 400 | } |
Chris Lattner | 118dd0c | 2004-03-15 04:11:30 +0000 | [diff] [blame] | 401 | } |
Duncan Sands | dff6710 | 2007-12-01 07:51:45 +0000 | [diff] [blame] | 402 | if (!FoundMod) return true; |
Chris Lattner | 118dd0c | 2004-03-15 04:11:30 +0000 | [diff] [blame] | 403 | } |
| 404 | |
| 405 | // FIXME: This should use mod/ref information to see if we can hoist or sink |
| 406 | // the call. |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 407 | |
Chris Lattner | 118dd0c | 2004-03-15 04:11:30 +0000 | [diff] [blame] | 408 | return false; |
Chris Lattner | ed6dfc2 | 2003-12-09 19:32:44 +0000 | [diff] [blame] | 409 | } |
| 410 | |
Reid Spencer | 3da59db | 2006-11-27 01:05:10 +0000 | [diff] [blame] | 411 | // Otherwise these instructions are hoistable/sinkable |
Reid Spencer | 832254e | 2007-02-02 02:16:23 +0000 | [diff] [blame] | 412 | return isa<BinaryOperator>(I) || isa<CastInst>(I) || |
Dan Gohman | d8af90c | 2007-06-05 16:05:55 +0000 | [diff] [blame] | 413 | isa<SelectInst>(I) || isa<GetElementPtrInst>(I) || isa<CmpInst>(I) || |
| 414 | isa<InsertElementInst>(I) || isa<ExtractElementInst>(I) || |
| 415 | isa<ShuffleVectorInst>(I); |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 416 | } |
| 417 | |
| 418 | /// isNotUsedInLoop - Return true if the only users of this instruction are |
| 419 | /// outside of the loop. If this is true, we can sink the instruction to the |
| 420 | /// exit blocks of the loop. |
| 421 | /// |
| 422 | bool LICM::isNotUsedInLoop(Instruction &I) { |
Chris Lattner | ea9403f | 2003-12-11 22:23:32 +0000 | [diff] [blame] | 423 | for (Value::use_iterator UI = I.use_begin(), E = I.use_end(); UI != E; ++UI) { |
| 424 | Instruction *User = cast<Instruction>(*UI); |
| 425 | if (PHINode *PN = dyn_cast<PHINode>(User)) { |
| 426 | // PHI node uses occur in predecessor blocks! |
| 427 | for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) |
| 428 | if (PN->getIncomingValue(i) == &I) |
| 429 | if (CurLoop->contains(PN->getIncomingBlock(i))) |
| 430 | return false; |
Dan Gohman | 92329c7 | 2009-12-18 01:24:09 +0000 | [diff] [blame] | 431 | } else if (CurLoop->contains(User)) { |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 432 | return false; |
Chris Lattner | ea9403f | 2003-12-11 22:23:32 +0000 | [diff] [blame] | 433 | } |
| 434 | } |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 435 | return true; |
| 436 | } |
| 437 | |
| 438 | |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 439 | /// sink - When an instruction is found to only be used outside of the loop, |
Chris Lattner | a3df8a9 | 2003-12-19 08:18:16 +0000 | [diff] [blame] | 440 | /// this function moves it to the exit blocks and patches up SSA form as needed. |
| 441 | /// This method is guaranteed to remove the original instruction from its |
| 442 | /// position, and may either delete it or move it to outside of the loop. |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 443 | /// |
| 444 | void LICM::sink(Instruction &I) { |
Nick Lewycky | 39a3831 | 2010-07-30 20:27:01 +0000 | [diff] [blame] | 445 | DEBUG(dbgs() << "LICM sinking instruction: " << I << "\n"); |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 446 | |
Devang Patel | b7211a2 | 2007-08-21 00:31:24 +0000 | [diff] [blame] | 447 | SmallVector<BasicBlock*, 8> ExitBlocks; |
Chris Lattner | d7bc19d | 2010-08-29 04:28:20 +0000 | [diff] [blame] | 448 | CurLoop->getUniqueExitBlocks(ExitBlocks); |
Chris Lattner | df45bd3 | 2003-12-10 20:43:29 +0000 | [diff] [blame] | 449 | |
| 450 | if (isa<LoadInst>(I)) ++NumMovedLoads; |
Chris Lattner | 118dd0c | 2004-03-15 04:11:30 +0000 | [diff] [blame] | 451 | else if (isa<CallInst>(I)) ++NumMovedCalls; |
Chris Lattner | df45bd3 | 2003-12-10 20:43:29 +0000 | [diff] [blame] | 452 | ++NumSunk; |
| 453 | Changed = true; |
| 454 | |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 455 | // The case where there is only a single exit node of this loop is common |
| 456 | // enough that we handle it as a special (more efficient) case. It is more |
| 457 | // efficient to handle because there are no PHI nodes that need to be placed. |
| 458 | if (ExitBlocks.size() == 1) { |
Eli Friedman | 30a121b | 2011-05-27 18:37:52 +0000 | [diff] [blame] | 459 | if (!DT->dominates(I.getParent(), ExitBlocks[0])) { |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 460 | // Instruction is not used, just delete it. |
Chris Lattner | 2741c97 | 2004-05-23 21:20:19 +0000 | [diff] [blame] | 461 | CurAST->deleteValue(&I); |
Devang Patel | 1bf5ebc | 2009-10-13 21:41:20 +0000 | [diff] [blame] | 462 | // If I has users in unreachable blocks, eliminate. |
Devang Patel | 228ebd0 | 2009-10-13 22:56:32 +0000 | [diff] [blame] | 463 | // If I is not void type then replaceAllUsesWith undef. |
| 464 | // This allows ValueHandlers and custom metadata to adjust itself. |
Chris Lattner | a6a36f5 | 2010-08-29 04:55:06 +0000 | [diff] [blame] | 465 | if (!I.use_empty()) |
Devang Patel | 228ebd0 | 2009-10-13 22:56:32 +0000 | [diff] [blame] | 466 | I.replaceAllUsesWith(UndefValue::get(I.getType())); |
Chris Lattner | 3c80a51 | 2006-06-26 19:10:05 +0000 | [diff] [blame] | 467 | I.eraseFromParent(); |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 468 | } else { |
| 469 | // Move the instruction to the start of the exit block, after any PHI |
| 470 | // nodes in it. |
Bill Wendling | 26665de | 2011-08-18 23:42:36 +0000 | [diff] [blame] | 471 | I.moveBefore(ExitBlocks[0]->getFirstInsertionPt()); |
Chris Lattner | 4282e32 | 2010-08-29 17:46:00 +0000 | [diff] [blame] | 472 | |
| 473 | // This instruction is no longer in the AST for the current loop, because |
| 474 | // we just sunk it out of the loop. If we just sunk it into an outer |
| 475 | // loop, we will rediscover the operation when we process it. |
| 476 | CurAST->deleteValue(&I); |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 477 | } |
Chris Lattner | d7bc19d | 2010-08-29 04:28:20 +0000 | [diff] [blame] | 478 | return; |
| 479 | } |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 480 | |
Chris Lattner | d7bc19d | 2010-08-29 04:28:20 +0000 | [diff] [blame] | 481 | if (ExitBlocks.empty()) { |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 482 | // The instruction is actually dead if there ARE NO exit blocks. |
Chris Lattner | 2741c97 | 2004-05-23 21:20:19 +0000 | [diff] [blame] | 483 | CurAST->deleteValue(&I); |
Devang Patel | 1bf5ebc | 2009-10-13 21:41:20 +0000 | [diff] [blame] | 484 | // If I has users in unreachable blocks, eliminate. |
Devang Patel | 228ebd0 | 2009-10-13 22:56:32 +0000 | [diff] [blame] | 485 | // If I is not void type then replaceAllUsesWith undef. |
| 486 | // This allows ValueHandlers and custom metadata to adjust itself. |
Chris Lattner | a6a36f5 | 2010-08-29 04:55:06 +0000 | [diff] [blame] | 487 | if (!I.use_empty()) |
Devang Patel | 228ebd0 | 2009-10-13 22:56:32 +0000 | [diff] [blame] | 488 | I.replaceAllUsesWith(UndefValue::get(I.getType())); |
Chris Lattner | 3c80a51 | 2006-06-26 19:10:05 +0000 | [diff] [blame] | 489 | I.eraseFromParent(); |
Chris Lattner | d7bc19d | 2010-08-29 04:28:20 +0000 | [diff] [blame] | 490 | return; |
| 491 | } |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 492 | |
Chris Lattner | a6a36f5 | 2010-08-29 04:55:06 +0000 | [diff] [blame] | 493 | // Otherwise, if we have multiple exits, use the SSAUpdater to do all of the |
| 494 | // hard work of inserting PHI nodes as necessary. |
| 495 | SmallVector<PHINode*, 8> NewPHIs; |
| 496 | SSAUpdater SSA(&NewPHIs); |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 497 | |
Chris Lattner | a6a36f5 | 2010-08-29 04:55:06 +0000 | [diff] [blame] | 498 | if (!I.use_empty()) |
Duncan Sands | fc6e29d | 2010-09-02 08:14:03 +0000 | [diff] [blame] | 499 | SSA.Initialize(I.getType(), I.getName()); |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 500 | |
Chris Lattner | a6a36f5 | 2010-08-29 04:55:06 +0000 | [diff] [blame] | 501 | // Insert a copy of the instruction in each exit block of the loop that is |
| 502 | // dominated by the instruction. Each exit block is known to only be in the |
| 503 | // ExitBlocks list once. |
| 504 | BasicBlock *InstOrigBB = I.getParent(); |
Chris Lattner | d7bc19d | 2010-08-29 04:28:20 +0000 | [diff] [blame] | 505 | unsigned NumInserted = 0; |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 506 | |
Chris Lattner | d7bc19d | 2010-08-29 04:28:20 +0000 | [diff] [blame] | 507 | for (unsigned i = 0, e = ExitBlocks.size(); i != e; ++i) { |
| 508 | BasicBlock *ExitBlock = ExitBlocks[i]; |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 509 | |
Chris Lattner | 83fc584 | 2011-01-02 18:45:39 +0000 | [diff] [blame] | 510 | if (!DT->dominates(InstOrigBB, ExitBlock)) |
Chris Lattner | d7bc19d | 2010-08-29 04:28:20 +0000 | [diff] [blame] | 511 | continue; |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 512 | |
Chris Lattner | a6a36f5 | 2010-08-29 04:55:06 +0000 | [diff] [blame] | 513 | // Insert the code after the last PHI node. |
Bill Wendling | 26665de | 2011-08-18 23:42:36 +0000 | [diff] [blame] | 514 | BasicBlock::iterator InsertPt = ExitBlock->getFirstInsertionPt(); |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 515 | |
Chris Lattner | d7bc19d | 2010-08-29 04:28:20 +0000 | [diff] [blame] | 516 | // If this is the first exit block processed, just move the original |
| 517 | // instruction, otherwise clone the original instruction and insert |
| 518 | // the copy. |
| 519 | Instruction *New; |
Chris Lattner | a6a36f5 | 2010-08-29 04:55:06 +0000 | [diff] [blame] | 520 | if (NumInserted++ == 0) { |
| 521 | I.moveBefore(InsertPt); |
Chris Lattner | d7bc19d | 2010-08-29 04:28:20 +0000 | [diff] [blame] | 522 | New = &I; |
| 523 | } else { |
| 524 | New = I.clone(); |
Chris Lattner | d7bc19d | 2010-08-29 04:28:20 +0000 | [diff] [blame] | 525 | if (!I.getName().empty()) |
| 526 | New->setName(I.getName()+".le"); |
| 527 | ExitBlock->getInstList().insert(InsertPt, New); |
Chris Lattner | eaa2430 | 2004-07-27 07:38:32 +0000 | [diff] [blame] | 528 | } |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 529 | |
Chris Lattner | a6a36f5 | 2010-08-29 04:55:06 +0000 | [diff] [blame] | 530 | // Now that we have inserted the instruction, inform SSAUpdater. |
| 531 | if (!I.use_empty()) |
| 532 | SSA.AddAvailableValue(ExitBlock, New); |
Chris Lattner | d7bc19d | 2010-08-29 04:28:20 +0000 | [diff] [blame] | 533 | } |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 534 | |
Eli Friedman | c955d21 | 2011-05-27 18:04:04 +0000 | [diff] [blame] | 535 | // If the instruction doesn't dominate any exit blocks, it must be dead. |
| 536 | if (NumInserted == 0) { |
| 537 | CurAST->deleteValue(&I); |
| 538 | if (!I.use_empty()) |
| 539 | I.replaceAllUsesWith(UndefValue::get(I.getType())); |
| 540 | I.eraseFromParent(); |
| 541 | return; |
| 542 | } |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 543 | |
Chris Lattner | a6a36f5 | 2010-08-29 04:55:06 +0000 | [diff] [blame] | 544 | // Next, rewrite uses of the instruction, inserting PHI nodes as needed. |
| 545 | for (Value::use_iterator UI = I.use_begin(), UE = I.use_end(); UI != UE; ) { |
| 546 | // Grab the use before incrementing the iterator. |
| 547 | Use &U = UI.getUse(); |
| 548 | // Increment the iterator before removing the use from the list. |
| 549 | ++UI; |
| 550 | SSA.RewriteUseAfterInsertions(U); |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 551 | } |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 552 | |
Chris Lattner | a6a36f5 | 2010-08-29 04:55:06 +0000 | [diff] [blame] | 553 | // Update CurAST for NewPHIs if I had pointer type. |
| 554 | if (I.getType()->isPointerTy()) |
| 555 | for (unsigned i = 0, e = NewPHIs.size(); i != e; ++i) |
Chris Lattner | 9c28201 | 2010-09-02 22:19:10 +0000 | [diff] [blame] | 556 | CurAST->copyValue(&I, NewPHIs[i]); |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 557 | |
Chris Lattner | 9891750 | 2010-08-29 18:00:00 +0000 | [diff] [blame] | 558 | // Finally, remove the instruction from CurAST. It is no longer in the loop. |
| 559 | CurAST->deleteValue(&I); |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 560 | } |
Chris Lattner | 952eaee | 2002-09-29 21:46:09 +0000 | [diff] [blame] | 561 | |
Chris Lattner | 9417059 | 2002-09-26 16:52:07 +0000 | [diff] [blame] | 562 | /// hoist - When an instruction is found to only use loop invariant operands |
| 563 | /// that is safe to hoist, this instruction is called to do the dirty work. |
| 564 | /// |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 565 | void LICM::hoist(Instruction &I) { |
David Greene | f158269 | 2010-01-05 01:27:30 +0000 | [diff] [blame] | 566 | DEBUG(dbgs() << "LICM hoisting to " << Preheader->getName() << ": " |
Evan Cheng | 67d1d1f | 2009-10-12 22:25:23 +0000 | [diff] [blame] | 567 | << I << "\n"); |
Chris Lattner | ed6dfc2 | 2003-12-09 19:32:44 +0000 | [diff] [blame] | 568 | |
Chris Lattner | d9a5dae | 2010-08-29 18:18:40 +0000 | [diff] [blame] | 569 | // Move the new node to the Preheader, before its terminator. |
| 570 | I.moveBefore(Preheader->getTerminator()); |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 571 | |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 572 | if (isa<LoadInst>(I)) ++NumMovedLoads; |
Chris Lattner | 118dd0c | 2004-03-15 04:11:30 +0000 | [diff] [blame] | 573 | else if (isa<CallInst>(I)) ++NumMovedCalls; |
Chris Lattner | 9646e6b | 2002-09-26 16:38:03 +0000 | [diff] [blame] | 574 | ++NumHoisted; |
Chris Lattner | e0e734e | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 575 | Changed = true; |
| 576 | } |
| 577 | |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 578 | /// isSafeToExecuteUnconditionally - Only sink or hoist an instruction if it is |
| 579 | /// not a trapping instruction or if it is a trapping instruction and is |
| 580 | /// guaranteed to execute. |
Tanya Lattner | 9966c03 | 2003-08-05 18:45:46 +0000 | [diff] [blame] | 581 | /// |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 582 | bool LICM::isSafeToExecuteUnconditionally(Instruction &Inst) { |
Chris Lattner | 92094b4 | 2003-12-09 17:18:00 +0000 | [diff] [blame] | 583 | // If it is not a trapping instruction, it is always safe to hoist. |
Eli Friedman | 0b79a77 | 2009-07-17 04:28:42 +0000 | [diff] [blame] | 584 | if (Inst.isSafeToSpeculativelyExecute()) |
| 585 | return true; |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 586 | |
Eli Friedman | 73bfa4a | 2011-07-20 21:37:47 +0000 | [diff] [blame] | 587 | return isGuaranteedToExecute(Inst); |
| 588 | } |
| 589 | |
| 590 | bool LICM::isGuaranteedToExecute(Instruction &Inst) { |
Chris Lattner | 92094b4 | 2003-12-09 17:18:00 +0000 | [diff] [blame] | 591 | // Otherwise we have to check to make sure that the instruction dominates all |
| 592 | // of the exit blocks. If it doesn't, then there is a path out of the loop |
| 593 | // which does not execute this instruction, so we can't hoist it. |
Tanya Lattner | 9966c03 | 2003-08-05 18:45:46 +0000 | [diff] [blame] | 594 | |
Chris Lattner | 92094b4 | 2003-12-09 17:18:00 +0000 | [diff] [blame] | 595 | // If the instruction is in the header block for the loop (which is very |
| 596 | // common), it is always guaranteed to dominate the exit blocks. Since this |
| 597 | // is a common case, and can save some work, check it now. |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 598 | if (Inst.getParent() == CurLoop->getHeader()) |
Chris Lattner | 92094b4 | 2003-12-09 17:18:00 +0000 | [diff] [blame] | 599 | return true; |
Tanya Lattner | 9966c03 | 2003-08-05 18:45:46 +0000 | [diff] [blame] | 600 | |
Chris Lattner | 92094b4 | 2003-12-09 17:18:00 +0000 | [diff] [blame] | 601 | // Get the exit blocks for the current loop. |
Devang Patel | b7211a2 | 2007-08-21 00:31:24 +0000 | [diff] [blame] | 602 | SmallVector<BasicBlock*, 8> ExitBlocks; |
Chris Lattner | 5fa802f | 2004-04-18 22:15:13 +0000 | [diff] [blame] | 603 | CurLoop->getExitBlocks(ExitBlocks); |
Chris Lattner | 92094b4 | 2003-12-09 17:18:00 +0000 | [diff] [blame] | 604 | |
Chris Lattner | 83fc584 | 2011-01-02 18:45:39 +0000 | [diff] [blame] | 605 | // Verify that the block dominates each of the exit blocks of the loop. |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 606 | for (unsigned i = 0, e = ExitBlocks.size(); i != e; ++i) |
Chris Lattner | 83fc584 | 2011-01-02 18:45:39 +0000 | [diff] [blame] | 607 | if (!DT->dominates(Inst.getParent(), ExitBlocks[i])) |
Chris Lattner | a270651 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 608 | return false; |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 609 | |
Tanya Lattner | 9966c03 | 2003-08-05 18:45:46 +0000 | [diff] [blame] | 610 | return true; |
| 611 | } |
| 612 | |
Chris Lattner | deaf55f | 2011-01-15 00:12:35 +0000 | [diff] [blame] | 613 | namespace { |
| 614 | class LoopPromoter : public LoadAndStorePromoter { |
| 615 | Value *SomePtr; // Designated pointer to store to. |
| 616 | SmallPtrSet<Value*, 4> &PointerMustAliases; |
| 617 | SmallVectorImpl<BasicBlock*> &LoopExitBlocks; |
| 618 | AliasSetTracker &AST; |
Eli Friedman | dda266d | 2011-05-27 20:31:51 +0000 | [diff] [blame] | 619 | DebugLoc DL; |
Tobias Grosser | df7102b | 2011-07-06 19:19:55 +0000 | [diff] [blame] | 620 | int Alignment; |
Chris Lattner | deaf55f | 2011-01-15 00:12:35 +0000 | [diff] [blame] | 621 | public: |
| 622 | LoopPromoter(Value *SP, |
| 623 | const SmallVectorImpl<Instruction*> &Insts, SSAUpdater &S, |
| 624 | SmallPtrSet<Value*, 4> &PMA, |
Eli Friedman | dda266d | 2011-05-27 20:31:51 +0000 | [diff] [blame] | 625 | SmallVectorImpl<BasicBlock*> &LEB, AliasSetTracker &ast, |
Tobias Grosser | df7102b | 2011-07-06 19:19:55 +0000 | [diff] [blame] | 626 | DebugLoc dl, int alignment) |
Devang Patel | 231a5ab | 2011-07-06 21:09:55 +0000 | [diff] [blame] | 627 | : LoadAndStorePromoter(Insts, S), SomePtr(SP), |
Tobias Grosser | df7102b | 2011-07-06 19:19:55 +0000 | [diff] [blame] | 628 | PointerMustAliases(PMA), LoopExitBlocks(LEB), AST(ast), DL(dl), |
| 629 | Alignment(alignment) {} |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 630 | |
Chris Lattner | deaf55f | 2011-01-15 00:12:35 +0000 | [diff] [blame] | 631 | virtual bool isInstInList(Instruction *I, |
| 632 | const SmallVectorImpl<Instruction*> &) const { |
| 633 | Value *Ptr; |
| 634 | if (LoadInst *LI = dyn_cast<LoadInst>(I)) |
| 635 | Ptr = LI->getOperand(0); |
| 636 | else |
| 637 | Ptr = cast<StoreInst>(I)->getPointerOperand(); |
| 638 | return PointerMustAliases.count(Ptr); |
| 639 | } |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 640 | |
Chris Lattner | deaf55f | 2011-01-15 00:12:35 +0000 | [diff] [blame] | 641 | virtual void doExtraRewritesBeforeFinalDeletion() const { |
| 642 | // Insert stores after in the loop exit blocks. Each exit block gets a |
| 643 | // store of the live-out values that feed them. Since we've already told |
| 644 | // the SSA updater about the defs in the loop and the preheader |
| 645 | // definition, it is all set and we can start using it. |
| 646 | for (unsigned i = 0, e = LoopExitBlocks.size(); i != e; ++i) { |
| 647 | BasicBlock *ExitBlock = LoopExitBlocks[i]; |
| 648 | Value *LiveInValue = SSA.GetValueInMiddleOfBlock(ExitBlock); |
Bill Wendling | 26665de | 2011-08-18 23:42:36 +0000 | [diff] [blame] | 649 | Instruction *InsertPos = ExitBlock->getFirstInsertionPt(); |
Eli Friedman | dda266d | 2011-05-27 20:31:51 +0000 | [diff] [blame] | 650 | StoreInst *NewSI = new StoreInst(LiveInValue, SomePtr, InsertPos); |
Tobias Grosser | df7102b | 2011-07-06 19:19:55 +0000 | [diff] [blame] | 651 | NewSI->setAlignment(Alignment); |
Eli Friedman | dda266d | 2011-05-27 20:31:51 +0000 | [diff] [blame] | 652 | NewSI->setDebugLoc(DL); |
Chris Lattner | deaf55f | 2011-01-15 00:12:35 +0000 | [diff] [blame] | 653 | } |
| 654 | } |
| 655 | |
| 656 | virtual void replaceLoadWithValue(LoadInst *LI, Value *V) const { |
| 657 | // Update alias analysis. |
| 658 | AST.copyValue(LI, V); |
| 659 | } |
| 660 | virtual void instructionDeleted(Instruction *I) const { |
| 661 | AST.deleteValue(I); |
| 662 | } |
| 663 | }; |
| 664 | } // end anon namespace |
| 665 | |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 666 | /// PromoteAliasSet - Try to promote memory values to scalars by sinking |
Chris Lattner | 2e6e741 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 667 | /// stores out of the loop and moving loads to before the loop. We do this by |
| 668 | /// looping over the stores in the loop, looking for stores to Must pointers |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 669 | /// which are loop invariant. |
Chris Lattner | 2e6e741 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 670 | /// |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 671 | void LICM::PromoteAliasSet(AliasSet &AS) { |
| 672 | // We can promote this alias set if it has a store, if it is a "Must" alias |
| 673 | // set, if the pointer is loop invariant, and if we are not eliminating any |
| 674 | // volatile loads or stores. |
| 675 | if (AS.isForwardingAliasSet() || !AS.isMod() || !AS.isMustAlias() || |
| 676 | AS.isVolatile() || !CurLoop->isLoopInvariant(AS.begin()->getValue())) |
| 677 | return; |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 678 | |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 679 | assert(!AS.empty() && |
| 680 | "Must alias set should have at least one pointer element in it!"); |
| 681 | Value *SomePtr = AS.begin()->getValue(); |
Chris Lattner | 2e6e741 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 682 | |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 683 | // It isn't safe to promote a load/store from the loop if the load/store is |
| 684 | // conditional. For example, turning: |
Chris Lattner | 2e6e741 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 685 | // |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 686 | // for () { if (c) *P += 1; } |
Chris Lattner | 2e6e741 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 687 | // |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 688 | // into: |
| 689 | // |
| 690 | // tmp = *P; for () { if (c) tmp +=1; } *P = tmp; |
| 691 | // |
| 692 | // is not safe, because *P may only be valid to access if 'c' is true. |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 693 | // |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 694 | // It is safe to promote P if all uses are direct load/stores and if at |
| 695 | // least one is guaranteed to be executed. |
| 696 | bool GuaranteedToExecute = false; |
Tobias Grosser | df7102b | 2011-07-06 19:19:55 +0000 | [diff] [blame] | 697 | |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 698 | SmallVector<Instruction*, 64> LoopUses; |
| 699 | SmallPtrSet<Value*, 4> PointerMustAliases; |
Chris Lattner | 2e6e741 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 700 | |
Tobias Grosser | df7102b | 2011-07-06 19:19:55 +0000 | [diff] [blame] | 701 | // We start with an alignment of one and try to find instructions that allow |
| 702 | // us to prove better alignment. |
| 703 | unsigned Alignment = 1; |
| 704 | |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 705 | // Check that all of the pointers in the alias set have the same type. We |
| 706 | // cannot (yet) promote a memory location that is loaded and stored in |
| 707 | // different sizes. |
| 708 | for (AliasSet::iterator ASI = AS.begin(), E = AS.end(); ASI != E; ++ASI) { |
| 709 | Value *ASIV = ASI->getValue(); |
| 710 | PointerMustAliases.insert(ASIV); |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 711 | |
Chris Lattner | 29d9293 | 2008-05-22 00:53:38 +0000 | [diff] [blame] | 712 | // Check that all of the pointers in the alias set have the same type. We |
| 713 | // cannot (yet) promote a memory location that is loaded and stored in |
| 714 | // different sizes. |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 715 | if (SomePtr->getType() != ASIV->getType()) |
| 716 | return; |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 717 | |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 718 | for (Value::use_iterator UI = ASIV->use_begin(), UE = ASIV->use_end(); |
Chris Lattner | 19d9d43 | 2008-05-22 03:22:42 +0000 | [diff] [blame] | 719 | UI != UE; ++UI) { |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 720 | // Ignore instructions that are outside the loop. |
Chris Lattner | 29d9293 | 2008-05-22 00:53:38 +0000 | [diff] [blame] | 721 | Instruction *Use = dyn_cast<Instruction>(*UI); |
Dan Gohman | 92329c7 | 2009-12-18 01:24:09 +0000 | [diff] [blame] | 722 | if (!Use || !CurLoop->contains(Use)) |
Chris Lattner | 29d9293 | 2008-05-22 00:53:38 +0000 | [diff] [blame] | 723 | continue; |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 724 | |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 725 | // If there is an non-load/store instruction in the loop, we can't promote |
| 726 | // it. |
Eli Friedman | 9767156 | 2011-08-15 20:52:09 +0000 | [diff] [blame] | 727 | if (LoadInst *load = dyn_cast<LoadInst>(Use)) { |
| 728 | assert(!load->isVolatile() && "AST broken"); |
| 729 | if (!load->isSimple()) |
| 730 | return; |
Tobias Grosser | df7102b | 2011-07-06 19:19:55 +0000 | [diff] [blame] | 731 | } else if (StoreInst *store = dyn_cast<StoreInst>(Use)) { |
Chris Lattner | 1dec0d2 | 2010-12-19 05:57:25 +0000 | [diff] [blame] | 732 | // Stores *of* the pointer are not interesting, only stores *to* the |
| 733 | // pointer. |
| 734 | if (Use->getOperand(1) != ASIV) |
| 735 | continue; |
Eli Friedman | 9767156 | 2011-08-15 20:52:09 +0000 | [diff] [blame] | 736 | assert(!store->isVolatile() && "AST broken"); |
| 737 | if (!store->isSimple()) |
| 738 | return; |
Eli Friedman | 73bfa4a | 2011-07-20 21:37:47 +0000 | [diff] [blame] | 739 | |
| 740 | // Note that we only check GuaranteedToExecute inside the store case |
| 741 | // so that we do not introduce stores where they did not exist before |
| 742 | // (which would break the LLVM concurrency model). |
| 743 | |
| 744 | // If the alignment of this instruction allows us to specify a more |
| 745 | // restrictive (and performant) alignment and if we are sure this |
| 746 | // instruction will be executed, update the alignment. |
| 747 | // Larger is better, with the exception of 0 being the best alignment. |
Eli Friedman | 9767156 | 2011-08-15 20:52:09 +0000 | [diff] [blame] | 748 | unsigned InstAlignment = store->getAlignment(); |
Eli Friedman | 73bfa4a | 2011-07-20 21:37:47 +0000 | [diff] [blame] | 749 | if ((InstAlignment > Alignment || InstAlignment == 0) |
| 750 | && (Alignment != 0)) |
| 751 | if (isGuaranteedToExecute(*Use)) { |
| 752 | GuaranteedToExecute = true; |
| 753 | Alignment = InstAlignment; |
| 754 | } |
| 755 | |
| 756 | if (!GuaranteedToExecute) |
| 757 | GuaranteedToExecute = isGuaranteedToExecute(*Use); |
| 758 | |
Chris Lattner | 0cccd76 | 2010-09-06 05:11:24 +0000 | [diff] [blame] | 759 | } else |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 760 | return; // Not a load or store. |
Tobias Grosser | df7102b | 2011-07-06 19:19:55 +0000 | [diff] [blame] | 761 | |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 762 | LoopUses.push_back(Use); |
| 763 | } |
| 764 | } |
Tobias Grosser | df7102b | 2011-07-06 19:19:55 +0000 | [diff] [blame] | 765 | |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 766 | // If there isn't a guaranteed-to-execute instruction, we can't promote. |
| 767 | if (!GuaranteedToExecute) |
| 768 | return; |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 769 | |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 770 | // Otherwise, this is safe to promote, lets do it! |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 771 | DEBUG(dbgs() << "LICM: Promoting value stored to in loop: " <<*SomePtr<<'\n'); |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 772 | Changed = true; |
| 773 | ++NumPromoted; |
| 774 | |
Eli Friedman | dda266d | 2011-05-27 20:31:51 +0000 | [diff] [blame] | 775 | // Grab a debug location for the inserted loads/stores; given that the |
| 776 | // inserted loads/stores have little relation to the original loads/stores, |
| 777 | // this code just arbitrarily picks a location from one, since any debug |
| 778 | // location is better than none. |
| 779 | DebugLoc DL = LoopUses[0]->getDebugLoc(); |
| 780 | |
Chris Lattner | deaf55f | 2011-01-15 00:12:35 +0000 | [diff] [blame] | 781 | SmallVector<BasicBlock*, 8> ExitBlocks; |
| 782 | CurLoop->getUniqueExitBlocks(ExitBlocks); |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 783 | |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 784 | // We use the SSAUpdater interface to insert phi nodes as required. |
| 785 | SmallVector<PHINode*, 16> NewPHIs; |
| 786 | SSAUpdater SSA(&NewPHIs); |
Chris Lattner | deaf55f | 2011-01-15 00:12:35 +0000 | [diff] [blame] | 787 | LoopPromoter Promoter(SomePtr, LoopUses, SSA, PointerMustAliases, ExitBlocks, |
Tobias Grosser | df7102b | 2011-07-06 19:19:55 +0000 | [diff] [blame] | 788 | *CurAST, DL, Alignment); |
Tobias Grosser | a3574fb | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 789 | |
Chris Lattner | deaf55f | 2011-01-15 00:12:35 +0000 | [diff] [blame] | 790 | // Set up the preheader to have a definition of the value. It is the live-out |
| 791 | // value from the preheader that uses in the loop will use. |
| 792 | LoadInst *PreheaderLoad = |
| 793 | new LoadInst(SomePtr, SomePtr->getName()+".promoted", |
| 794 | Preheader->getTerminator()); |
Tobias Grosser | df7102b | 2011-07-06 19:19:55 +0000 | [diff] [blame] | 795 | PreheaderLoad->setAlignment(Alignment); |
Eli Friedman | dda266d | 2011-05-27 20:31:51 +0000 | [diff] [blame] | 796 | PreheaderLoad->setDebugLoc(DL); |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 797 | SSA.AddAvailableValue(Preheader, PreheaderLoad); |
| 798 | |
Chris Lattner | deaf55f | 2011-01-15 00:12:35 +0000 | [diff] [blame] | 799 | // Rewrite all the loads in the loop and remember all the definitions from |
| 800 | // stores in the loop. |
| 801 | Promoter.run(LoopUses); |
Eli Friedman | 0e38219 | 2011-04-07 01:35:06 +0000 | [diff] [blame] | 802 | |
| 803 | // If the SSAUpdater didn't use the load in the preheader, just zap it now. |
| 804 | if (PreheaderLoad->use_empty()) |
| 805 | PreheaderLoad->eraseFromParent(); |
Chris Lattner | f5e84aa | 2002-08-22 21:39:55 +0000 | [diff] [blame] | 806 | } |
Devang Patel | 91d22c8 | 2007-07-31 08:01:41 +0000 | [diff] [blame] | 807 | |
Chris Lattner | e488064 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 808 | |
Devang Patel | 91d22c8 | 2007-07-31 08:01:41 +0000 | [diff] [blame] | 809 | /// cloneBasicBlockAnalysis - Simple Analysis hook. Clone alias set info. |
| 810 | void LICM::cloneBasicBlockAnalysis(BasicBlock *From, BasicBlock *To, Loop *L) { |
Chris Lattner | 4282e32 | 2010-08-29 17:46:00 +0000 | [diff] [blame] | 811 | AliasSetTracker *AST = LoopToAliasSetMap.lookup(L); |
Devang Patel | 91d22c8 | 2007-07-31 08:01:41 +0000 | [diff] [blame] | 812 | if (!AST) |
| 813 | return; |
| 814 | |
| 815 | AST->copyValue(From, To); |
| 816 | } |
| 817 | |
| 818 | /// deleteAnalysisValue - Simple Analysis hook. Delete value V from alias |
| 819 | /// set. |
| 820 | void LICM::deleteAnalysisValue(Value *V, Loop *L) { |
Chris Lattner | 4282e32 | 2010-08-29 17:46:00 +0000 | [diff] [blame] | 821 | AliasSetTracker *AST = LoopToAliasSetMap.lookup(L); |
Devang Patel | 91d22c8 | 2007-07-31 08:01:41 +0000 | [diff] [blame] | 822 | if (!AST) |
| 823 | return; |
| 824 | |
| 825 | AST->deleteValue(V); |
| 826 | } |