Chris Lattner | 6ec05f5 | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 1 | //===-- LICM.cpp - Loop Invariant Code Motion Pass ------------------------===// |
Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 2 | // |
John Criswell | 482202a | 2003-10-20 19:43:21 +0000 | [diff] [blame] | 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
Chris Lattner | f3ebc3f | 2007-12-29 20:36:04 +0000 | [diff] [blame] | 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 7 | // |
John Criswell | 482202a | 2003-10-20 19:43:21 +0000 | [diff] [blame] | 8 | //===----------------------------------------------------------------------===// |
Chris Lattner | 6ec05f5 | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 9 | // |
Chris Lattner | c051768 | 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 | 547192d6 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 14 | // live in registers, thus hoisting and sinking "invariant" loads and stores. |
Chris Lattner | c051768 | 2003-12-09 17:18:00 +0000 | [diff] [blame] | 15 | // |
| 16 | // This pass uses alias analysis for two purposes: |
Chris Lattner | 45d67d6 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 17 | // |
Chris Lattner | 289ba2a | 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 | 45d67d6 | 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 | 1dc98b4 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 29 | // the SSAUpdater to construct the appropriate SSA form for the value. |
Chris Lattner | 6ec05f5 | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 30 | // |
Chris Lattner | 6ec05f5 | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 31 | //===----------------------------------------------------------------------===// |
| 32 | |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 33 | #include "llvm/Transforms/Scalar/LICM.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 34 | #include "llvm/ADT/Statistic.h" |
Chris Lattner | a51fa88 | 2002-08-22 21:39:55 +0000 | [diff] [blame] | 35 | #include "llvm/Analysis/AliasAnalysis.h" |
Chris Lattner | 0592bb7 | 2003-03-03 23:32:45 +0000 | [diff] [blame] | 36 | #include "llvm/Analysis/AliasSetTracker.h" |
Chandler Carruth | 7b560d4 | 2015-09-09 17:55:00 +0000 | [diff] [blame] | 37 | #include "llvm/Analysis/BasicAliasAnalysis.h" |
Philip Reames | b54c8e6 | 2016-03-09 22:59:30 +0000 | [diff] [blame] | 38 | #include "llvm/Analysis/CaptureTracking.h" |
Chris Lattner | 030f020 | 2010-08-31 23:00:16 +0000 | [diff] [blame] | 39 | #include "llvm/Analysis/ConstantFolding.h" |
Chandler Carruth | 7b560d4 | 2015-09-09 17:55:00 +0000 | [diff] [blame] | 40 | #include "llvm/Analysis/GlobalsModRef.h" |
Philip Reames | e0a5454 | 2016-03-09 23:07:53 +0000 | [diff] [blame] | 41 | #include "llvm/Analysis/Loads.h" |
Chris Lattner | 030f020 | 2010-08-31 23:00:16 +0000 | [diff] [blame] | 42 | #include "llvm/Analysis/LoopInfo.h" |
| 43 | #include "llvm/Analysis/LoopPass.h" |
Philip Reames | b54c8e6 | 2016-03-09 22:59:30 +0000 | [diff] [blame] | 44 | #include "llvm/Analysis/MemoryBuiltins.h" |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 45 | #include "llvm/Analysis/OptimizationDiagnosticInfo.h" |
Chandler Carruth | abfa3e5 | 2014-01-24 01:59:49 +0000 | [diff] [blame] | 46 | #include "llvm/Analysis/ScalarEvolution.h" |
Chandler Carruth | 7b560d4 | 2015-09-09 17:55:00 +0000 | [diff] [blame] | 47 | #include "llvm/Analysis/ScalarEvolutionAliasAnalysis.h" |
Benjamin Kramer | 799003b | 2015-03-23 19:32:43 +0000 | [diff] [blame] | 48 | #include "llvm/Analysis/TargetLibraryInfo.h" |
Dan Gohman | 75d7d5e | 2011-12-14 23:49:11 +0000 | [diff] [blame] | 49 | #include "llvm/Analysis/ValueTracking.h" |
Chandler Carruth | 1305dc3 | 2014-03-04 11:45:46 +0000 | [diff] [blame] | 50 | #include "llvm/IR/CFG.h" |
Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 51 | #include "llvm/IR/Constants.h" |
| 52 | #include "llvm/IR/DataLayout.h" |
| 53 | #include "llvm/IR/DerivedTypes.h" |
Chandler Carruth | 5ad5f15 | 2014-01-13 09:26:24 +0000 | [diff] [blame] | 54 | #include "llvm/IR/Dominators.h" |
Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 55 | #include "llvm/IR/Instructions.h" |
| 56 | #include "llvm/IR/IntrinsicInst.h" |
| 57 | #include "llvm/IR/LLVMContext.h" |
Chris Lattner | 473988c | 2013-01-05 16:44:07 +0000 | [diff] [blame] | 58 | #include "llvm/IR/Metadata.h" |
Chandler Carruth | aa0ab63 | 2014-03-04 12:09:19 +0000 | [diff] [blame] | 59 | #include "llvm/IR/PredIteratorCache.h" |
Reid Spencer | 7c16caa | 2004-09-01 22:55:40 +0000 | [diff] [blame] | 60 | #include "llvm/Support/CommandLine.h" |
| 61 | #include "llvm/Support/Debug.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 62 | #include "llvm/Support/raw_ostream.h" |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 63 | #include "llvm/Transforms/Scalar.h" |
Chandler Carruth | 3bab7e1 | 2017-01-11 09:43:56 +0000 | [diff] [blame] | 64 | #include "llvm/Transforms/Scalar/LoopPassManager.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 65 | #include "llvm/Transforms/Utils/Local.h" |
Chandler Carruth | 8765cf7 | 2014-01-25 04:07:24 +0000 | [diff] [blame] | 66 | #include "llvm/Transforms/Utils/LoopUtils.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 67 | #include "llvm/Transforms/Utils/SSAUpdater.h" |
Chris Lattner | 6ec05f5 | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 68 | #include <algorithm> |
Benjamin Kramer | 82de7d3 | 2016-05-27 14:27:24 +0000 | [diff] [blame] | 69 | #include <utility> |
Chris Lattner | c051768 | 2003-12-09 17:18:00 +0000 | [diff] [blame] | 70 | using namespace llvm; |
Brian Gaeke | 960707c | 2003-11-11 22:41:34 +0000 | [diff] [blame] | 71 | |
Chandler Carruth | 964daaa | 2014-04-22 02:55:47 +0000 | [diff] [blame] | 72 | #define DEBUG_TYPE "licm" |
| 73 | |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 74 | STATISTIC(NumSunk, "Number of instructions sunk out of loop"); |
| 75 | STATISTIC(NumHoisted, "Number of instructions hoisted out of loop"); |
Chris Lattner | 79a42ac | 2006-12-19 21:40:18 +0000 | [diff] [blame] | 76 | STATISTIC(NumMovedLoads, "Number of load insts hoisted or sunk"); |
| 77 | STATISTIC(NumMovedCalls, "Number of call insts hoisted or sunk"); |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 78 | STATISTIC(NumPromoted, "Number of memory locations promoted to registers"); |
Chris Lattner | 79a42ac | 2006-12-19 21:40:18 +0000 | [diff] [blame] | 79 | |
Dan Gohman | d78c400 | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 80 | static cl::opt<bool> |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 81 | DisablePromotion("disable-licm-promotion", cl::Hidden, |
| 82 | cl::desc("Disable memory promotion in LICM pass")); |
Chris Lattner | 45d67d6 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 83 | |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 84 | static bool inSubLoop(BasicBlock *BB, Loop *CurLoop, LoopInfo *LI); |
David Majnemer | 42a0730 | 2016-01-04 03:37:39 +0000 | [diff] [blame] | 85 | static bool isNotUsedInLoop(const Instruction &I, const Loop *CurLoop, |
Evgeniy Stepanov | 122f984 | 2016-06-10 20:03:17 +0000 | [diff] [blame] | 86 | const LoopSafetyInfo *SafetyInfo); |
Sanjoy Das | 7a2e2be | 2016-01-28 15:51:58 +0000 | [diff] [blame] | 87 | static bool hoist(Instruction &I, const DominatorTree *DT, const Loop *CurLoop, |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 88 | const LoopSafetyInfo *SafetyInfo, |
| 89 | OptimizationRemarkEmitter *ORE); |
Pete Cooper | 0cabcf2 | 2015-05-13 01:12:18 +0000 | [diff] [blame] | 90 | static bool sink(Instruction &I, const LoopInfo *LI, const DominatorTree *DT, |
David Majnemer | 42a0730 | 2016-01-04 03:37:39 +0000 | [diff] [blame] | 91 | const Loop *CurLoop, AliasSetTracker *CurAST, |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 92 | const LoopSafetyInfo *SafetyInfo, |
| 93 | OptimizationRemarkEmitter *ORE); |
Adam Nemet | e2aaf3a | 2017-01-11 04:39:49 +0000 | [diff] [blame] | 94 | static bool isSafeToExecuteUnconditionally(Instruction &Inst, |
Pete Cooper | 0cabcf2 | 2015-05-13 01:12:18 +0000 | [diff] [blame] | 95 | const DominatorTree *DT, |
| 96 | const Loop *CurLoop, |
Evgeniy Stepanov | 122f984 | 2016-06-10 20:03:17 +0000 | [diff] [blame] | 97 | const LoopSafetyInfo *SafetyInfo, |
Adam Nemet | e2aaf3a | 2017-01-11 04:39:49 +0000 | [diff] [blame] | 98 | OptimizationRemarkEmitter *ORE, |
Philip Reames | b47b9c2 | 2015-05-22 02:14:05 +0000 | [diff] [blame] | 99 | const Instruction *CtxI = nullptr); |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 100 | static bool pointerInvalidatedByLoop(Value *V, uint64_t Size, |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 101 | const AAMDNodes &AAInfo, |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 102 | AliasSetTracker *CurAST); |
David Majnemer | 42a0730 | 2016-01-04 03:37:39 +0000 | [diff] [blame] | 103 | static Instruction * |
| 104 | CloneInstructionInExitBlock(Instruction &I, BasicBlock &ExitBlock, PHINode &PN, |
| 105 | const LoopInfo *LI, |
Evgeniy Stepanov | 122f984 | 2016-06-10 20:03:17 +0000 | [diff] [blame] | 106 | const LoopSafetyInfo *SafetyInfo); |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 107 | |
Dan Gohman | d78c400 | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 108 | namespace { |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 109 | struct LoopInvariantCodeMotion { |
| 110 | bool runOnLoop(Loop *L, AliasAnalysis *AA, LoopInfo *LI, DominatorTree *DT, |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 111 | TargetLibraryInfo *TLI, ScalarEvolution *SE, |
| 112 | OptimizationRemarkEmitter *ORE, bool DeleteAST); |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 113 | |
| 114 | DenseMap<Loop *, AliasSetTracker *> &getLoopToAliasSetMap() { |
| 115 | return LoopToAliasSetMap; |
Dehao Chen | 7ef5820 | 2016-07-11 22:45:24 +0000 | [diff] [blame] | 116 | } |
| 117 | |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 118 | private: |
| 119 | DenseMap<Loop *, AliasSetTracker *> LoopToAliasSetMap; |
| 120 | |
| 121 | AliasSetTracker *collectAliasInfoForLoop(Loop *L, LoopInfo *LI, |
| 122 | AliasAnalysis *AA); |
| 123 | }; |
| 124 | |
| 125 | struct LegacyLICMPass : public LoopPass { |
| 126 | static char ID; // Pass identification, replacement for typeid |
| 127 | LegacyLICMPass() : LoopPass(ID) { |
| 128 | initializeLegacyLICMPassPass(*PassRegistry::getPassRegistry()); |
| 129 | } |
| 130 | |
| 131 | bool runOnLoop(Loop *L, LPPassManager &LPM) override { |
Davide Italiano | 34f9438 | 2016-12-23 13:12:50 +0000 | [diff] [blame] | 132 | if (skipLoop(L)) { |
| 133 | // If we have run LICM on a previous loop but now we are skipping |
| 134 | // (because we've hit the opt-bisect limit), we need to clear the |
| 135 | // loop alias information. |
Davide Italiano | b9ff23a | 2016-12-23 15:02:35 +0000 | [diff] [blame] | 136 | for (auto <AS : LICM.getLoopToAliasSetMap()) |
| 137 | delete LTAS.second; |
Davide Italiano | 34f9438 | 2016-12-23 13:12:50 +0000 | [diff] [blame] | 138 | LICM.getLoopToAliasSetMap().clear(); |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 139 | return false; |
Davide Italiano | 34f9438 | 2016-12-23 13:12:50 +0000 | [diff] [blame] | 140 | } |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 141 | |
| 142 | auto *SE = getAnalysisIfAvailable<ScalarEvolutionWrapperPass>(); |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 143 | // For the old PM, we can't use OptimizationRemarkEmitter as an analysis |
| 144 | // pass. Function analyses need to be preserved across loop transformations |
| 145 | // but ORE cannot be preserved (see comment before the pass definition). |
| 146 | OptimizationRemarkEmitter ORE(L->getHeader()->getParent()); |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 147 | return LICM.runOnLoop(L, |
| 148 | &getAnalysis<AAResultsWrapperPass>().getAAResults(), |
| 149 | &getAnalysis<LoopInfoWrapperPass>().getLoopInfo(), |
| 150 | &getAnalysis<DominatorTreeWrapperPass>().getDomTree(), |
| 151 | &getAnalysis<TargetLibraryInfoWrapperPass>().getTLI(), |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 152 | SE ? &SE->getSE() : nullptr, &ORE, false); |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 153 | } |
Chris Lattner | 6ec05f5 | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 154 | |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 155 | /// This transformation requires natural loop information & requires that |
| 156 | /// loop preheaders be inserted into the CFG... |
| 157 | /// |
| 158 | void getAnalysisUsage(AnalysisUsage &AU) const override { |
| 159 | AU.setPreservesCFG(); |
| 160 | AU.addRequired<TargetLibraryInfoWrapperPass>(); |
| 161 | getLoopAnalysisUsage(AU); |
| 162 | } |
Chris Lattner | 6ec05f5 | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 163 | |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 164 | using llvm::Pass::doFinalization; |
Matt Beaumont-Gay | abfc446 | 2012-12-04 05:41:27 +0000 | [diff] [blame] | 165 | |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 166 | bool doFinalization() override { |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 167 | assert(LICM.getLoopToAliasSetMap().empty() && |
| 168 | "Didn't free loop alias sets"); |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 169 | return false; |
| 170 | } |
Devang Patel | 69730c9 | 2007-03-07 04:41:30 +0000 | [diff] [blame] | 171 | |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 172 | private: |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 173 | LoopInvariantCodeMotion LICM; |
Chris Lattner | 6ec05f5 | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 174 | |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 175 | /// cloneBasicBlockAnalysis - Simple Analysis hook. Clone alias set info. |
| 176 | void cloneBasicBlockAnalysis(BasicBlock *From, BasicBlock *To, |
| 177 | Loop *L) override; |
Devang Patel | b98a097 | 2007-07-31 08:01:41 +0000 | [diff] [blame] | 178 | |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 179 | /// deleteAnalysisValue - Simple Analysis hook. Delete value V from alias |
| 180 | /// set. |
| 181 | void deleteAnalysisValue(Value *V, Loop *L) override; |
Devang Patel | b98a097 | 2007-07-31 08:01:41 +0000 | [diff] [blame] | 182 | |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 183 | /// Simple Analysis hook. Delete loop L from alias set map. |
| 184 | void deleteAnalysisLoop(Loop *L) override; |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 185 | }; |
Alexander Kornienko | f00654e | 2015-06-23 09:49:53 +0000 | [diff] [blame] | 186 | } |
Chris Lattner | 6ec05f5 | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 187 | |
Chandler Carruth | 410eaeb | 2017-01-11 06:23:21 +0000 | [diff] [blame] | 188 | PreservedAnalyses LICMPass::run(Loop &L, LoopAnalysisManager &AM, |
| 189 | LoopStandardAnalysisResults &AR, LPMUpdater &) { |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 190 | const auto &FAM = |
Chandler Carruth | 410eaeb | 2017-01-11 06:23:21 +0000 | [diff] [blame] | 191 | AM.getResult<FunctionAnalysisManagerLoopProxy>(L, AR).getManager(); |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 192 | Function *F = L.getHeader()->getParent(); |
| 193 | |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 194 | auto *ORE = FAM.getCachedResult<OptimizationRemarkEmitterAnalysis>(*F); |
Chandler Carruth | 410eaeb | 2017-01-11 06:23:21 +0000 | [diff] [blame] | 195 | // FIXME: This should probably be optional rather than required. |
| 196 | if (!ORE) |
| 197 | report_fatal_error("LICM: OptimizationRemarkEmitterAnalysis not " |
| 198 | "cached at a higher level"); |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 199 | |
| 200 | LoopInvariantCodeMotion LICM; |
Chandler Carruth | 410eaeb | 2017-01-11 06:23:21 +0000 | [diff] [blame] | 201 | if (!LICM.runOnLoop(&L, &AR.AA, &AR.LI, &AR.DT, &AR.TLI, &AR.SE, ORE, true)) |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 202 | return PreservedAnalyses::all(); |
| 203 | |
Chandler Carruth | ca68a3e | 2017-01-15 06:32:49 +0000 | [diff] [blame^] | 204 | auto PA = getLoopPassPreservedAnalyses(); |
| 205 | PA.preserveSet<CFGAnalyses>(); |
| 206 | return PA; |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 207 | } |
| 208 | |
| 209 | char LegacyLICMPass::ID = 0; |
| 210 | INITIALIZE_PASS_BEGIN(LegacyLICMPass, "licm", "Loop Invariant Code Motion", |
| 211 | false, false) |
Chandler Carruth | 31088a9 | 2016-02-19 10:45:18 +0000 | [diff] [blame] | 212 | INITIALIZE_PASS_DEPENDENCY(LoopPass) |
Chandler Carruth | b98f63d | 2015-01-15 10:41:28 +0000 | [diff] [blame] | 213 | INITIALIZE_PASS_DEPENDENCY(TargetLibraryInfoWrapperPass) |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 214 | INITIALIZE_PASS_END(LegacyLICMPass, "licm", "Loop Invariant Code Motion", false, |
| 215 | false) |
Dan Gohman | d78c400 | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 216 | |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 217 | Pass *llvm::createLICMPass() { return new LegacyLICMPass(); } |
Chris Lattner | 6ec05f5 | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 218 | |
Devang Patel | d8b1ceb | 2007-07-31 16:52:25 +0000 | [diff] [blame] | 219 | /// Hoist expressions out of the specified loop. Note, alias info for inner |
Tobias Grosser | a3928f5 | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 220 | /// loop is not preserved so it is not a good idea to run LICM multiple |
Devang Patel | d8b1ceb | 2007-07-31 16:52:25 +0000 | [diff] [blame] | 221 | /// times on one loop. |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 222 | /// We should delete AST for inner loops in the new pass manager to avoid |
| 223 | /// memory leak. |
Chris Lattner | f64f2d3 | 2002-09-26 16:52:07 +0000 | [diff] [blame] | 224 | /// |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 225 | bool LoopInvariantCodeMotion::runOnLoop(Loop *L, AliasAnalysis *AA, |
| 226 | LoopInfo *LI, DominatorTree *DT, |
| 227 | TargetLibraryInfo *TLI, |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 228 | ScalarEvolution *SE, |
| 229 | OptimizationRemarkEmitter *ORE, |
| 230 | bool DeleteAST) { |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 231 | bool Changed = false; |
Chad Rosier | 43a3306 | 2011-12-02 01:26:24 +0000 | [diff] [blame] | 232 | |
Chandler Carruth | fc25854 | 2014-02-11 12:52:27 +0000 | [diff] [blame] | 233 | assert(L->isLCSSAForm(*DT) && "Loop is not in LCSSA form."); |
| 234 | |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 235 | AliasSetTracker *CurAST = collectAliasInfoForLoop(L, LI, AA); |
Chris Lattner | 6ec05f5 | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 236 | |
Chris Lattner | d57f3f5 | 2002-09-26 19:40:25 +0000 | [diff] [blame] | 237 | // Get the preheader block to move instructions into... |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 238 | BasicBlock *Preheader = L->getLoopPreheader(); |
Chris Lattner | d57f3f5 | 2002-09-26 19:40:25 +0000 | [diff] [blame] | 239 | |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 240 | // Compute loop safety information. |
Evgeniy Stepanov | 122f984 | 2016-06-10 20:03:17 +0000 | [diff] [blame] | 241 | LoopSafetyInfo SafetyInfo; |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 242 | computeLoopSafetyInfo(&SafetyInfo, L); |
Nadav Rotem | 03dcd85 | 2012-09-04 10:25:04 +0000 | [diff] [blame] | 243 | |
Chris Lattner | 6ec05f5 | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 244 | // We want to visit all of the instructions in this loop... that are not parts |
| 245 | // of our subloops (they have already had their invariants hoisted out of |
| 246 | // their loop, into this loop, so there is no need to process the BODIES of |
| 247 | // the subloops). |
| 248 | // |
Chris Lattner | 6443769 | 2002-09-29 21:46:09 +0000 | [diff] [blame] | 249 | // Traverse the body of the loop in depth first order on the dominator tree so |
| 250 | // that we are guaranteed to see definitions before we see uses. This allows |
Nick Lewycky | a0d49da | 2007-08-18 15:08:56 +0000 | [diff] [blame] | 251 | // us to sink instructions in one pass, without iteration. After sinking |
Chris Lattner | 547192d6 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 252 | // instructions, we perform another pass to hoist them out of the loop. |
Chris Lattner | 6443769 | 2002-09-29 21:46:09 +0000 | [diff] [blame] | 253 | // |
Dan Gohman | a83ac2d | 2009-11-05 21:11:53 +0000 | [diff] [blame] | 254 | if (L->hasDedicatedExits()) |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 255 | Changed |= sinkRegion(DT->getNode(L->getHeader()), AA, LI, DT, TLI, L, |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 256 | CurAST, &SafetyInfo, ORE); |
Dan Gohman | a83ac2d | 2009-11-05 21:11:53 +0000 | [diff] [blame] | 257 | if (Preheader) |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 258 | Changed |= hoistRegion(DT->getNode(L->getHeader()), AA, LI, DT, TLI, L, |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 259 | CurAST, &SafetyInfo, ORE); |
Chris Lattner | 6ec05f5 | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 260 | |
Chris Lattner | 45d67d6 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 261 | // Now that all loop invariants have been removed from the loop, promote any |
Chris Lattner | 1dc98b4 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 262 | // memory references to scalars that we can. |
Michael Kuperstein | 5566092 | 2016-12-29 22:51:22 +0000 | [diff] [blame] | 263 | // Don't sink stores from loops without dedicated block exits. Exits |
| 264 | // containing indirect branches are not transformed by loop simplify, |
| 265 | // make sure we catch that. An additional load may be generated in the |
| 266 | // preheader for SSA updater, so also avoid sinking when no preheader |
| 267 | // is available. |
| 268 | if (!DisablePromotion && Preheader && L->hasDedicatedExits()) { |
Michael Kuperstein | ff36bae | 2016-12-29 23:11:19 +0000 | [diff] [blame] | 269 | // Figure out the loop exits and their insertion points |
Dan Gohman | b948736 | 2012-08-08 00:00:26 +0000 | [diff] [blame] | 270 | SmallVector<BasicBlock *, 8> ExitBlocks; |
Michael Kuperstein | ff36bae | 2016-12-29 23:11:19 +0000 | [diff] [blame] | 271 | L->getUniqueExitBlocks(ExitBlocks); |
Dan Gohman | b948736 | 2012-08-08 00:00:26 +0000 | [diff] [blame] | 272 | |
Michael Kuperstein | ff36bae | 2016-12-29 23:11:19 +0000 | [diff] [blame] | 273 | // We can't insert into a catchswitch. |
| 274 | bool HasCatchSwitch = llvm::any_of(ExitBlocks, [](BasicBlock *Exit) { |
| 275 | return isa<CatchSwitchInst>(Exit->getTerminator()); |
| 276 | }); |
Michael Kuperstein | b6da9cf | 2016-12-29 22:37:13 +0000 | [diff] [blame] | 277 | |
Michael Kuperstein | ff36bae | 2016-12-29 23:11:19 +0000 | [diff] [blame] | 278 | if (!HasCatchSwitch) { |
| 279 | SmallVector<Instruction *, 8> InsertPts; |
| 280 | InsertPts.reserve(ExitBlocks.size()); |
| 281 | for (BasicBlock *ExitBlock : ExitBlocks) |
| 282 | InsertPts.push_back(&*ExitBlock->getFirstInsertionPt()); |
Chandler Carruth | 1665152 | 2014-02-01 13:35:14 +0000 | [diff] [blame] | 283 | |
Michael Kuperstein | ff36bae | 2016-12-29 23:11:19 +0000 | [diff] [blame] | 284 | PredIteratorCache PIC; |
Michael Kuperstein | b6da9cf | 2016-12-29 22:37:13 +0000 | [diff] [blame] | 285 | |
Michael Kuperstein | ff36bae | 2016-12-29 23:11:19 +0000 | [diff] [blame] | 286 | bool Promoted = false; |
| 287 | |
| 288 | // Loop over all of the alias sets in the tracker object. |
| 289 | for (AliasSet &AS : *CurAST) |
| 290 | Promoted |= |
| 291 | promoteLoopAccessesToScalars(AS, ExitBlocks, InsertPts, PIC, LI, DT, |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 292 | TLI, L, CurAST, &SafetyInfo, ORE); |
Michael Kuperstein | ff36bae | 2016-12-29 23:11:19 +0000 | [diff] [blame] | 293 | |
| 294 | // Once we have promoted values across the loop body we have to |
| 295 | // recursively reform LCSSA as any nested loop may now have values defined |
| 296 | // within the loop used in the outer loop. |
| 297 | // FIXME: This is really heavy handed. It would be a bit better to use an |
| 298 | // SSAUpdater strategy during promotion that was LCSSA aware and reformed |
| 299 | // it as it went. |
| 300 | if (Promoted) |
| 301 | formLCSSARecursively(*L, *DT, LI, SE); |
| 302 | |
| 303 | Changed |= Promoted; |
| 304 | } |
Chris Lattner | 1dc98b4 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 305 | } |
Tobias Grosser | a3928f5 | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 306 | |
Chandler Carruth | fc25854 | 2014-02-11 12:52:27 +0000 | [diff] [blame] | 307 | // Check that neither this loop nor its parent have had LCSSA broken. LICM is |
| 308 | // specifically moving instructions across the loop boundary and so it is |
| 309 | // especially in need of sanity checking here. |
| 310 | assert(L->isLCSSAForm(*DT) && "Loop not left in LCSSA form after LICM!"); |
| 311 | assert((!L->getParentLoop() || L->getParentLoop()->isLCSSAForm(*DT)) && |
| 312 | "Parent loop not left in LCSSA form after LICM!"); |
Chandler Carruth | 8765cf7 | 2014-01-25 04:07:24 +0000 | [diff] [blame] | 313 | |
Chris Lattner | cc9cbc6 | 2010-08-29 17:46:00 +0000 | [diff] [blame] | 314 | // If this loop is nested inside of another one, save the alias information |
| 315 | // for when we process the outer loop. |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 316 | if (L->getParentLoop() && !DeleteAST) |
Chris Lattner | cc9cbc6 | 2010-08-29 17:46:00 +0000 | [diff] [blame] | 317 | LoopToAliasSetMap[L] = CurAST; |
| 318 | else |
| 319 | delete CurAST; |
Sanjoy Das | 4ae3920 | 2016-05-03 17:50:11 +0000 | [diff] [blame] | 320 | |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 321 | if (Changed && SE) |
| 322 | SE->forgetLoopDispositions(L); |
Devang Patel | 69730c9 | 2007-03-07 04:41:30 +0000 | [diff] [blame] | 323 | return Changed; |
Chris Lattner | 6ec05f5 | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 324 | } |
| 325 | |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 326 | /// Walk the specified region of the CFG (defined by all blocks dominated by |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 327 | /// the specified block, and that are in the current loop) in reverse depth |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 328 | /// first order w.r.t the DominatorTree. This allows us to visit uses before |
| 329 | /// definitions, allowing us to sink a loop body in one pass without iteration. |
Chris Lattner | 547192d6 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 330 | /// |
Mehdi Amini | a28d91d | 2015-03-10 02:37:25 +0000 | [diff] [blame] | 331 | bool llvm::sinkRegion(DomTreeNode *N, AliasAnalysis *AA, LoopInfo *LI, |
| 332 | DominatorTree *DT, TargetLibraryInfo *TLI, Loop *CurLoop, |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 333 | AliasSetTracker *CurAST, LoopSafetyInfo *SafetyInfo, |
| 334 | OptimizationRemarkEmitter *ORE) { |
Chris Lattner | 547192d6 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 335 | |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 336 | // Verify inputs. |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 337 | assert(N != nullptr && AA != nullptr && LI != nullptr && DT != nullptr && |
| 338 | CurLoop != nullptr && CurAST != nullptr && SafetyInfo != nullptr && |
| 339 | "Unexpected input to sinkRegion"); |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 340 | |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 341 | BasicBlock *BB = N->getBlock(); |
Chris Lattner | 547192d6 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 342 | // If this subregion is not in the top level loop at all, exit. |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 343 | if (!CurLoop->contains(BB)) |
| 344 | return false; |
Chris Lattner | 547192d6 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 345 | |
Chris Lattner | 263f804 | 2010-08-29 18:22:25 +0000 | [diff] [blame] | 346 | // We are processing blocks in reverse dfo, so process children first. |
Sanjay Patel | 9913322 | 2016-01-13 23:01:57 +0000 | [diff] [blame] | 347 | bool Changed = false; |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 348 | const std::vector<DomTreeNode *> &Children = N->getChildren(); |
Sanjay Patel | 9f088ab | 2016-01-08 22:59:42 +0000 | [diff] [blame] | 349 | for (DomTreeNode *Child : Children) |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 350 | Changed |= |
| 351 | sinkRegion(Child, AA, LI, DT, TLI, CurLoop, CurAST, SafetyInfo, ORE); |
Sanjay Patel | 9f088ab | 2016-01-08 22:59:42 +0000 | [diff] [blame] | 352 | |
Chris Lattner | 547192d6 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 353 | // Only need to process the contents of this block if it is not part of a |
| 354 | // subloop (which would already have been processed). |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 355 | if (inSubLoop(BB, CurLoop, LI)) |
| 356 | return Changed; |
Chris Lattner | 547192d6 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 357 | |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 358 | for (BasicBlock::iterator II = BB->end(); II != BB->begin();) { |
Chris Lattner | 9184601 | 2003-12-19 08:18:16 +0000 | [diff] [blame] | 359 | Instruction &I = *--II; |
Tobias Grosser | a3928f5 | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 360 | |
Chris Lattner | 263f804 | 2010-08-29 18:22:25 +0000 | [diff] [blame] | 361 | // If the instruction is dead, we would try to sink it because it isn't used |
| 362 | // in the loop, instead, just delete it. |
Benjamin Kramer | 8bcc971 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 363 | if (isInstructionTriviallyDead(&I, TLI)) { |
Chris Lattner | f58382e | 2010-08-29 18:42:23 +0000 | [diff] [blame] | 364 | DEBUG(dbgs() << "LICM deleting dead inst: " << I << '\n'); |
Chris Lattner | 263f804 | 2010-08-29 18:22:25 +0000 | [diff] [blame] | 365 | ++II; |
| 366 | CurAST->deleteValue(&I); |
| 367 | I.eraseFromParent(); |
| 368 | Changed = true; |
| 369 | continue; |
| 370 | } |
Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 371 | |
Chris Lattner | 547192d6 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 372 | // Check to see if we can sink this instruction to the exit blocks |
| 373 | // of the loop. We can do this if the all users of the instruction are |
| 374 | // outside of the loop. In this case, it doesn't even matter if the |
| 375 | // operands of the instruction are loop invariant. |
| 376 | // |
David Majnemer | 42a0730 | 2016-01-04 03:37:39 +0000 | [diff] [blame] | 377 | if (isNotUsedInLoop(I, CurLoop, SafetyInfo) && |
Adam Nemet | 81941b3 | 2017-01-11 04:39:45 +0000 | [diff] [blame] | 378 | canSinkOrHoistInst(I, AA, DT, CurLoop, CurAST, SafetyInfo, ORE)) { |
Chris Lattner | 9184601 | 2003-12-19 08:18:16 +0000 | [diff] [blame] | 379 | ++II; |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 380 | Changed |= sink(I, LI, DT, CurLoop, CurAST, SafetyInfo, ORE); |
Chris Lattner | 9184601 | 2003-12-19 08:18:16 +0000 | [diff] [blame] | 381 | } |
Chris Lattner | 547192d6 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 382 | } |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 383 | return Changed; |
Chris Lattner | 547192d6 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 384 | } |
| 385 | |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 386 | /// Walk the specified region of the CFG (defined by all blocks dominated by |
| 387 | /// the specified block, and that are in the current loop) in depth first |
| 388 | /// order w.r.t the DominatorTree. This allows us to visit definitions before |
| 389 | /// uses, allowing us to hoist a loop body in one pass without iteration. |
Chris Lattner | 6443769 | 2002-09-29 21:46:09 +0000 | [diff] [blame] | 390 | /// |
Mehdi Amini | a28d91d | 2015-03-10 02:37:25 +0000 | [diff] [blame] | 391 | bool llvm::hoistRegion(DomTreeNode *N, AliasAnalysis *AA, LoopInfo *LI, |
| 392 | DominatorTree *DT, TargetLibraryInfo *TLI, Loop *CurLoop, |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 393 | AliasSetTracker *CurAST, LoopSafetyInfo *SafetyInfo, |
| 394 | OptimizationRemarkEmitter *ORE) { |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 395 | // Verify inputs. |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 396 | assert(N != nullptr && AA != nullptr && LI != nullptr && DT != nullptr && |
| 397 | CurLoop != nullptr && CurAST != nullptr && SafetyInfo != nullptr && |
| 398 | "Unexpected input to hoistRegion"); |
Sanjay Patel | 9913322 | 2016-01-13 23:01:57 +0000 | [diff] [blame] | 399 | |
Owen Anderson | c24701e | 2007-04-24 06:40:39 +0000 | [diff] [blame] | 400 | BasicBlock *BB = N->getBlock(); |
Sanjay Patel | 9913322 | 2016-01-13 23:01:57 +0000 | [diff] [blame] | 401 | |
Chris Lattner | 05e8630 | 2002-09-29 22:26:07 +0000 | [diff] [blame] | 402 | // If this subregion is not in the top level loop at all, exit. |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 403 | if (!CurLoop->contains(BB)) |
| 404 | return false; |
Sanjay Patel | 9913322 | 2016-01-13 23:01:57 +0000 | [diff] [blame] | 405 | |
Chris Lattner | aaaea51 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 406 | // Only need to process the contents of this block if it is not part of a |
| 407 | // subloop (which would already have been processed). |
Sanjay Patel | 9913322 | 2016-01-13 23:01:57 +0000 | [diff] [blame] | 408 | bool Changed = false; |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 409 | if (!inSubLoop(BB, CurLoop, LI)) |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 410 | for (BasicBlock::iterator II = BB->begin(), E = BB->end(); II != E;) { |
Chris Lattner | aaaea51 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 411 | Instruction &I = *II++; |
Chris Lattner | 030f020 | 2010-08-31 23:00:16 +0000 | [diff] [blame] | 412 | // Try constant folding this instruction. If all the operands are |
| 413 | // constants, it is technically hoistable, but it would be better to just |
| 414 | // fold it. |
Mehdi Amini | a28d91d | 2015-03-10 02:37:25 +0000 | [diff] [blame] | 415 | if (Constant *C = ConstantFoldInstruction( |
| 416 | &I, I.getModule()->getDataLayout(), TLI)) { |
Chris Lattner | 030f020 | 2010-08-31 23:00:16 +0000 | [diff] [blame] | 417 | DEBUG(dbgs() << "LICM folding inst: " << I << " --> " << *C << '\n'); |
| 418 | CurAST->copyValue(&I, C); |
Chris Lattner | 030f020 | 2010-08-31 23:00:16 +0000 | [diff] [blame] | 419 | I.replaceAllUsesWith(C); |
David Majnemer | 522a911 | 2016-07-22 04:54:44 +0000 | [diff] [blame] | 420 | if (isInstructionTriviallyDead(&I, TLI)) { |
| 421 | CurAST->deleteValue(&I); |
| 422 | I.eraseFromParent(); |
| 423 | } |
Andrew Kaylor | 7353cf4 | 2017-01-05 18:53:24 +0000 | [diff] [blame] | 424 | Changed = true; |
Chris Lattner | 030f020 | 2010-08-31 23:00:16 +0000 | [diff] [blame] | 425 | continue; |
| 426 | } |
Tobias Grosser | a3928f5 | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 427 | |
Chris Lattner | 547192d6 | 2003-12-19 07:22:45 +0000 | [diff] [blame] | 428 | // Try hoisting the instruction out to the preheader. We can only do this |
| 429 | // if all of the operands of the instruction are loop invariant and if it |
| 430 | // is safe to hoist the instruction. |
| 431 | // |
Mehdi Amini | a28d91d | 2015-03-10 02:37:25 +0000 | [diff] [blame] | 432 | if (CurLoop->hasLoopInvariantOperands(&I) && |
Adam Nemet | 81941b3 | 2017-01-11 04:39:45 +0000 | [diff] [blame] | 433 | canSinkOrHoistInst(I, AA, DT, CurLoop, CurAST, SafetyInfo, ORE) && |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 434 | isSafeToExecuteUnconditionally( |
Adam Nemet | e2aaf3a | 2017-01-11 04:39:49 +0000 | [diff] [blame] | 435 | I, DT, CurLoop, SafetyInfo, ORE, |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 436 | CurLoop->getLoopPreheader()->getTerminator())) |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 437 | Changed |= hoist(I, DT, CurLoop, SafetyInfo, ORE); |
Chris Lattner | 030f020 | 2010-08-31 23:00:16 +0000 | [diff] [blame] | 438 | } |
Chris Lattner | 6443769 | 2002-09-29 21:46:09 +0000 | [diff] [blame] | 439 | |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 440 | const std::vector<DomTreeNode *> &Children = N->getChildren(); |
Sanjay Patel | 9f088ab | 2016-01-08 22:59:42 +0000 | [diff] [blame] | 441 | for (DomTreeNode *Child : Children) |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 442 | Changed |= |
| 443 | hoistRegion(Child, AA, LI, DT, TLI, CurLoop, CurAST, SafetyInfo, ORE); |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 444 | return Changed; |
| 445 | } |
| 446 | |
| 447 | /// Computes loop safety information, checks loop body & header |
Benjamin Kramer | df005cb | 2015-08-08 18:27:36 +0000 | [diff] [blame] | 448 | /// for the possibility of may throw exception. |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 449 | /// |
Evgeniy Stepanov | 122f984 | 2016-06-10 20:03:17 +0000 | [diff] [blame] | 450 | void llvm::computeLoopSafetyInfo(LoopSafetyInfo *SafetyInfo, Loop *CurLoop) { |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 451 | assert(CurLoop != nullptr && "CurLoop cant be null"); |
| 452 | BasicBlock *Header = CurLoop->getHeader(); |
| 453 | // Setting default safety values. |
| 454 | SafetyInfo->MayThrow = false; |
| 455 | SafetyInfo->HeaderMayThrow = false; |
Benjamin Kramer | df005cb | 2015-08-08 18:27:36 +0000 | [diff] [blame] | 456 | // Iterate over header and compute safety info. |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 457 | for (BasicBlock::iterator I = Header->begin(), E = Header->end(); |
| 458 | (I != E) && !SafetyInfo->HeaderMayThrow; ++I) |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 459 | SafetyInfo->HeaderMayThrow |= |
| 460 | !isGuaranteedToTransferExecutionToSuccessor(&*I); |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 461 | |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 462 | SafetyInfo->MayThrow = SafetyInfo->HeaderMayThrow; |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 463 | // Iterate over loop instructions and compute safety info. |
| 464 | for (Loop::block_iterator BB = CurLoop->block_begin(), |
| 465 | BBE = CurLoop->block_end(); |
| 466 | (BB != BBE) && !SafetyInfo->MayThrow; ++BB) |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 467 | for (BasicBlock::iterator I = (*BB)->begin(), E = (*BB)->end(); |
| 468 | (I != E) && !SafetyInfo->MayThrow; ++I) |
Eli Friedman | f1da33e | 2016-06-11 21:48:25 +0000 | [diff] [blame] | 469 | SafetyInfo->MayThrow |= !isGuaranteedToTransferExecutionToSuccessor(&*I); |
David Majnemer | 42a0730 | 2016-01-04 03:37:39 +0000 | [diff] [blame] | 470 | |
| 471 | // Compute funclet colors if we might sink/hoist in a function with a funclet |
| 472 | // personality routine. |
| 473 | Function *Fn = CurLoop->getHeader()->getParent(); |
| 474 | if (Fn->hasPersonalityFn()) |
| 475 | if (Constant *PersonalityFn = Fn->getPersonalityFn()) |
| 476 | if (isFuncletEHPersonality(classifyEHPersonality(PersonalityFn))) |
| 477 | SafetyInfo->BlockColors = colorEHFunclets(*Fn); |
Chris Lattner | 6443769 | 2002-09-29 21:46:09 +0000 | [diff] [blame] | 478 | } |
| 479 | |
Dehao Chen | b94c09ba | 2016-10-27 16:30:08 +0000 | [diff] [blame] | 480 | bool llvm::canSinkOrHoistInst(Instruction &I, AAResults *AA, DominatorTree *DT, |
| 481 | Loop *CurLoop, AliasSetTracker *CurAST, |
Adam Nemet | 81941b3 | 2017-01-11 04:39:45 +0000 | [diff] [blame] | 482 | LoopSafetyInfo *SafetyInfo, |
| 483 | OptimizationRemarkEmitter *ORE) { |
Chris Lattner | 65c1193 | 2003-12-09 19:32:44 +0000 | [diff] [blame] | 484 | // Loads have extra constraints we have to verify before we can hoist them. |
| 485 | if (LoadInst *LI = dyn_cast<LoadInst>(&I)) { |
Eli Friedman | 91386c7 | 2011-08-15 20:52:09 +0000 | [diff] [blame] | 486 | if (!LI->isUnordered()) |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 487 | return false; // Don't hoist volatile/atomic loads! |
Chris Lattner | 65c1193 | 2003-12-09 19:32:44 +0000 | [diff] [blame] | 488 | |
Chris Lattner | 8a8fb90 | 2008-07-23 05:06:28 +0000 | [diff] [blame] | 489 | // Loads from constant memory are always safe to move, even if they end up |
| 490 | // in the same alias set as something that ends up being modified. |
Dan Gohman | cbc6ebb | 2009-11-19 19:00:10 +0000 | [diff] [blame] | 491 | if (AA->pointsToConstantMemory(LI->getOperand(0))) |
Chris Lattner | 8a8fb90 | 2008-07-23 05:06:28 +0000 | [diff] [blame] | 492 | return true; |
Philip Reames | 5a3f5f7 | 2014-10-21 00:13:20 +0000 | [diff] [blame] | 493 | if (LI->getMetadata(LLVMContext::MD_invariant_load)) |
Pete Cooper | 9ee2209 | 2011-11-08 19:30:00 +0000 | [diff] [blame] | 494 | return true; |
Tobias Grosser | a3928f5 | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 495 | |
Chris Lattner | 65c1193 | 2003-12-09 19:32:44 +0000 | [diff] [blame] | 496 | // Don't hoist loads which have may-aliased stores in loop. |
Dan Gohman | f372cf8 | 2010-10-19 22:54:46 +0000 | [diff] [blame] | 497 | uint64_t Size = 0; |
Chris Lattner | b137409 | 2004-11-26 21:20:09 +0000 | [diff] [blame] | 498 | if (LI->getType()->isSized()) |
Dehao Chen | 4b5e7f7 | 2016-09-02 01:59:27 +0000 | [diff] [blame] | 499 | Size = I.getModule()->getDataLayout().getTypeStoreSize(LI->getType()); |
Hal Finkel | cc39b67 | 2014-07-24 12:16:19 +0000 | [diff] [blame] | 500 | |
| 501 | AAMDNodes AAInfo; |
| 502 | LI->getAAMetadata(AAInfo); |
| 503 | |
Adam Nemet | 81941b3 | 2017-01-11 04:39:45 +0000 | [diff] [blame] | 504 | bool Invalidated = |
| 505 | pointerInvalidatedByLoop(LI->getOperand(0), Size, AAInfo, CurAST); |
| 506 | // Check loop-invariant address because this may also be a sinkable load |
| 507 | // whose address is not necessarily loop-invariant. |
| 508 | if (ORE && Invalidated && CurLoop->isLoopInvariant(LI->getPointerOperand())) |
| 509 | ORE->emit(OptimizationRemarkMissed( |
| 510 | DEBUG_TYPE, "LoadWithLoopInvariantAddressInvalidated", LI) |
| 511 | << "failed to move load with loop-invariant address " |
| 512 | "because the loop may invalidate its value"); |
| 513 | |
| 514 | return !Invalidated; |
Chris Lattner | 20cda26 | 2004-03-15 04:11:30 +0000 | [diff] [blame] | 515 | } else if (CallInst *CI = dyn_cast<CallInst>(&I)) { |
Eli Friedman | 942e1c1 | 2011-05-27 18:37:52 +0000 | [diff] [blame] | 516 | // Don't sink or hoist dbg info; it's legal, but not useful. |
Dehao Chen | 4b5e7f7 | 2016-09-02 01:59:27 +0000 | [diff] [blame] | 517 | if (isa<DbgInfoIntrinsic>(I)) |
Eli Friedman | 942e1c1 | 2011-05-27 18:37:52 +0000 | [diff] [blame] | 518 | return false; |
| 519 | |
David Majnemer | 42a0730 | 2016-01-04 03:37:39 +0000 | [diff] [blame] | 520 | // Don't sink calls which can throw. |
| 521 | if (CI->mayThrow()) |
| 522 | return false; |
| 523 | |
Eli Friedman | 942e1c1 | 2011-05-27 18:37:52 +0000 | [diff] [blame] | 524 | // Handle simple cases by querying alias analysis. |
Chandler Carruth | 194f59c | 2015-07-22 23:15:57 +0000 | [diff] [blame] | 525 | FunctionModRefBehavior Behavior = AA->getModRefBehavior(CI); |
| 526 | if (Behavior == FMRB_DoesNotAccessMemory) |
Duncan Sands | 68b6f50 | 2007-12-01 07:51:45 +0000 | [diff] [blame] | 527 | return true; |
Dan Gohman | 0f17507 | 2010-11-09 19:58:21 +0000 | [diff] [blame] | 528 | if (AliasAnalysis::onlyReadsMemory(Behavior)) { |
Philip Reames | 5f99423 | 2015-09-21 22:27:59 +0000 | [diff] [blame] | 529 | // A readonly argmemonly function only reads from memory pointed to by |
| 530 | // it's arguments with arbitrary offsets. If we can prove there are no |
| 531 | // writes to this memory in the loop, we can hoist or sink. |
| 532 | if (AliasAnalysis::onlyAccessesArgPointees(Behavior)) { |
| 533 | for (Value *Op : CI->arg_operands()) |
| 534 | if (Op->getType()->isPointerTy() && |
| 535 | pointerInvalidatedByLoop(Op, MemoryLocation::UnknownSize, |
| 536 | AAMDNodes(), CurAST)) |
| 537 | return false; |
| 538 | return true; |
| 539 | } |
Duncan Sands | 68b6f50 | 2007-12-01 07:51:45 +0000 | [diff] [blame] | 540 | // If this call only reads from memory and there are no writes to memory |
| 541 | // in the loop, we can hoist or sink the call as appropriate. |
| 542 | bool FoundMod = false; |
Sanjay Patel | 9f088ab | 2016-01-08 22:59:42 +0000 | [diff] [blame] | 543 | for (AliasSet &AS : *CurAST) { |
Duncan Sands | 68b6f50 | 2007-12-01 07:51:45 +0000 | [diff] [blame] | 544 | if (!AS.isForwardingAliasSet() && AS.isMod()) { |
| 545 | FoundMod = true; |
| 546 | break; |
Chris Lattner | 20cda26 | 2004-03-15 04:11:30 +0000 | [diff] [blame] | 547 | } |
Chris Lattner | 20cda26 | 2004-03-15 04:11:30 +0000 | [diff] [blame] | 548 | } |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 549 | if (!FoundMod) |
| 550 | return true; |
Chris Lattner | 20cda26 | 2004-03-15 04:11:30 +0000 | [diff] [blame] | 551 | } |
| 552 | |
Nadav Rotem | 03dcd85 | 2012-09-04 10:25:04 +0000 | [diff] [blame] | 553 | // FIXME: This should use mod/ref information to see if we can hoist or |
| 554 | // sink the call. |
Dehao Chen | 4b5e7f7 | 2016-09-02 01:59:27 +0000 | [diff] [blame] | 555 | |
Chris Lattner | 20cda26 | 2004-03-15 04:11:30 +0000 | [diff] [blame] | 556 | return false; |
Chris Lattner | 65c1193 | 2003-12-09 19:32:44 +0000 | [diff] [blame] | 557 | } |
| 558 | |
Dehao Chen | 4b5e7f7 | 2016-09-02 01:59:27 +0000 | [diff] [blame] | 559 | // Only these instructions are hoistable/sinkable. |
| 560 | if (!isa<BinaryOperator>(I) && !isa<CastInst>(I) && !isa<SelectInst>(I) && |
| 561 | !isa<GetElementPtrInst>(I) && !isa<CmpInst>(I) && |
| 562 | !isa<InsertElementInst>(I) && !isa<ExtractElementInst>(I) && |
| 563 | !isa<ShuffleVectorInst>(I) && !isa<ExtractValueInst>(I) && |
| 564 | !isa<InsertValueInst>(I)) |
| 565 | return false; |
| 566 | |
Dehao Chen | 92abc7e | 2016-10-03 18:52:08 +0000 | [diff] [blame] | 567 | // SafetyInfo is nullptr if we are checking for sinking from preheader to |
| 568 | // loop body. It will be always safe as there is no speculative execution. |
| 569 | if (!SafetyInfo) |
| 570 | return true; |
| 571 | |
Dehao Chen | 4b5e7f7 | 2016-09-02 01:59:27 +0000 | [diff] [blame] | 572 | // TODO: Plumb the context instruction through to make hoisting and sinking |
| 573 | // more powerful. Hoisting of loads already works due to the special casing |
| 574 | // above. |
| 575 | return isSafeToExecuteUnconditionally(I, DT, CurLoop, SafetyInfo, nullptr); |
Chris Lattner | aaaea51 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 576 | } |
| 577 | |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 578 | /// Returns true if a PHINode is a trivially replaceable with an |
Chandler Carruth | 8765cf7 | 2014-01-25 04:07:24 +0000 | [diff] [blame] | 579 | /// Instruction. |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 580 | /// This is true when all incoming values are that instruction. |
| 581 | /// This pattern occurs most often with LCSSA PHI nodes. |
Chandler Carruth | 8765cf7 | 2014-01-25 04:07:24 +0000 | [diff] [blame] | 582 | /// |
Pete Cooper | 47e80cd | 2015-05-12 20:05:20 +0000 | [diff] [blame] | 583 | static bool isTriviallyReplacablePHI(const PHINode &PN, const Instruction &I) { |
Pete Cooper | 833f34d | 2015-05-12 20:05:31 +0000 | [diff] [blame] | 584 | for (const Value *IncValue : PN.incoming_values()) |
| 585 | if (IncValue != &I) |
Chandler Carruth | 8765cf7 | 2014-01-25 04:07:24 +0000 | [diff] [blame] | 586 | return false; |
| 587 | |
| 588 | return true; |
| 589 | } |
| 590 | |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 591 | /// Return true if the only users of this instruction are outside of |
| 592 | /// the loop. If this is true, we can sink the instruction to the exit |
| 593 | /// blocks of the loop. |
Chris Lattner | aaaea51 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 594 | /// |
David Majnemer | 42a0730 | 2016-01-04 03:37:39 +0000 | [diff] [blame] | 595 | static bool isNotUsedInLoop(const Instruction &I, const Loop *CurLoop, |
Evgeniy Stepanov | 122f984 | 2016-06-10 20:03:17 +0000 | [diff] [blame] | 596 | const LoopSafetyInfo *SafetyInfo) { |
David Majnemer | 42a0730 | 2016-01-04 03:37:39 +0000 | [diff] [blame] | 597 | const auto &BlockColors = SafetyInfo->BlockColors; |
Pete Cooper | 0cabcf2 | 2015-05-13 01:12:18 +0000 | [diff] [blame] | 598 | for (const User *U : I.users()) { |
| 599 | const Instruction *UI = cast<Instruction>(U); |
| 600 | if (const PHINode *PN = dyn_cast<PHINode>(UI)) { |
David Majnemer | 42a0730 | 2016-01-04 03:37:39 +0000 | [diff] [blame] | 601 | const BasicBlock *BB = PN->getParent(); |
| 602 | // We cannot sink uses in catchswitches. |
| 603 | if (isa<CatchSwitchInst>(BB->getTerminator())) |
| 604 | return false; |
| 605 | |
| 606 | // We need to sink a callsite to a unique funclet. Avoid sinking if the |
| 607 | // phi use is too muddled. |
| 608 | if (isa<CallInst>(I)) |
| 609 | if (!BlockColors.empty() && |
| 610 | BlockColors.find(const_cast<BasicBlock *>(BB))->second.size() != 1) |
| 611 | return false; |
| 612 | |
Chandler Carruth | 8765cf7 | 2014-01-25 04:07:24 +0000 | [diff] [blame] | 613 | // A PHI node where all of the incoming values are this instruction are |
| 614 | // special -- they can just be RAUW'ed with the instruction and thus |
| 615 | // don't require a use in the predecessor. This is a particular important |
| 616 | // special case because it is the pattern found in LCSSA form. |
| 617 | if (isTriviallyReplacablePHI(*PN, I)) { |
| 618 | if (CurLoop->contains(PN)) |
| 619 | return false; |
| 620 | else |
| 621 | continue; |
| 622 | } |
| 623 | |
| 624 | // Otherwise, PHI node uses occur in predecessor blocks if the incoming |
| 625 | // values. Check for such a use being inside the loop. |
Chris Lattner | 34399dd | 2003-12-11 22:23:32 +0000 | [diff] [blame] | 626 | for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) |
| 627 | if (PN->getIncomingValue(i) == &I) |
| 628 | if (CurLoop->contains(PN->getIncomingBlock(i))) |
| 629 | return false; |
Chandler Carruth | 8765cf7 | 2014-01-25 04:07:24 +0000 | [diff] [blame] | 630 | |
| 631 | continue; |
Chris Lattner | 34399dd | 2003-12-11 22:23:32 +0000 | [diff] [blame] | 632 | } |
Chandler Carruth | 8765cf7 | 2014-01-25 04:07:24 +0000 | [diff] [blame] | 633 | |
Chandler Carruth | cdf4788 | 2014-03-09 03:16:01 +0000 | [diff] [blame] | 634 | if (CurLoop->contains(UI)) |
Chandler Carruth | 8765cf7 | 2014-01-25 04:07:24 +0000 | [diff] [blame] | 635 | return false; |
Chris Lattner | 34399dd | 2003-12-11 22:23:32 +0000 | [diff] [blame] | 636 | } |
Chris Lattner | aaaea51 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 637 | return true; |
| 638 | } |
| 639 | |
David Majnemer | 42a0730 | 2016-01-04 03:37:39 +0000 | [diff] [blame] | 640 | static Instruction * |
| 641 | CloneInstructionInExitBlock(Instruction &I, BasicBlock &ExitBlock, PHINode &PN, |
| 642 | const LoopInfo *LI, |
Evgeniy Stepanov | 122f984 | 2016-06-10 20:03:17 +0000 | [diff] [blame] | 643 | const LoopSafetyInfo *SafetyInfo) { |
David Majnemer | 42a0730 | 2016-01-04 03:37:39 +0000 | [diff] [blame] | 644 | Instruction *New; |
| 645 | if (auto *CI = dyn_cast<CallInst>(&I)) { |
| 646 | const auto &BlockColors = SafetyInfo->BlockColors; |
| 647 | |
| 648 | // Sinking call-sites need to be handled differently from other |
| 649 | // instructions. The cloned call-site needs a funclet bundle operand |
| 650 | // appropriate for it's location in the CFG. |
| 651 | SmallVector<OperandBundleDef, 1> OpBundles; |
| 652 | for (unsigned BundleIdx = 0, BundleEnd = CI->getNumOperandBundles(); |
| 653 | BundleIdx != BundleEnd; ++BundleIdx) { |
| 654 | OperandBundleUse Bundle = CI->getOperandBundleAt(BundleIdx); |
| 655 | if (Bundle.getTagID() == LLVMContext::OB_funclet) |
| 656 | continue; |
| 657 | |
| 658 | OpBundles.emplace_back(Bundle); |
| 659 | } |
| 660 | |
| 661 | if (!BlockColors.empty()) { |
| 662 | const ColorVector &CV = BlockColors.find(&ExitBlock)->second; |
| 663 | assert(CV.size() == 1 && "non-unique color for exit block!"); |
| 664 | BasicBlock *BBColor = CV.front(); |
| 665 | Instruction *EHPad = BBColor->getFirstNonPHI(); |
| 666 | if (EHPad->isEHPad()) |
| 667 | OpBundles.emplace_back("funclet", EHPad); |
| 668 | } |
| 669 | |
| 670 | New = CallInst::Create(CI, OpBundles); |
| 671 | } else { |
| 672 | New = I.clone(); |
| 673 | } |
| 674 | |
Evgeniy Stepanov | d99cca2 | 2014-06-25 09:17:21 +0000 | [diff] [blame] | 675 | ExitBlock.getInstList().insert(ExitBlock.getFirstInsertionPt(), New); |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 676 | if (!I.getName().empty()) |
| 677 | New->setName(I.getName() + ".le"); |
Evgeniy Stepanov | d99cca2 | 2014-06-25 09:17:21 +0000 | [diff] [blame] | 678 | |
| 679 | // Build LCSSA PHI nodes for any in-loop operands. Note that this is |
| 680 | // particularly cheap because we can rip off the PHI node that we're |
| 681 | // replacing for the number and blocks of the predecessors. |
| 682 | // OPT: If this shows up in a profile, we can instead finish sinking all |
| 683 | // invariant instructions, and then walk their operands to re-establish |
| 684 | // LCSSA. That will eliminate creating PHI nodes just to nuke them when |
| 685 | // sinking bottom-up. |
| 686 | for (User::op_iterator OI = New->op_begin(), OE = New->op_end(); OI != OE; |
| 687 | ++OI) |
| 688 | if (Instruction *OInst = dyn_cast<Instruction>(*OI)) |
| 689 | if (Loop *OLoop = LI->getLoopFor(OInst->getParent())) |
| 690 | if (!OLoop->contains(&PN)) { |
| 691 | PHINode *OpPN = |
| 692 | PHINode::Create(OInst->getType(), PN.getNumIncomingValues(), |
Duncan P. N. Exon Smith | be4d8cb | 2015-10-13 19:26:58 +0000 | [diff] [blame] | 693 | OInst->getName() + ".lcssa", &ExitBlock.front()); |
Evgeniy Stepanov | d99cca2 | 2014-06-25 09:17:21 +0000 | [diff] [blame] | 694 | for (unsigned i = 0, e = PN.getNumIncomingValues(); i != e; ++i) |
| 695 | OpPN->addIncoming(OInst, PN.getIncomingBlock(i)); |
| 696 | *OI = OpPN; |
| 697 | } |
| 698 | return New; |
| 699 | } |
| 700 | |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 701 | /// When an instruction is found to only be used outside of the loop, this |
| 702 | /// function moves it to the exit blocks and patches up SSA form as needed. |
Chris Lattner | 9184601 | 2003-12-19 08:18:16 +0000 | [diff] [blame] | 703 | /// This method is guaranteed to remove the original instruction from its |
| 704 | /// position, and may either delete it or move it to outside of the loop. |
Chris Lattner | aaaea51 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 705 | /// |
Pete Cooper | 0cabcf2 | 2015-05-13 01:12:18 +0000 | [diff] [blame] | 706 | static bool sink(Instruction &I, const LoopInfo *LI, const DominatorTree *DT, |
David Majnemer | 42a0730 | 2016-01-04 03:37:39 +0000 | [diff] [blame] | 707 | const Loop *CurLoop, AliasSetTracker *CurAST, |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 708 | const LoopSafetyInfo *SafetyInfo, |
| 709 | OptimizationRemarkEmitter *ORE) { |
Nick Lewycky | 299c6df | 2010-07-30 20:27:01 +0000 | [diff] [blame] | 710 | DEBUG(dbgs() << "LICM sinking instruction: " << I << "\n"); |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 711 | ORE->emit(OptimizationRemark(DEBUG_TYPE, "InstSunk", &I) |
| 712 | << "sinking " << ore::NV("Inst", &I)); |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 713 | bool Changed = false; |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 714 | if (isa<LoadInst>(I)) |
| 715 | ++NumMovedLoads; |
| 716 | else if (isa<CallInst>(I)) |
| 717 | ++NumMovedCalls; |
Chris Lattner | 55c2113 | 2003-12-10 20:43:29 +0000 | [diff] [blame] | 718 | ++NumSunk; |
| 719 | Changed = true; |
| 720 | |
Chandler Carruth | fc25854 | 2014-02-11 12:52:27 +0000 | [diff] [blame] | 721 | #ifndef NDEBUG |
| 722 | SmallVector<BasicBlock *, 32> ExitBlocks; |
| 723 | CurLoop->getUniqueExitBlocks(ExitBlocks); |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 724 | SmallPtrSet<BasicBlock *, 32> ExitBlockSet(ExitBlocks.begin(), |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 725 | ExitBlocks.end()); |
Chandler Carruth | fc25854 | 2014-02-11 12:52:27 +0000 | [diff] [blame] | 726 | #endif |
Chandler Carruth | 8765cf7 | 2014-01-25 04:07:24 +0000 | [diff] [blame] | 727 | |
Evgeniy Stepanov | 10280da | 2014-06-25 07:54:58 +0000 | [diff] [blame] | 728 | // Clones of this instruction. Don't create more than one per exit block! |
| 729 | SmallDenseMap<BasicBlock *, Instruction *, 32> SunkCopies; |
| 730 | |
Chandler Carruth | fc25854 | 2014-02-11 12:52:27 +0000 | [diff] [blame] | 731 | // If this instruction is only used outside of the loop, then all users are |
| 732 | // PHI nodes in exit blocks due to LCSSA form. Just RAUW them with clones of |
| 733 | // the instruction. |
| 734 | while (!I.use_empty()) { |
David Majnemer | 6bc83e0 | 2015-07-12 03:53:05 +0000 | [diff] [blame] | 735 | Value::user_iterator UI = I.user_begin(); |
| 736 | auto *User = cast<Instruction>(*UI); |
David Majnemer | 4942810 | 2014-09-02 16:22:00 +0000 | [diff] [blame] | 737 | if (!DT->isReachableFromEntry(User->getParent())) { |
| 738 | User->replaceUsesOfWith(&I, UndefValue::get(I.getType())); |
| 739 | continue; |
| 740 | } |
Chandler Carruth | fc25854 | 2014-02-11 12:52:27 +0000 | [diff] [blame] | 741 | // The user must be a PHI node. |
David Majnemer | 4942810 | 2014-09-02 16:22:00 +0000 | [diff] [blame] | 742 | PHINode *PN = cast<PHINode>(User); |
Chris Lattner | cc9cbc6 | 2010-08-29 17:46:00 +0000 | [diff] [blame] | 743 | |
David Majnemer | 6bc83e0 | 2015-07-12 03:53:05 +0000 | [diff] [blame] | 744 | // Surprisingly, instructions can be used outside of loops without any |
| 745 | // exits. This can only happen in PHI nodes if the incoming block is |
| 746 | // unreachable. |
| 747 | Use &U = UI.getUse(); |
| 748 | BasicBlock *BB = PN->getIncomingBlock(U); |
| 749 | if (!DT->isReachableFromEntry(BB)) { |
| 750 | U = UndefValue::get(I.getType()); |
| 751 | continue; |
| 752 | } |
| 753 | |
Chandler Carruth | fc25854 | 2014-02-11 12:52:27 +0000 | [diff] [blame] | 754 | BasicBlock *ExitBlock = PN->getParent(); |
| 755 | assert(ExitBlockSet.count(ExitBlock) && |
| 756 | "The LCSSA PHI is not in an exit block!"); |
| 757 | |
Evgeniy Stepanov | 10280da | 2014-06-25 07:54:58 +0000 | [diff] [blame] | 758 | Instruction *New; |
| 759 | auto It = SunkCopies.find(ExitBlock); |
Evgeniy Stepanov | d99cca2 | 2014-06-25 09:17:21 +0000 | [diff] [blame] | 760 | if (It != SunkCopies.end()) |
Evgeniy Stepanov | 10280da | 2014-06-25 07:54:58 +0000 | [diff] [blame] | 761 | New = It->second; |
Evgeniy Stepanov | d99cca2 | 2014-06-25 09:17:21 +0000 | [diff] [blame] | 762 | else |
| 763 | New = SunkCopies[ExitBlock] = |
David Majnemer | 42a0730 | 2016-01-04 03:37:39 +0000 | [diff] [blame] | 764 | CloneInstructionInExitBlock(I, *ExitBlock, *PN, LI, SafetyInfo); |
Chandler Carruth | fc25854 | 2014-02-11 12:52:27 +0000 | [diff] [blame] | 765 | |
| 766 | PN->replaceAllUsesWith(New); |
| 767 | PN->eraseFromParent(); |
Chris Lattner | cd96b4d | 2010-08-29 04:28:20 +0000 | [diff] [blame] | 768 | } |
Tobias Grosser | a3928f5 | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 769 | |
Chris Lattner | 1a1ed69 | 2010-08-29 18:00:00 +0000 | [diff] [blame] | 770 | CurAST->deleteValue(&I); |
Chandler Carruth | fc25854 | 2014-02-11 12:52:27 +0000 | [diff] [blame] | 771 | I.eraseFromParent(); |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 772 | return Changed; |
Chris Lattner | aaaea51 | 2003-12-10 06:41:05 +0000 | [diff] [blame] | 773 | } |
Chris Lattner | 6443769 | 2002-09-29 21:46:09 +0000 | [diff] [blame] | 774 | |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 775 | /// When an instruction is found to only use loop invariant operands that |
| 776 | /// is safe to hoist, this instruction is called to do the dirty work. |
Chris Lattner | f64f2d3 | 2002-09-26 16:52:07 +0000 | [diff] [blame] | 777 | /// |
Sanjoy Das | 7a2e2be | 2016-01-28 15:51:58 +0000 | [diff] [blame] | 778 | static bool hoist(Instruction &I, const DominatorTree *DT, const Loop *CurLoop, |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 779 | const LoopSafetyInfo *SafetyInfo, |
| 780 | OptimizationRemarkEmitter *ORE) { |
Sanjoy Das | 7a2e2be | 2016-01-28 15:51:58 +0000 | [diff] [blame] | 781 | auto *Preheader = CurLoop->getLoopPreheader(); |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 782 | DEBUG(dbgs() << "LICM hoisting to " << Preheader->getName() << ": " << I |
| 783 | << "\n"); |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 784 | ORE->emit(OptimizationRemark(DEBUG_TYPE, "Hoisted", &I) |
| 785 | << "hosting " << ore::NV("Inst", &I)); |
Sanjoy Das | 7a2e2be | 2016-01-28 15:51:58 +0000 | [diff] [blame] | 786 | |
| 787 | // Metadata can be dependent on conditions we are hoisting above. |
| 788 | // Conservatively strip all metadata on the instruction unless we were |
| 789 | // guaranteed to execute I if we entered the loop, in which case the metadata |
| 790 | // is valid in the loop preheader. |
| 791 | if (I.hasMetadataOtherThanDebugLoc() && |
| 792 | // The check on hasMetadataOtherThanDebugLoc is to prevent us from burning |
| 793 | // time in isGuaranteedToExecute if we don't actually have anything to |
| 794 | // drop. It is a compile time optimization, not required for correctness. |
| 795 | !isGuaranteedToExecute(I, DT, CurLoop, SafetyInfo)) |
| 796 | I.dropUnknownNonDebugMetadata(); |
| 797 | |
Chris Lattner | 6ac0659 | 2010-08-29 18:18:40 +0000 | [diff] [blame] | 798 | // Move the new node to the Preheader, before its terminator. |
| 799 | I.moveBefore(Preheader->getTerminator()); |
Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 800 | |
Wolfgang Pieb | c17a279 | 2017-01-06 18:38:57 +0000 | [diff] [blame] | 801 | // Do not retain debug locations when we are moving instructions to different |
| 802 | // basic blocks, because we want to avoid jumpy line tables. Calls, however, |
| 803 | // need to retain their debug locs because they may be inlined. |
| 804 | // FIXME: How do we retain source locations without causing poor debugging |
| 805 | // behavior? |
| 806 | if (!isa<CallInst>(I)) |
| 807 | I.setDebugLoc(DebugLoc()); |
| 808 | |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 809 | if (isa<LoadInst>(I)) |
| 810 | ++NumMovedLoads; |
| 811 | else if (isa<CallInst>(I)) |
| 812 | ++NumMovedCalls; |
Chris Lattner | 718b221 | 2002-09-26 16:38:03 +0000 | [diff] [blame] | 813 | ++NumHoisted; |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 814 | return true; |
Chris Lattner | 6ec05f5 | 2002-05-10 22:44:58 +0000 | [diff] [blame] | 815 | } |
| 816 | |
Sanjoy Das | f8a0db5 | 2015-05-18 18:07:00 +0000 | [diff] [blame] | 817 | /// Only sink or hoist an instruction if it is not a trapping instruction, |
| 818 | /// or if the instruction is known not to trap when moved to the preheader. |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 819 | /// or if it is a trapping instruction and is guaranteed to execute. |
Adam Nemet | e2aaf3a | 2017-01-11 04:39:49 +0000 | [diff] [blame] | 820 | static bool isSafeToExecuteUnconditionally(Instruction &Inst, |
Pete Cooper | 0cabcf2 | 2015-05-13 01:12:18 +0000 | [diff] [blame] | 821 | const DominatorTree *DT, |
| 822 | const Loop *CurLoop, |
Evgeniy Stepanov | 122f984 | 2016-06-10 20:03:17 +0000 | [diff] [blame] | 823 | const LoopSafetyInfo *SafetyInfo, |
Adam Nemet | e2aaf3a | 2017-01-11 04:39:49 +0000 | [diff] [blame] | 824 | OptimizationRemarkEmitter *ORE, |
Philip Reames | b47b9c2 | 2015-05-22 02:14:05 +0000 | [diff] [blame] | 825 | const Instruction *CtxI) { |
Sean Silva | 45835e7 | 2016-07-02 23:47:27 +0000 | [diff] [blame] | 826 | if (isSafeToSpeculativelyExecute(&Inst, CtxI, DT)) |
Eli Friedman | b8f6a4f | 2009-07-17 04:28:42 +0000 | [diff] [blame] | 827 | return true; |
Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 828 | |
Adam Nemet | e2aaf3a | 2017-01-11 04:39:49 +0000 | [diff] [blame] | 829 | bool GuaranteedToExecute = |
| 830 | isGuaranteedToExecute(Inst, DT, CurLoop, SafetyInfo); |
| 831 | |
| 832 | if (!GuaranteedToExecute) { |
| 833 | auto *LI = dyn_cast<LoadInst>(&Inst); |
| 834 | if (LI && CurLoop->isLoopInvariant(LI->getPointerOperand())) |
| 835 | ORE->emit(OptimizationRemarkMissed( |
| 836 | DEBUG_TYPE, "LoadWithLoopInvariantAddressCondExecuted", LI) |
| 837 | << "failed to hoist load with loop-invariant address " |
| 838 | "because load is conditionally executed"); |
| 839 | } |
| 840 | |
| 841 | return GuaranteedToExecute; |
Eli Friedman | 0cdc148 | 2011-07-20 21:37:47 +0000 | [diff] [blame] | 842 | } |
| 843 | |
Chris Lattner | b68ec5c | 2011-01-15 00:12:35 +0000 | [diff] [blame] | 844 | namespace { |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 845 | class LoopPromoter : public LoadAndStorePromoter { |
| 846 | Value *SomePtr; // Designated pointer to store to. |
| 847 | SmallPtrSetImpl<Value *> &PointerMustAliases; |
| 848 | SmallVectorImpl<BasicBlock *> &LoopExitBlocks; |
| 849 | SmallVectorImpl<Instruction *> &LoopInsertPts; |
| 850 | PredIteratorCache &PredCache; |
| 851 | AliasSetTracker &AST; |
| 852 | LoopInfo &LI; |
| 853 | DebugLoc DL; |
| 854 | int Alignment; |
| 855 | AAMDNodes AATags; |
Chandler Carruth | fc25854 | 2014-02-11 12:52:27 +0000 | [diff] [blame] | 856 | |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 857 | Value *maybeInsertLCSSAPHI(Value *V, BasicBlock *BB) const { |
| 858 | if (Instruction *I = dyn_cast<Instruction>(V)) |
| 859 | if (Loop *L = LI.getLoopFor(I->getParent())) |
| 860 | if (!L->contains(BB)) { |
| 861 | // We need to create an LCSSA PHI node for the incoming value and |
| 862 | // store that. |
| 863 | PHINode *PN = PHINode::Create(I->getType(), PredCache.size(BB), |
| 864 | I->getName() + ".lcssa", &BB->front()); |
| 865 | for (BasicBlock *Pred : PredCache.get(BB)) |
| 866 | PN->addIncoming(I, Pred); |
| 867 | return PN; |
| 868 | } |
| 869 | return V; |
| 870 | } |
Chandler Carruth | fc25854 | 2014-02-11 12:52:27 +0000 | [diff] [blame] | 871 | |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 872 | public: |
| 873 | LoopPromoter(Value *SP, ArrayRef<const Instruction *> Insts, SSAUpdater &S, |
| 874 | SmallPtrSetImpl<Value *> &PMA, |
| 875 | SmallVectorImpl<BasicBlock *> &LEB, |
| 876 | SmallVectorImpl<Instruction *> &LIP, PredIteratorCache &PIC, |
| 877 | AliasSetTracker &ast, LoopInfo &li, DebugLoc dl, int alignment, |
| 878 | const AAMDNodes &AATags) |
| 879 | : LoadAndStorePromoter(Insts, S), SomePtr(SP), PointerMustAliases(PMA), |
| 880 | LoopExitBlocks(LEB), LoopInsertPts(LIP), PredCache(PIC), AST(ast), |
Benjamin Kramer | 82de7d3 | 2016-05-27 14:27:24 +0000 | [diff] [blame] | 881 | LI(li), DL(std::move(dl)), Alignment(alignment), AATags(AATags) {} |
Tobias Grosser | a3928f5 | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 882 | |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 883 | bool isInstInList(Instruction *I, |
| 884 | const SmallVectorImpl<Instruction *> &) const override { |
| 885 | Value *Ptr; |
| 886 | if (LoadInst *LI = dyn_cast<LoadInst>(I)) |
| 887 | Ptr = LI->getOperand(0); |
| 888 | else |
| 889 | Ptr = cast<StoreInst>(I)->getPointerOperand(); |
| 890 | return PointerMustAliases.count(Ptr); |
| 891 | } |
Tobias Grosser | a3928f5 | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 892 | |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 893 | void doExtraRewritesBeforeFinalDeletion() const override { |
| 894 | // Insert stores after in the loop exit blocks. Each exit block gets a |
| 895 | // store of the live-out values that feed them. Since we've already told |
| 896 | // the SSA updater about the defs in the loop and the preheader |
| 897 | // definition, it is all set and we can start using it. |
| 898 | for (unsigned i = 0, e = LoopExitBlocks.size(); i != e; ++i) { |
| 899 | BasicBlock *ExitBlock = LoopExitBlocks[i]; |
| 900 | Value *LiveInValue = SSA.GetValueInMiddleOfBlock(ExitBlock); |
| 901 | LiveInValue = maybeInsertLCSSAPHI(LiveInValue, ExitBlock); |
| 902 | Value *Ptr = maybeInsertLCSSAPHI(SomePtr, ExitBlock); |
| 903 | Instruction *InsertPos = LoopInsertPts[i]; |
| 904 | StoreInst *NewSI = new StoreInst(LiveInValue, Ptr, InsertPos); |
| 905 | NewSI->setAlignment(Alignment); |
| 906 | NewSI->setDebugLoc(DL); |
| 907 | if (AATags) |
| 908 | NewSI->setAAMetadata(AATags); |
Chris Lattner | b68ec5c | 2011-01-15 00:12:35 +0000 | [diff] [blame] | 909 | } |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 910 | } |
Chris Lattner | b68ec5c | 2011-01-15 00:12:35 +0000 | [diff] [blame] | 911 | |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 912 | void replaceLoadWithValue(LoadInst *LI, Value *V) const override { |
| 913 | // Update alias analysis. |
| 914 | AST.copyValue(LI, V); |
| 915 | } |
| 916 | void instructionDeleted(Instruction *I) const override { AST.deleteValue(I); } |
| 917 | }; |
Alexander Kornienko | f00654e | 2015-06-23 09:49:53 +0000 | [diff] [blame] | 918 | } // end anon namespace |
Chris Lattner | b68ec5c | 2011-01-15 00:12:35 +0000 | [diff] [blame] | 919 | |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 920 | /// Try to promote memory values to scalars by sinking stores out of the |
| 921 | /// loop and moving loads to before the loop. We do this by looping over |
| 922 | /// the stores in the loop, looking for stores to Must pointers which are |
| 923 | /// loop invariant. |
Chris Lattner | 45d67d6 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 924 | /// |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 925 | bool llvm::promoteLoopAccessesToScalars( |
| 926 | AliasSet &AS, SmallVectorImpl<BasicBlock *> &ExitBlocks, |
| 927 | SmallVectorImpl<Instruction *> &InsertPts, PredIteratorCache &PIC, |
| 928 | LoopInfo *LI, DominatorTree *DT, const TargetLibraryInfo *TLI, |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 929 | Loop *CurLoop, AliasSetTracker *CurAST, LoopSafetyInfo *SafetyInfo, |
| 930 | OptimizationRemarkEmitter *ORE) { |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 931 | // Verify inputs. |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 932 | assert(LI != nullptr && DT != nullptr && CurLoop != nullptr && |
| 933 | CurAST != nullptr && SafetyInfo != nullptr && |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 934 | "Unexpected Input to promoteLoopAccessesToScalars"); |
Sanjay Patel | 9913322 | 2016-01-13 23:01:57 +0000 | [diff] [blame] | 935 | |
Chris Lattner | 1dc98b4 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 936 | // We can promote this alias set if it has a store, if it is a "Must" alias |
| 937 | // set, if the pointer is loop invariant, and if we are not eliminating any |
| 938 | // volatile loads or stores. |
| 939 | if (AS.isForwardingAliasSet() || !AS.isMod() || !AS.isMustAlias() || |
| 940 | AS.isVolatile() || !CurLoop->isLoopInvariant(AS.begin()->getValue())) |
Sanjay Patel | 9913322 | 2016-01-13 23:01:57 +0000 | [diff] [blame] | 941 | return false; |
Tobias Grosser | a3928f5 | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 942 | |
Chris Lattner | 1dc98b4 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 943 | assert(!AS.empty() && |
| 944 | "Must alias set should have at least one pointer element in it!"); |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 945 | |
Chris Lattner | 1dc98b4 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 946 | Value *SomePtr = AS.begin()->getValue(); |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 947 | BasicBlock *Preheader = CurLoop->getLoopPreheader(); |
Chris Lattner | 45d67d6 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 948 | |
Chris Lattner | 1dc98b4 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 949 | // It isn't safe to promote a load/store from the loop if the load/store is |
| 950 | // conditional. For example, turning: |
Chris Lattner | 45d67d6 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 951 | // |
Chris Lattner | 1dc98b4 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 952 | // for () { if (c) *P += 1; } |
Chris Lattner | 45d67d6 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 953 | // |
Chris Lattner | 1dc98b4 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 954 | // into: |
| 955 | // |
| 956 | // tmp = *P; for () { if (c) tmp +=1; } *P = tmp; |
| 957 | // |
| 958 | // is not safe, because *P may only be valid to access if 'c' is true. |
Tobias Grosser | a3928f5 | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 959 | // |
Philip Reames | b54c8e6 | 2016-03-09 22:59:30 +0000 | [diff] [blame] | 960 | // The safety property divides into two parts: |
Michael Kuperstein | 62b98c3 | 2016-12-30 00:39:00 +0000 | [diff] [blame] | 961 | // p1) The memory may not be dereferenceable on entry to the loop. In this |
Philip Reames | b54c8e6 | 2016-03-09 22:59:30 +0000 | [diff] [blame] | 962 | // case, we can't insert the required load in the preheader. |
Michael Kuperstein | 62b98c3 | 2016-12-30 00:39:00 +0000 | [diff] [blame] | 963 | // p2) The memory model does not allow us to insert a store along any dynamic |
Philip Reames | b54c8e6 | 2016-03-09 22:59:30 +0000 | [diff] [blame] | 964 | // path which did not originally have one. |
| 965 | // |
Michael Kuperstein | 62b98c3 | 2016-12-30 00:39:00 +0000 | [diff] [blame] | 966 | // If at least one store is guaranteed to execute, both properties are |
| 967 | // satisfied, and promotion is legal. |
Michael Kuperstein | c9acad1 | 2017-01-05 20:42:06 +0000 | [diff] [blame] | 968 | // |
Michael Kuperstein | 62b98c3 | 2016-12-30 00:39:00 +0000 | [diff] [blame] | 969 | // This, however, is not a necessary condition. Even if no store/load is |
Michael Kuperstein | c9acad1 | 2017-01-05 20:42:06 +0000 | [diff] [blame] | 970 | // guaranteed to execute, we can still establish these properties. |
| 971 | // We can establish (p1) by proving that hoisting the load into the preheader |
| 972 | // is safe (i.e. proving dereferenceability on all paths through the loop). We |
Michael Kuperstein | 62b98c3 | 2016-12-30 00:39:00 +0000 | [diff] [blame] | 973 | // can use any access within the alias set to prove dereferenceability, |
Philip Reames | b54c8e6 | 2016-03-09 22:59:30 +0000 | [diff] [blame] | 974 | // since they're all must alias. |
Michael Kuperstein | c9acad1 | 2017-01-05 20:42:06 +0000 | [diff] [blame] | 975 | // |
| 976 | // There are two ways establish (p2): |
| 977 | // a) Prove the location is thread-local. In this case the memory model |
Michael Kuperstein | 62b98c3 | 2016-12-30 00:39:00 +0000 | [diff] [blame] | 978 | // requirement does not apply, and stores are safe to insert. |
Michael Kuperstein | c9acad1 | 2017-01-05 20:42:06 +0000 | [diff] [blame] | 979 | // b) Prove a store dominates every exit block. In this case, if an exit |
| 980 | // blocks is reached, the original dynamic path would have taken us through |
| 981 | // the store, so inserting a store into the exit block is safe. Note that this |
| 982 | // is different from the store being guaranteed to execute. For instance, |
| 983 | // if an exception is thrown on the first iteration of the loop, the original |
| 984 | // store is never executed, but the exit blocks are not executed either. |
Michael Kuperstein | 62b98c3 | 2016-12-30 00:39:00 +0000 | [diff] [blame] | 985 | |
| 986 | bool DereferenceableInPH = false; |
| 987 | bool SafeToInsertStore = false; |
Philip Reames | b54c8e6 | 2016-03-09 22:59:30 +0000 | [diff] [blame] | 988 | |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 989 | SmallVector<Instruction *, 64> LoopUses; |
| 990 | SmallPtrSet<Value *, 4> PointerMustAliases; |
Chris Lattner | 45d67d6 | 2003-02-24 03:52:32 +0000 | [diff] [blame] | 991 | |
Tobias Grosser | 4a5d9a9 | 2011-07-06 19:19:55 +0000 | [diff] [blame] | 992 | // We start with an alignment of one and try to find instructions that allow |
| 993 | // us to prove better alignment. |
| 994 | unsigned Alignment = 1; |
Hal Finkel | cc39b67 | 2014-07-24 12:16:19 +0000 | [diff] [blame] | 995 | AAMDNodes AATags; |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 996 | |
Philip Reames | b54c8e6 | 2016-03-09 22:59:30 +0000 | [diff] [blame] | 997 | const DataLayout &MDL = Preheader->getModule()->getDataLayout(); |
| 998 | |
Eli Friedman | ee89505 | 2016-06-05 22:13:52 +0000 | [diff] [blame] | 999 | if (SafetyInfo->MayThrow) { |
| 1000 | // If a loop can throw, we have to insert a store along each unwind edge. |
| 1001 | // That said, we can't actually make the unwind edge explicit. Therefore, |
| 1002 | // we have to prove that the store is dead along the unwind edge. |
| 1003 | // |
| 1004 | // Currently, this code just special-cases alloca instructions. |
| 1005 | if (!isa<AllocaInst>(GetUnderlyingObject(SomePtr, MDL))) |
| 1006 | return false; |
| 1007 | } |
| 1008 | |
Chris Lattner | 1dc98b4 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 1009 | // Check that all of the pointers in the alias set have the same type. We |
| 1010 | // cannot (yet) promote a memory location that is loaded and stored in |
Hal Finkel | cc39b67 | 2014-07-24 12:16:19 +0000 | [diff] [blame] | 1011 | // different sizes. While we are at it, collect alignment and AA info. |
Benjamin Kramer | 135f735 | 2016-06-26 12:28:59 +0000 | [diff] [blame] | 1012 | for (const auto &ASI : AS) { |
| 1013 | Value *ASIV = ASI.getValue(); |
Chris Lattner | 1dc98b4 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 1014 | PointerMustAliases.insert(ASIV); |
Tobias Grosser | a3928f5 | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 1015 | |
Chris Lattner | f12c08d | 2008-05-22 00:53:38 +0000 | [diff] [blame] | 1016 | // Check that all of the pointers in the alias set have the same type. We |
| 1017 | // cannot (yet) promote a memory location that is loaded and stored in |
| 1018 | // different sizes. |
Chris Lattner | 1dc98b4 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 1019 | if (SomePtr->getType() != ASIV->getType()) |
Michael Kuperstein | 4a86a19 | 2016-12-30 00:43:22 +0000 | [diff] [blame] | 1020 | return false; |
Tobias Grosser | a3928f5 | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 1021 | |
Chandler Carruth | cdf4788 | 2014-03-09 03:16:01 +0000 | [diff] [blame] | 1022 | for (User *U : ASIV->users()) { |
Chris Lattner | 1dc98b4 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 1023 | // Ignore instructions that are outside the loop. |
Chandler Carruth | cdf4788 | 2014-03-09 03:16:01 +0000 | [diff] [blame] | 1024 | Instruction *UI = dyn_cast<Instruction>(U); |
| 1025 | if (!UI || !CurLoop->contains(UI)) |
Chris Lattner | f12c08d | 2008-05-22 00:53:38 +0000 | [diff] [blame] | 1026 | continue; |
Tobias Grosser | a3928f5 | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 1027 | |
Chris Lattner | 1dc98b4 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 1028 | // If there is an non-load/store instruction in the loop, we can't promote |
| 1029 | // it. |
Adam Nemet | e2aaf3a | 2017-01-11 04:39:49 +0000 | [diff] [blame] | 1030 | if (LoadInst *Load = dyn_cast<LoadInst>(UI)) { |
Sanjay Patel | 9f49b68 | 2016-01-08 22:05:03 +0000 | [diff] [blame] | 1031 | assert(!Load->isVolatile() && "AST broken"); |
| 1032 | if (!Load->isSimple()) |
Michael Kuperstein | 4a86a19 | 2016-12-30 00:43:22 +0000 | [diff] [blame] | 1033 | return false; |
Philip Reames | b54c8e6 | 2016-03-09 22:59:30 +0000 | [diff] [blame] | 1034 | |
Michael Kuperstein | 62b98c3 | 2016-12-30 00:39:00 +0000 | [diff] [blame] | 1035 | if (!DereferenceableInPH) |
| 1036 | DereferenceableInPH = isSafeToExecuteUnconditionally( |
Adam Nemet | e2aaf3a | 2017-01-11 04:39:49 +0000 | [diff] [blame] | 1037 | *Load, DT, CurLoop, SafetyInfo, ORE, Preheader->getTerminator()); |
Sanjay Patel | 9f49b68 | 2016-01-08 22:05:03 +0000 | [diff] [blame] | 1038 | } else if (const StoreInst *Store = dyn_cast<StoreInst>(UI)) { |
Chris Lattner | 408a684 | 2010-12-19 05:57:25 +0000 | [diff] [blame] | 1039 | // Stores *of* the pointer are not interesting, only stores *to* the |
| 1040 | // pointer. |
Chandler Carruth | cdf4788 | 2014-03-09 03:16:01 +0000 | [diff] [blame] | 1041 | if (UI->getOperand(1) != ASIV) |
Chris Lattner | 408a684 | 2010-12-19 05:57:25 +0000 | [diff] [blame] | 1042 | continue; |
Sanjay Patel | 9f49b68 | 2016-01-08 22:05:03 +0000 | [diff] [blame] | 1043 | assert(!Store->isVolatile() && "AST broken"); |
| 1044 | if (!Store->isSimple()) |
Michael Kuperstein | 4a86a19 | 2016-12-30 00:43:22 +0000 | [diff] [blame] | 1045 | return false; |
Eli Friedman | 0cdc148 | 2011-07-20 21:37:47 +0000 | [diff] [blame] | 1046 | |
Michael Kuperstein | 62b98c3 | 2016-12-30 00:39:00 +0000 | [diff] [blame] | 1047 | // If the store is guaranteed to execute, both properties are satisfied. |
| 1048 | // We may want to check if a store is guaranteed to execute even if we |
| 1049 | // already know that promotion is safe, since it may have higher |
| 1050 | // alignment than any other guaranteed stores, in which case we can |
| 1051 | // raise the alignment on the promoted store. |
Sanjay Patel | 9f49b68 | 2016-01-08 22:05:03 +0000 | [diff] [blame] | 1052 | unsigned InstAlignment = Store->getAlignment(); |
Michael Kuperstein | 62b98c3 | 2016-12-30 00:39:00 +0000 | [diff] [blame] | 1053 | if (!InstAlignment) |
| 1054 | InstAlignment = |
| 1055 | MDL.getABITypeAlignment(Store->getValueOperand()->getType()); |
| 1056 | |
| 1057 | if (!DereferenceableInPH || !SafeToInsertStore || |
| 1058 | (InstAlignment > Alignment)) { |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 1059 | if (isGuaranteedToExecute(*UI, DT, CurLoop, SafetyInfo)) { |
Michael Kuperstein | 62b98c3 | 2016-12-30 00:39:00 +0000 | [diff] [blame] | 1060 | DereferenceableInPH = true; |
| 1061 | SafeToInsertStore = true; |
| 1062 | Alignment = std::max(Alignment, InstAlignment); |
Eli Friedman | 0cdc148 | 2011-07-20 21:37:47 +0000 | [diff] [blame] | 1063 | } |
Anna Thomas | 6715135 | 2016-06-24 12:38:45 +0000 | [diff] [blame] | 1064 | } |
Philip Reames | b54c8e6 | 2016-03-09 22:59:30 +0000 | [diff] [blame] | 1065 | |
Michael Kuperstein | c9acad1 | 2017-01-05 20:42:06 +0000 | [diff] [blame] | 1066 | // If a store dominates all exit blocks, it is safe to sink. |
| 1067 | // As explained above, if an exit block was executed, a dominating |
| 1068 | // store must have been been executed at least once, so we are not |
| 1069 | // introducing stores on paths that did not have them. |
| 1070 | // Note that this only looks at explicit exit blocks. If we ever |
| 1071 | // start sinking stores into unwind edges (see above), this will break. |
| 1072 | if (!SafeToInsertStore) |
| 1073 | SafeToInsertStore = llvm::all_of(ExitBlocks, [&](BasicBlock *Exit) { |
| 1074 | return DT->dominates(Store->getParent(), Exit); |
| 1075 | }); |
| 1076 | |
Michael Kuperstein | 62b98c3 | 2016-12-30 00:39:00 +0000 | [diff] [blame] | 1077 | // If the store is not guaranteed to execute, we may still get |
| 1078 | // deref info through it. |
| 1079 | if (!DereferenceableInPH) { |
| 1080 | DereferenceableInPH = isDereferenceableAndAlignedPointer( |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 1081 | Store->getPointerOperand(), Store->getAlignment(), MDL, |
Sean Silva | 45835e7 | 2016-07-02 23:47:27 +0000 | [diff] [blame] | 1082 | Preheader->getTerminator(), DT); |
Philip Reames | b54c8e6 | 2016-03-09 22:59:30 +0000 | [diff] [blame] | 1083 | } |
Chris Lattner | be90190 | 2010-09-06 05:11:24 +0000 | [diff] [blame] | 1084 | } else |
Michael Kuperstein | 4a86a19 | 2016-12-30 00:43:22 +0000 | [diff] [blame] | 1085 | return false; // Not a load or store. |
Tobias Grosser | 4a5d9a9 | 2011-07-06 19:19:55 +0000 | [diff] [blame] | 1086 | |
Hal Finkel | cc39b67 | 2014-07-24 12:16:19 +0000 | [diff] [blame] | 1087 | // Merge the AA tags. |
Chris Lattner | f5cca68 | 2012-12-31 08:37:17 +0000 | [diff] [blame] | 1088 | if (LoopUses.empty()) { |
Hal Finkel | cc39b67 | 2014-07-24 12:16:19 +0000 | [diff] [blame] | 1089 | // On the first load/store, just take its AA tags. |
| 1090 | UI->getAAMetadata(AATags); |
| 1091 | } else if (AATags) { |
| 1092 | UI->getAAMetadata(AATags, /* Merge = */ true); |
Chris Lattner | f5cca68 | 2012-12-31 08:37:17 +0000 | [diff] [blame] | 1093 | } |
Chandler Carruth | cdf4788 | 2014-03-09 03:16:01 +0000 | [diff] [blame] | 1094 | |
| 1095 | LoopUses.push_back(UI); |
Chris Lattner | 1dc98b4 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 1096 | } |
| 1097 | } |
Tobias Grosser | 4a5d9a9 | 2011-07-06 19:19:55 +0000 | [diff] [blame] | 1098 | |
Michael Kuperstein | 62b98c3 | 2016-12-30 00:39:00 +0000 | [diff] [blame] | 1099 | |
| 1100 | // If we couldn't prove we can hoist the load, bail. |
| 1101 | if (!DereferenceableInPH) |
Michael Kuperstein | 4a86a19 | 2016-12-30 00:43:22 +0000 | [diff] [blame] | 1102 | return false; |
Michael Kuperstein | 62b98c3 | 2016-12-30 00:39:00 +0000 | [diff] [blame] | 1103 | |
| 1104 | // We know we can hoist the load, but don't have a guaranteed store. |
| 1105 | // Check whether the location is thread-local. If it is, then we can insert |
| 1106 | // stores along paths which originally didn't have them without violating the |
| 1107 | // memory model. |
| 1108 | if (!SafeToInsertStore) { |
Philip Reames | b54c8e6 | 2016-03-09 22:59:30 +0000 | [diff] [blame] | 1109 | Value *Object = GetUnderlyingObject(SomePtr, MDL); |
Michael Kuperstein | 62b98c3 | 2016-12-30 00:39:00 +0000 | [diff] [blame] | 1110 | SafeToInsertStore = |
Michael Kuperstein | 76e06c8 | 2016-12-30 01:03:17 +0000 | [diff] [blame] | 1111 | (isAllocLikeFn(Object, TLI) || isa<AllocaInst>(Object)) && |
| 1112 | !PointerMayBeCaptured(Object, true, true); |
Philip Reames | b54c8e6 | 2016-03-09 22:59:30 +0000 | [diff] [blame] | 1113 | } |
Michael Kuperstein | ff36bae | 2016-12-29 23:11:19 +0000 | [diff] [blame] | 1114 | |
Michael Kuperstein | 62b98c3 | 2016-12-30 00:39:00 +0000 | [diff] [blame] | 1115 | // If we've still failed to prove we can sink the store, give up. |
| 1116 | if (!SafeToInsertStore) |
Michael Kuperstein | 4a86a19 | 2016-12-30 00:43:22 +0000 | [diff] [blame] | 1117 | return false; |
Tobias Grosser | a3928f5 | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 1118 | |
Chris Lattner | 1dc98b4 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 1119 | // Otherwise, this is safe to promote, lets do it! |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 1120 | DEBUG(dbgs() << "LICM: Promoting value stored to in loop: " << *SomePtr |
| 1121 | << '\n'); |
Adam Nemet | 358433c | 2017-01-11 04:39:35 +0000 | [diff] [blame] | 1122 | ORE->emit( |
| 1123 | OptimizationRemark(DEBUG_TYPE, "PromoteLoopAccessesToScalar", LoopUses[0]) |
| 1124 | << "Moving accesses to memory location out of the loop"); |
Chris Lattner | 1dc98b4 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 1125 | ++NumPromoted; |
| 1126 | |
Eli Friedman | ddf7f55 | 2011-05-27 20:31:51 +0000 | [diff] [blame] | 1127 | // Grab a debug location for the inserted loads/stores; given that the |
| 1128 | // inserted loads/stores have little relation to the original loads/stores, |
| 1129 | // this code just arbitrarily picks a location from one, since any debug |
| 1130 | // location is better than none. |
| 1131 | DebugLoc DL = LoopUses[0]->getDebugLoc(); |
| 1132 | |
Chris Lattner | 1dc98b4 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 1133 | // We use the SSAUpdater interface to insert phi nodes as required. |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 1134 | SmallVector<PHINode *, 16> NewPHIs; |
Chris Lattner | 1dc98b4 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 1135 | SSAUpdater SSA(&NewPHIs); |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 1136 | LoopPromoter Promoter(SomePtr, LoopUses, SSA, PointerMustAliases, ExitBlocks, |
Hal Finkel | cc39b67 | 2014-07-24 12:16:19 +0000 | [diff] [blame] | 1137 | InsertPts, PIC, *CurAST, *LI, DL, Alignment, AATags); |
Tobias Grosser | a3928f5 | 2011-07-06 19:20:02 +0000 | [diff] [blame] | 1138 | |
Chris Lattner | b68ec5c | 2011-01-15 00:12:35 +0000 | [diff] [blame] | 1139 | // Set up the preheader to have a definition of the value. It is the live-out |
| 1140 | // value from the preheader that uses in the loop will use. |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 1141 | LoadInst *PreheaderLoad = new LoadInst( |
| 1142 | SomePtr, SomePtr->getName() + ".promoted", Preheader->getTerminator()); |
Tobias Grosser | 4a5d9a9 | 2011-07-06 19:19:55 +0000 | [diff] [blame] | 1143 | PreheaderLoad->setAlignment(Alignment); |
Eli Friedman | ddf7f55 | 2011-05-27 20:31:51 +0000 | [diff] [blame] | 1144 | PreheaderLoad->setDebugLoc(DL); |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 1145 | if (AATags) |
| 1146 | PreheaderLoad->setAAMetadata(AATags); |
Chris Lattner | 1dc98b4 | 2010-08-29 06:43:52 +0000 | [diff] [blame] | 1147 | SSA.AddAvailableValue(Preheader, PreheaderLoad); |
| 1148 | |
Chris Lattner | b68ec5c | 2011-01-15 00:12:35 +0000 | [diff] [blame] | 1149 | // Rewrite all the loads in the loop and remember all the definitions from |
| 1150 | // stores in the loop. |
| 1151 | Promoter.run(LoopUses); |
Eli Friedman | c5f22a7 | 2011-04-07 01:35:06 +0000 | [diff] [blame] | 1152 | |
| 1153 | // If the SSAUpdater didn't use the load in the preheader, just zap it now. |
| 1154 | if (PreheaderLoad->use_empty()) |
| 1155 | PreheaderLoad->eraseFromParent(); |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 1156 | |
Michael Kuperstein | 4a86a19 | 2016-12-30 00:43:22 +0000 | [diff] [blame] | 1157 | return true; |
Chris Lattner | a51fa88 | 2002-08-22 21:39:55 +0000 | [diff] [blame] | 1158 | } |
Devang Patel | b98a097 | 2007-07-31 08:01:41 +0000 | [diff] [blame] | 1159 | |
Roman Gareev | 036c088 | 2016-02-15 14:48:50 +0000 | [diff] [blame] | 1160 | /// Returns an owning pointer to an alias set which incorporates aliasing info |
Chandler Carruth | ad8cb38 | 2016-02-27 04:34:07 +0000 | [diff] [blame] | 1161 | /// from L and all subloops of L. |
Xinliang David Li | cbb5e02 | 2016-08-11 22:34:00 +0000 | [diff] [blame] | 1162 | /// FIXME: In new pass manager, there is no helper function to handle loop |
| 1163 | /// analysis such as cloneBasicBlockAnalysis, so the AST needs to be recomputed |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 1164 | /// from scratch for every loop. Hook up with the helper functions when |
| 1165 | /// available in the new pass manager to avoid redundant computation. |
| 1166 | AliasSetTracker * |
| 1167 | LoopInvariantCodeMotion::collectAliasInfoForLoop(Loop *L, LoopInfo *LI, |
| 1168 | AliasAnalysis *AA) { |
Roman Gareev | 036c088 | 2016-02-15 14:48:50 +0000 | [diff] [blame] | 1169 | AliasSetTracker *CurAST = nullptr; |
Chandler Carruth | ad8cb38 | 2016-02-27 04:34:07 +0000 | [diff] [blame] | 1170 | SmallVector<Loop *, 4> RecomputeLoops; |
Roman Gareev | 036c088 | 2016-02-15 14:48:50 +0000 | [diff] [blame] | 1171 | for (Loop *InnerL : L->getSubLoops()) { |
Chandler Carruth | ad8cb38 | 2016-02-27 04:34:07 +0000 | [diff] [blame] | 1172 | auto MapI = LoopToAliasSetMap.find(InnerL); |
| 1173 | // If the AST for this inner loop is missing it may have been merged into |
| 1174 | // some other loop's AST and then that loop unrolled, and so we need to |
| 1175 | // recompute it. |
| 1176 | if (MapI == LoopToAliasSetMap.end()) { |
| 1177 | RecomputeLoops.push_back(InnerL); |
| 1178 | continue; |
| 1179 | } |
| 1180 | AliasSetTracker *InnerAST = MapI->second; |
Roman Gareev | 036c088 | 2016-02-15 14:48:50 +0000 | [diff] [blame] | 1181 | |
| 1182 | if (CurAST != nullptr) { |
| 1183 | // What if InnerLoop was modified by other passes ? |
| 1184 | CurAST->add(*InnerAST); |
| 1185 | |
| 1186 | // Once we've incorporated the inner loop's AST into ours, we don't need |
| 1187 | // the subloop's anymore. |
| 1188 | delete InnerAST; |
| 1189 | } else { |
| 1190 | CurAST = InnerAST; |
| 1191 | } |
Chandler Carruth | ad8cb38 | 2016-02-27 04:34:07 +0000 | [diff] [blame] | 1192 | LoopToAliasSetMap.erase(MapI); |
Roman Gareev | 036c088 | 2016-02-15 14:48:50 +0000 | [diff] [blame] | 1193 | } |
| 1194 | if (CurAST == nullptr) |
| 1195 | CurAST = new AliasSetTracker(*AA); |
Chandler Carruth | ad8cb38 | 2016-02-27 04:34:07 +0000 | [diff] [blame] | 1196 | |
| 1197 | auto mergeLoop = [&](Loop *L) { |
| 1198 | // Loop over the body of this loop, looking for calls, invokes, and stores. |
| 1199 | // Because subloops have already been incorporated into AST, we skip blocks |
| 1200 | // in subloops. |
| 1201 | for (BasicBlock *BB : L->blocks()) |
| 1202 | if (LI->getLoopFor(BB) == L) // Ignore blocks in subloops. |
| 1203 | CurAST->add(*BB); // Incorporate the specified basic block |
| 1204 | }; |
| 1205 | |
| 1206 | // Add everything from the sub loops that are no longer directly available. |
| 1207 | for (Loop *InnerL : RecomputeLoops) |
| 1208 | mergeLoop(InnerL); |
| 1209 | |
| 1210 | // And merge in this loop. |
| 1211 | mergeLoop(L); |
| 1212 | |
Roman Gareev | 036c088 | 2016-02-15 14:48:50 +0000 | [diff] [blame] | 1213 | return CurAST; |
| 1214 | } |
| 1215 | |
Ashutosh Nema | 4780262 | 2015-08-13 11:18:35 +0000 | [diff] [blame] | 1216 | /// Simple analysis hook. Clone alias set info. |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 1217 | /// |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 1218 | void LegacyLICMPass::cloneBasicBlockAnalysis(BasicBlock *From, BasicBlock *To, |
| 1219 | Loop *L) { |
| 1220 | AliasSetTracker *AST = LICM.getLoopToAliasSetMap().lookup(L); |
Devang Patel | b98a097 | 2007-07-31 08:01:41 +0000 | [diff] [blame] | 1221 | if (!AST) |
| 1222 | return; |
| 1223 | |
| 1224 | AST->copyValue(From, To); |
| 1225 | } |
| 1226 | |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 1227 | /// Simple Analysis hook. Delete value V from alias set |
| 1228 | /// |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 1229 | void LegacyLICMPass::deleteAnalysisValue(Value *V, Loop *L) { |
| 1230 | AliasSetTracker *AST = LICM.getLoopToAliasSetMap().lookup(L); |
Devang Patel | b98a097 | 2007-07-31 08:01:41 +0000 | [diff] [blame] | 1231 | if (!AST) |
| 1232 | return; |
| 1233 | |
| 1234 | AST->deleteValue(V); |
| 1235 | } |
David Peixotto | 0d4d5e6 | 2014-09-24 16:48:31 +0000 | [diff] [blame] | 1236 | |
| 1237 | /// Simple Analysis hook. Delete value L from alias set map. |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 1238 | /// |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 1239 | void LegacyLICMPass::deleteAnalysisLoop(Loop *L) { |
| 1240 | AliasSetTracker *AST = LICM.getLoopToAliasSetMap().lookup(L); |
David Peixotto | 0d4d5e6 | 2014-09-24 16:48:31 +0000 | [diff] [blame] | 1241 | if (!AST) |
| 1242 | return; |
| 1243 | |
| 1244 | delete AST; |
Dehao Chen | 9cba1f4 | 2016-07-12 22:37:48 +0000 | [diff] [blame] | 1245 | LICM.getLoopToAliasSetMap().erase(L); |
David Peixotto | 0d4d5e6 | 2014-09-24 16:48:31 +0000 | [diff] [blame] | 1246 | } |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 1247 | |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 1248 | /// Return true if the body of this loop may store into the memory |
| 1249 | /// location pointed to by V. |
| 1250 | /// |
| 1251 | static bool pointerInvalidatedByLoop(Value *V, uint64_t Size, |
Dehao Chen | d55bc4c | 2016-05-05 00:54:54 +0000 | [diff] [blame] | 1252 | const AAMDNodes &AAInfo, |
Hal Finkel | 3d4269a | 2015-02-22 18:35:32 +0000 | [diff] [blame] | 1253 | AliasSetTracker *CurAST) { |
| 1254 | // Check to see if any of the basic blocks in CurLoop invalidate *V. |
| 1255 | return CurAST->getAliasSetForPointer(V, Size, AAInfo).isMod(); |
| 1256 | } |
| 1257 | |
| 1258 | /// Little predicate that returns true if the specified basic block is in |
| 1259 | /// a subloop of the current one, not the current one itself. |
| 1260 | /// |
| 1261 | static bool inSubLoop(BasicBlock *BB, Loop *CurLoop, LoopInfo *LI) { |
| 1262 | assert(CurLoop->contains(BB) && "Only valid if BB is IN the loop"); |
| 1263 | return LI->getLoopFor(BB) != CurLoop; |
| 1264 | } |