Chris Lattner | 6148c02 | 2001-12-03 17:28:42 +0000 | [diff] [blame] | 1 | //===- IndVarSimplify.cpp - Induction Variable Elimination ----------------===// |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 2 | // |
John Criswell | b576c94 | 2003-10-20 19:43:21 +0000 | [diff] [blame] | 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
Chris Lattner | 4ee451d | 2007-12-29 20:36:04 +0000 | [diff] [blame] | 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 7 | // |
John Criswell | b576c94 | 2003-10-20 19:43:21 +0000 | [diff] [blame] | 8 | //===----------------------------------------------------------------------===// |
Chris Lattner | 6148c02 | 2001-12-03 17:28:42 +0000 | [diff] [blame] | 9 | // |
Chris Lattner | 40bf8b4 | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 10 | // This transformation analyzes and transforms the induction variables (and |
| 11 | // computations derived from them) into simpler forms suitable for subsequent |
| 12 | // analysis and transformation. |
| 13 | // |
Reid Spencer | 47a53ac | 2006-08-18 09:01:07 +0000 | [diff] [blame] | 14 | // This transformation makes the following changes to each loop with an |
Chris Lattner | 40bf8b4 | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 15 | // identifiable induction variable: |
| 16 | // 1. All loops are transformed to have a SINGLE canonical induction variable |
| 17 | // which starts at zero and steps by one. |
| 18 | // 2. The canonical induction variable is guaranteed to be the first PHI node |
| 19 | // in the loop header block. |
Dan Gohman | ea73f3c | 2009-06-14 22:38:41 +0000 | [diff] [blame] | 20 | // 3. The canonical induction variable is guaranteed to be in a wide enough |
| 21 | // type so that IV expressions need not be (directly) zero-extended or |
| 22 | // sign-extended. |
| 23 | // 4. Any pointer arithmetic recurrences are raised to use array subscripts. |
Chris Lattner | 40bf8b4 | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 24 | // |
| 25 | // If the trip count of a loop is computable, this pass also makes the following |
| 26 | // changes: |
| 27 | // 1. The exit condition for the loop is canonicalized to compare the |
| 28 | // induction value against the exit value. This turns loops like: |
| 29 | // 'for (i = 7; i*i < 1000; ++i)' into 'for (i = 0; i != 25; ++i)' |
| 30 | // 2. Any use outside of the loop of an expression derived from the indvar |
| 31 | // is changed to compute the derived value outside of the loop, eliminating |
| 32 | // the dependence on the exit value of the induction variable. If the only |
| 33 | // purpose of the loop is to compute the exit value of some derived |
| 34 | // expression, this transformation will make the loop dead. |
| 35 | // |
| 36 | // This transformation should be followed by strength reduction after all of the |
Dan Gohman | c2c4cbf | 2009-05-19 20:38:47 +0000 | [diff] [blame] | 37 | // desired loop transformations have been performed. |
Chris Lattner | 6148c02 | 2001-12-03 17:28:42 +0000 | [diff] [blame] | 38 | // |
| 39 | //===----------------------------------------------------------------------===// |
| 40 | |
Chris Lattner | 0e5f499 | 2006-12-19 21:40:18 +0000 | [diff] [blame] | 41 | #define DEBUG_TYPE "indvars" |
Chris Lattner | 022103b | 2002-05-07 20:03:00 +0000 | [diff] [blame] | 42 | #include "llvm/Transforms/Scalar.h" |
Chris Lattner | 40bf8b4 | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 43 | #include "llvm/BasicBlock.h" |
Chris Lattner | 59fdaee | 2004-04-15 15:21:43 +0000 | [diff] [blame] | 44 | #include "llvm/Constants.h" |
Chris Lattner | 18b3c97 | 2003-12-22 05:02:01 +0000 | [diff] [blame] | 45 | #include "llvm/Instructions.h" |
Chris Lattner | 40bf8b4 | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 46 | #include "llvm/Type.h" |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 47 | #include "llvm/Analysis/Dominators.h" |
| 48 | #include "llvm/Analysis/IVUsers.h" |
Nate Begeman | 36f891b | 2005-07-30 00:12:19 +0000 | [diff] [blame] | 49 | #include "llvm/Analysis/ScalarEvolutionExpander.h" |
John Criswell | 47df12d | 2003-12-18 17:19:19 +0000 | [diff] [blame] | 50 | #include "llvm/Analysis/LoopInfo.h" |
Devang Patel | 5ee9997 | 2007-03-07 06:39:01 +0000 | [diff] [blame] | 51 | #include "llvm/Analysis/LoopPass.h" |
Chris Lattner | 455889a | 2002-02-12 22:39:50 +0000 | [diff] [blame] | 52 | #include "llvm/Support/CFG.h" |
Reid Spencer | 9133fe2 | 2007-02-05 23:32:05 +0000 | [diff] [blame] | 53 | #include "llvm/Support/Compiler.h" |
Chris Lattner | ee4f13a | 2007-01-07 01:14:12 +0000 | [diff] [blame] | 54 | #include "llvm/Support/Debug.h" |
John Criswell | 47df12d | 2003-12-18 17:19:19 +0000 | [diff] [blame] | 55 | #include "llvm/Transforms/Utils/Local.h" |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 56 | #include "llvm/Transforms/Utils/BasicBlockUtils.h" |
Reid Spencer | 551ccae | 2004-09-01 22:55:40 +0000 | [diff] [blame] | 57 | #include "llvm/Support/CommandLine.h" |
Reid Spencer | a54b7cb | 2007-01-12 07:05:14 +0000 | [diff] [blame] | 58 | #include "llvm/ADT/SmallVector.h" |
Reid Spencer | 551ccae | 2004-09-01 22:55:40 +0000 | [diff] [blame] | 59 | #include "llvm/ADT/Statistic.h" |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 60 | #include "llvm/ADT/STLExtras.h" |
John Criswell | 47df12d | 2003-12-18 17:19:19 +0000 | [diff] [blame] | 61 | using namespace llvm; |
Brian Gaeke | d0fde30 | 2003-11-11 22:41:34 +0000 | [diff] [blame] | 62 | |
Chris Lattner | 0e5f499 | 2006-12-19 21:40:18 +0000 | [diff] [blame] | 63 | STATISTIC(NumRemoved , "Number of aux indvars removed"); |
Chris Lattner | 0e5f499 | 2006-12-19 21:40:18 +0000 | [diff] [blame] | 64 | STATISTIC(NumInserted, "Number of canonical indvars added"); |
| 65 | STATISTIC(NumReplaced, "Number of exit values replaced"); |
| 66 | STATISTIC(NumLFTR , "Number of loop exit tests replaced"); |
Chris Lattner | 3324e71 | 2003-12-22 03:58:44 +0000 | [diff] [blame] | 67 | |
Chris Lattner | 0e5f499 | 2006-12-19 21:40:18 +0000 | [diff] [blame] | 68 | namespace { |
Devang Patel | 5ee9997 | 2007-03-07 06:39:01 +0000 | [diff] [blame] | 69 | class VISIBILITY_HIDDEN IndVarSimplify : public LoopPass { |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 70 | IVUsers *IU; |
Chris Lattner | 40bf8b4 | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 71 | LoopInfo *LI; |
| 72 | ScalarEvolution *SE; |
Dan Gohman | de53dc0 | 2009-06-27 05:16:57 +0000 | [diff] [blame^] | 73 | DominatorTree *DT; |
Chris Lattner | 15cad75 | 2003-12-23 07:47:09 +0000 | [diff] [blame] | 74 | bool Changed; |
Chris Lattner | 3324e71 | 2003-12-22 03:58:44 +0000 | [diff] [blame] | 75 | public: |
Devang Patel | 794fd75 | 2007-05-01 21:15:47 +0000 | [diff] [blame] | 76 | |
Nick Lewycky | ecd94c8 | 2007-05-06 13:37:16 +0000 | [diff] [blame] | 77 | static char ID; // Pass identification, replacement for typeid |
Dan Gohman | ae73dc1 | 2008-09-04 17:05:41 +0000 | [diff] [blame] | 78 | IndVarSimplify() : LoopPass(&ID) {} |
Devang Patel | 794fd75 | 2007-05-01 21:15:47 +0000 | [diff] [blame] | 79 | |
Dan Gohman | 60f8a63 | 2009-02-17 20:49:49 +0000 | [diff] [blame] | 80 | virtual bool runOnLoop(Loop *L, LPPassManager &LPM); |
| 81 | |
Devang Patel | 5ee9997 | 2007-03-07 06:39:01 +0000 | [diff] [blame] | 82 | virtual void getAnalysisUsage(AnalysisUsage &AU) const { |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 83 | AU.addRequired<DominatorTree>(); |
Devang Patel | bc533cd | 2007-09-10 18:08:23 +0000 | [diff] [blame] | 84 | AU.addRequired<ScalarEvolution>(); |
Devang Patel | 5ee9997 | 2007-03-07 06:39:01 +0000 | [diff] [blame] | 85 | AU.addRequiredID(LCSSAID); |
| 86 | AU.addRequiredID(LoopSimplifyID); |
Devang Patel | 5ee9997 | 2007-03-07 06:39:01 +0000 | [diff] [blame] | 87 | AU.addRequired<LoopInfo>(); |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 88 | AU.addRequired<IVUsers>(); |
Dan Gohman | 474cecf | 2009-02-23 16:29:41 +0000 | [diff] [blame] | 89 | AU.addPreserved<ScalarEvolution>(); |
Devang Patel | 5ee9997 | 2007-03-07 06:39:01 +0000 | [diff] [blame] | 90 | AU.addPreservedID(LoopSimplifyID); |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 91 | AU.addPreserved<IVUsers>(); |
Devang Patel | 5ee9997 | 2007-03-07 06:39:01 +0000 | [diff] [blame] | 92 | AU.addPreservedID(LCSSAID); |
| 93 | AU.setPreservesCFG(); |
| 94 | } |
Chris Lattner | 15cad75 | 2003-12-23 07:47:09 +0000 | [diff] [blame] | 95 | |
Chris Lattner | 40bf8b4 | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 96 | private: |
Devang Patel | 5ee9997 | 2007-03-07 06:39:01 +0000 | [diff] [blame] | 97 | |
Dan Gohman | 60f8a63 | 2009-02-17 20:49:49 +0000 | [diff] [blame] | 98 | void RewriteNonIntegerIVs(Loop *L); |
| 99 | |
Owen Anderson | 372b46c | 2009-06-22 21:39:50 +0000 | [diff] [blame] | 100 | ICmpInst *LinearFunctionTestReplace(Loop *L, const SCEV* BackedgeTakenCount, |
Dan Gohman | a575871 | 2009-02-17 15:57:39 +0000 | [diff] [blame] | 101 | Value *IndVar, |
Dan Gohman | c2390b1 | 2009-02-12 22:19:27 +0000 | [diff] [blame] | 102 | BasicBlock *ExitingBlock, |
| 103 | BranchInst *BI, |
Dan Gohman | 15cab28 | 2009-02-23 23:20:35 +0000 | [diff] [blame] | 104 | SCEVExpander &Rewriter); |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 105 | void RewriteLoopExitValues(Loop *L, const SCEV *BackedgeTakenCount, |
| 106 | SCEVExpander &Rewriter); |
Chris Lattner | 40bf8b4 | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 107 | |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 108 | void RewriteIVExpressions(Loop *L, const Type *LargestType, |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 109 | SCEVExpander &Rewriter); |
Devang Patel | d22a849 | 2008-09-09 21:41:07 +0000 | [diff] [blame] | 110 | |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 111 | void SinkUnusedInvariants(Loop *L); |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 112 | |
| 113 | void HandleFloatingPointIV(Loop *L, PHINode *PH); |
Chris Lattner | 3324e71 | 2003-12-22 03:58:44 +0000 | [diff] [blame] | 114 | }; |
Chris Lattner | 5e76140 | 2002-09-10 05:24:05 +0000 | [diff] [blame] | 115 | } |
Chris Lattner | 394437f | 2001-12-04 04:32:29 +0000 | [diff] [blame] | 116 | |
Dan Gohman | 844731a | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 117 | char IndVarSimplify::ID = 0; |
| 118 | static RegisterPass<IndVarSimplify> |
| 119 | X("indvars", "Canonicalize Induction Variables"); |
| 120 | |
Daniel Dunbar | 394f044 | 2008-10-22 23:32:42 +0000 | [diff] [blame] | 121 | Pass *llvm::createIndVarSimplifyPass() { |
Chris Lattner | 3324e71 | 2003-12-22 03:58:44 +0000 | [diff] [blame] | 122 | return new IndVarSimplify(); |
Chris Lattner | 394437f | 2001-12-04 04:32:29 +0000 | [diff] [blame] | 123 | } |
| 124 | |
Chris Lattner | 40bf8b4 | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 125 | /// LinearFunctionTestReplace - This method rewrites the exit condition of the |
Chris Lattner | 59fdaee | 2004-04-15 15:21:43 +0000 | [diff] [blame] | 126 | /// loop to be a canonical != comparison against the incremented loop induction |
| 127 | /// variable. This pass is able to rewrite the exit tests of any loop where the |
| 128 | /// SCEV analysis can determine a loop-invariant trip count of the loop, which |
| 129 | /// is actually a much broader range than just linear tests. |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 130 | ICmpInst *IndVarSimplify::LinearFunctionTestReplace(Loop *L, |
Owen Anderson | 372b46c | 2009-06-22 21:39:50 +0000 | [diff] [blame] | 131 | const SCEV* BackedgeTakenCount, |
Dan Gohman | c2390b1 | 2009-02-12 22:19:27 +0000 | [diff] [blame] | 132 | Value *IndVar, |
| 133 | BasicBlock *ExitingBlock, |
| 134 | BranchInst *BI, |
Dan Gohman | 15cab28 | 2009-02-23 23:20:35 +0000 | [diff] [blame] | 135 | SCEVExpander &Rewriter) { |
Chris Lattner | d244057 | 2004-04-15 20:26:22 +0000 | [diff] [blame] | 136 | // If the exiting block is not the same as the backedge block, we must compare |
| 137 | // against the preincremented value, otherwise we prefer to compare against |
| 138 | // the post-incremented value. |
Dan Gohman | c2390b1 | 2009-02-12 22:19:27 +0000 | [diff] [blame] | 139 | Value *CmpIndVar; |
Owen Anderson | 372b46c | 2009-06-22 21:39:50 +0000 | [diff] [blame] | 140 | const SCEV* RHS = BackedgeTakenCount; |
Dan Gohman | c2390b1 | 2009-02-12 22:19:27 +0000 | [diff] [blame] | 141 | if (ExitingBlock == L->getLoopLatch()) { |
Dan Gohman | 46bdfb0 | 2009-02-24 18:55:53 +0000 | [diff] [blame] | 142 | // Add one to the "backedge-taken" count to get the trip count. |
| 143 | // If this addition may overflow, we have to be more pessimistic and |
| 144 | // cast the induction variable before doing the add. |
Owen Anderson | 372b46c | 2009-06-22 21:39:50 +0000 | [diff] [blame] | 145 | const SCEV* Zero = SE->getIntegerSCEV(0, BackedgeTakenCount->getType()); |
| 146 | const SCEV* N = |
Dan Gohman | 46bdfb0 | 2009-02-24 18:55:53 +0000 | [diff] [blame] | 147 | SE->getAddExpr(BackedgeTakenCount, |
| 148 | SE->getIntegerSCEV(1, BackedgeTakenCount->getType())); |
Dan Gohman | c2390b1 | 2009-02-12 22:19:27 +0000 | [diff] [blame] | 149 | if ((isa<SCEVConstant>(N) && !N->isZero()) || |
| 150 | SE->isLoopGuardedByCond(L, ICmpInst::ICMP_NE, N, Zero)) { |
| 151 | // No overflow. Cast the sum. |
Dan Gohman | 46bdfb0 | 2009-02-24 18:55:53 +0000 | [diff] [blame] | 152 | RHS = SE->getTruncateOrZeroExtend(N, IndVar->getType()); |
Dan Gohman | c2390b1 | 2009-02-12 22:19:27 +0000 | [diff] [blame] | 153 | } else { |
| 154 | // Potential overflow. Cast before doing the add. |
Dan Gohman | 46bdfb0 | 2009-02-24 18:55:53 +0000 | [diff] [blame] | 155 | RHS = SE->getTruncateOrZeroExtend(BackedgeTakenCount, |
| 156 | IndVar->getType()); |
| 157 | RHS = SE->getAddExpr(RHS, |
| 158 | SE->getIntegerSCEV(1, IndVar->getType())); |
Dan Gohman | c2390b1 | 2009-02-12 22:19:27 +0000 | [diff] [blame] | 159 | } |
Chris Lattner | 59fdaee | 2004-04-15 15:21:43 +0000 | [diff] [blame] | 160 | |
Dan Gohman | 46bdfb0 | 2009-02-24 18:55:53 +0000 | [diff] [blame] | 161 | // The BackedgeTaken expression contains the number of times that the |
| 162 | // backedge branches to the loop header. This is one less than the |
| 163 | // number of times the loop executes, so use the incremented indvar. |
Dan Gohman | c2390b1 | 2009-02-12 22:19:27 +0000 | [diff] [blame] | 164 | CmpIndVar = L->getCanonicalInductionVariableIncrement(); |
Chris Lattner | d244057 | 2004-04-15 20:26:22 +0000 | [diff] [blame] | 165 | } else { |
| 166 | // We have to use the preincremented value... |
Dan Gohman | 46bdfb0 | 2009-02-24 18:55:53 +0000 | [diff] [blame] | 167 | RHS = SE->getTruncateOrZeroExtend(BackedgeTakenCount, |
| 168 | IndVar->getType()); |
Dan Gohman | c2390b1 | 2009-02-12 22:19:27 +0000 | [diff] [blame] | 169 | CmpIndVar = IndVar; |
Chris Lattner | d244057 | 2004-04-15 20:26:22 +0000 | [diff] [blame] | 170 | } |
Chris Lattner | 59fdaee | 2004-04-15 15:21:43 +0000 | [diff] [blame] | 171 | |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 172 | // Expand the code for the iteration count. |
Dan Gohman | 40a5a1b | 2009-06-24 01:18:18 +0000 | [diff] [blame] | 173 | assert(RHS->isLoopInvariant(L) && |
| 174 | "Computed iteration count is not loop invariant!"); |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 175 | Value *ExitCnt = Rewriter.expandCodeFor(RHS, IndVar->getType(), BI); |
Chris Lattner | 40bf8b4 | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 176 | |
Reid Spencer | e4d87aa | 2006-12-23 06:05:41 +0000 | [diff] [blame] | 177 | // Insert a new icmp_ne or icmp_eq instruction before the branch. |
| 178 | ICmpInst::Predicate Opcode; |
Chris Lattner | 40bf8b4 | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 179 | if (L->contains(BI->getSuccessor(0))) |
Reid Spencer | e4d87aa | 2006-12-23 06:05:41 +0000 | [diff] [blame] | 180 | Opcode = ICmpInst::ICMP_NE; |
Chris Lattner | 40bf8b4 | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 181 | else |
Reid Spencer | e4d87aa | 2006-12-23 06:05:41 +0000 | [diff] [blame] | 182 | Opcode = ICmpInst::ICMP_EQ; |
Chris Lattner | 40bf8b4 | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 183 | |
Dan Gohman | c2390b1 | 2009-02-12 22:19:27 +0000 | [diff] [blame] | 184 | DOUT << "INDVARS: Rewriting loop exit condition to:\n" |
| 185 | << " LHS:" << *CmpIndVar // includes a newline |
| 186 | << " op:\t" |
Dan Gohman | f108e2e | 2009-02-14 02:26:50 +0000 | [diff] [blame] | 187 | << (Opcode == ICmpInst::ICMP_NE ? "!=" : "==") << "\n" |
Dan Gohman | 46bdfb0 | 2009-02-24 18:55:53 +0000 | [diff] [blame] | 188 | << " RHS:\t" << *RHS << "\n"; |
Dan Gohman | c2390b1 | 2009-02-12 22:19:27 +0000 | [diff] [blame] | 189 | |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 190 | ICmpInst *Cond = new ICmpInst(Opcode, CmpIndVar, ExitCnt, "exitcond", BI); |
| 191 | |
| 192 | Instruction *OrigCond = cast<Instruction>(BI->getCondition()); |
Dan Gohman | 95bdbfa | 2009-05-24 19:11:38 +0000 | [diff] [blame] | 193 | // It's tempting to use replaceAllUsesWith here to fully replace the old |
| 194 | // comparison, but that's not immediately safe, since users of the old |
| 195 | // comparison may not be dominated by the new comparison. Instead, just |
| 196 | // update the branch to use the new comparison; in the common case this |
| 197 | // will make old comparison dead. |
| 198 | BI->setCondition(Cond); |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 199 | RecursivelyDeleteTriviallyDeadInstructions(OrigCond); |
| 200 | |
Chris Lattner | 40bf8b4 | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 201 | ++NumLFTR; |
| 202 | Changed = true; |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 203 | return Cond; |
Chris Lattner | 40bf8b4 | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 204 | } |
| 205 | |
Chris Lattner | 40bf8b4 | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 206 | /// RewriteLoopExitValues - Check to see if this loop has a computable |
| 207 | /// loop-invariant execution count. If so, this means that we can compute the |
| 208 | /// final value of any expressions that are recurrent in the loop, and |
| 209 | /// substitute the exit values from the loop into any instructions outside of |
| 210 | /// the loop that use the final values of the current expressions. |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 211 | /// |
| 212 | /// This is mostly redundant with the regular IndVarSimplify activities that |
| 213 | /// happen later, except that it's more powerful in some cases, because it's |
| 214 | /// able to brute-force evaluate arbitrary instructions as long as they have |
| 215 | /// constant operands at the beginning of the loop. |
Dan Gohman | 890f92b | 2009-04-18 17:56:28 +0000 | [diff] [blame] | 216 | void IndVarSimplify::RewriteLoopExitValues(Loop *L, |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 217 | const SCEV *BackedgeTakenCount, |
| 218 | SCEVExpander &Rewriter) { |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 219 | // Verify the input to the pass in already in LCSSA form. |
| 220 | assert(L->isLCSSAForm()); |
| 221 | |
Devang Patel | b7211a2 | 2007-08-21 00:31:24 +0000 | [diff] [blame] | 222 | SmallVector<BasicBlock*, 8> ExitBlocks; |
Chris Lattner | 9f3d738 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 223 | L->getUniqueExitBlocks(ExitBlocks); |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 224 | |
Chris Lattner | 9f3d738 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 225 | // Find all values that are computed inside the loop, but used outside of it. |
| 226 | // Because of LCSSA, these values will only occur in LCSSA PHI Nodes. Scan |
| 227 | // the exit blocks of the loop to find them. |
| 228 | for (unsigned i = 0, e = ExitBlocks.size(); i != e; ++i) { |
| 229 | BasicBlock *ExitBB = ExitBlocks[i]; |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 230 | |
Chris Lattner | 9f3d738 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 231 | // If there are no PHI nodes in this exit block, then no values defined |
| 232 | // inside the loop are used on this path, skip it. |
| 233 | PHINode *PN = dyn_cast<PHINode>(ExitBB->begin()); |
| 234 | if (!PN) continue; |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 235 | |
Chris Lattner | 9f3d738 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 236 | unsigned NumPreds = PN->getNumIncomingValues(); |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 237 | |
Chris Lattner | 9f3d738 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 238 | // Iterate over all of the PHI nodes. |
| 239 | BasicBlock::iterator BBI = ExitBB->begin(); |
| 240 | while ((PN = dyn_cast<PHINode>(BBI++))) { |
Torok Edwin | 3790fb0 | 2009-05-24 19:36:09 +0000 | [diff] [blame] | 241 | if (PN->use_empty()) |
| 242 | continue; // dead use, don't replace it |
Chris Lattner | 9f3d738 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 243 | // Iterate over all of the values in all the PHI nodes. |
| 244 | for (unsigned i = 0; i != NumPreds; ++i) { |
| 245 | // If the value being merged in is not integer or is not defined |
| 246 | // in the loop, skip it. |
| 247 | Value *InVal = PN->getIncomingValue(i); |
| 248 | if (!isa<Instruction>(InVal) || |
| 249 | // SCEV only supports integer expressions for now. |
Dan Gohman | 2d1be87 | 2009-04-16 03:18:22 +0000 | [diff] [blame] | 250 | (!isa<IntegerType>(InVal->getType()) && |
| 251 | !isa<PointerType>(InVal->getType()))) |
Chris Lattner | 9f3d738 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 252 | continue; |
Chris Lattner | 40bf8b4 | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 253 | |
Chris Lattner | 9f3d738 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 254 | // If this pred is for a subloop, not L itself, skip it. |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 255 | if (LI->getLoopFor(PN->getIncomingBlock(i)) != L) |
Chris Lattner | 9f3d738 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 256 | continue; // The Block is in a subloop, skip it. |
| 257 | |
| 258 | // Check that InVal is defined in the loop. |
| 259 | Instruction *Inst = cast<Instruction>(InVal); |
| 260 | if (!L->contains(Inst->getParent())) |
| 261 | continue; |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 262 | |
Chris Lattner | 9f3d738 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 263 | // Okay, this instruction has a user outside of the current loop |
| 264 | // and varies predictably *inside* the loop. Evaluate the value it |
| 265 | // contains when the loop exits, if possible. |
Owen Anderson | 372b46c | 2009-06-22 21:39:50 +0000 | [diff] [blame] | 266 | const SCEV* ExitValue = SE->getSCEVAtScope(Inst, L->getParentLoop()); |
Dan Gohman | d594e6f | 2009-05-24 23:25:42 +0000 | [diff] [blame] | 267 | if (!ExitValue->isLoopInvariant(L)) |
Chris Lattner | 9f3d738 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 268 | continue; |
Chris Lattner | 9caed54 | 2007-03-04 01:00:28 +0000 | [diff] [blame] | 269 | |
Chris Lattner | 9f3d738 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 270 | Changed = true; |
| 271 | ++NumReplaced; |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 272 | |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 273 | Value *ExitVal = Rewriter.expandCodeFor(ExitValue, PN->getType(), Inst); |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 274 | |
Chris Lattner | 9f3d738 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 275 | DOUT << "INDVARS: RLEV: AfterLoopVal = " << *ExitVal |
| 276 | << " LoopVal = " << *Inst << "\n"; |
| 277 | |
| 278 | PN->setIncomingValue(i, ExitVal); |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 279 | |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 280 | // If this instruction is dead now, delete it. |
| 281 | RecursivelyDeleteTriviallyDeadInstructions(Inst); |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 282 | |
Dan Gohman | 03ad698 | 2009-06-22 00:15:15 +0000 | [diff] [blame] | 283 | // If we're inserting code into the exit block rather than the |
| 284 | // preheader, we can (and have to) remove the PHI entirely. |
| 285 | // This is safe, because the NewVal won't be variant |
Chris Lattner | 9caed54 | 2007-03-04 01:00:28 +0000 | [diff] [blame] | 286 | // in the loop, so we don't need an LCSSA phi node anymore. |
Dan Gohman | 03ad698 | 2009-06-22 00:15:15 +0000 | [diff] [blame] | 287 | if (ExitBlocks.size() == 1) { |
Chris Lattner | 9f3d738 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 288 | PN->replaceAllUsesWith(ExitVal); |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 289 | RecursivelyDeleteTriviallyDeadInstructions(PN); |
Chris Lattner | 9f3d738 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 290 | break; |
Chris Lattner | c9838f2 | 2007-03-03 22:48:48 +0000 | [diff] [blame] | 291 | } |
| 292 | } |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 293 | if (ExitBlocks.size() != 1) { |
| 294 | // Clone the PHI and delete the original one. This lets IVUsers and |
| 295 | // any other maps purge the original user from their records. |
| 296 | PHINode *NewPN = PN->clone(); |
| 297 | NewPN->takeName(PN); |
| 298 | NewPN->insertBefore(PN); |
| 299 | PN->replaceAllUsesWith(NewPN); |
| 300 | PN->eraseFromParent(); |
| 301 | } |
Chris Lattner | c9838f2 | 2007-03-03 22:48:48 +0000 | [diff] [blame] | 302 | } |
| 303 | } |
Chris Lattner | 40bf8b4 | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 304 | } |
| 305 | |
Dan Gohman | 60f8a63 | 2009-02-17 20:49:49 +0000 | [diff] [blame] | 306 | void IndVarSimplify::RewriteNonIntegerIVs(Loop *L) { |
Dan Gohman | 2d1be87 | 2009-04-16 03:18:22 +0000 | [diff] [blame] | 307 | // First step. Check to see if there are any floating-point recurrences. |
Chris Lattner | 40bf8b4 | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 308 | // If there are, change them into integer recurrences, permitting analysis by |
| 309 | // the SCEV routines. |
| 310 | // |
| 311 | BasicBlock *Header = L->getHeader(); |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 312 | |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 313 | SmallVector<WeakVH, 8> PHIs; |
| 314 | for (BasicBlock::iterator I = Header->begin(); |
| 315 | PHINode *PN = dyn_cast<PHINode>(I); ++I) |
| 316 | PHIs.push_back(PN); |
| 317 | |
| 318 | for (unsigned i = 0, e = PHIs.size(); i != e; ++i) |
| 319 | if (PHINode *PN = dyn_cast_or_null<PHINode>(PHIs[i])) |
| 320 | HandleFloatingPointIV(L, PN); |
Chris Lattner | 40bf8b4 | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 321 | |
Dan Gohman | 2d1be87 | 2009-04-16 03:18:22 +0000 | [diff] [blame] | 322 | // If the loop previously had floating-point IV, ScalarEvolution |
Dan Gohman | 60f8a63 | 2009-02-17 20:49:49 +0000 | [diff] [blame] | 323 | // may not have been able to compute a trip count. Now that we've done some |
| 324 | // re-writing, the trip count may be computable. |
| 325 | if (Changed) |
Dan Gohman | 46bdfb0 | 2009-02-24 18:55:53 +0000 | [diff] [blame] | 326 | SE->forgetLoopBackedgeTakenCount(L); |
Dale Johannesen | c671d89 | 2009-04-15 23:31:51 +0000 | [diff] [blame] | 327 | } |
| 328 | |
Dan Gohman | c2390b1 | 2009-02-12 22:19:27 +0000 | [diff] [blame] | 329 | bool IndVarSimplify::runOnLoop(Loop *L, LPPassManager &LPM) { |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 330 | IU = &getAnalysis<IVUsers>(); |
Devang Patel | 5ee9997 | 2007-03-07 06:39:01 +0000 | [diff] [blame] | 331 | LI = &getAnalysis<LoopInfo>(); |
| 332 | SE = &getAnalysis<ScalarEvolution>(); |
Dan Gohman | de53dc0 | 2009-06-27 05:16:57 +0000 | [diff] [blame^] | 333 | DT = &getAnalysis<DominatorTree>(); |
Devang Patel | 5ee9997 | 2007-03-07 06:39:01 +0000 | [diff] [blame] | 334 | Changed = false; |
Dan Gohman | 60f8a63 | 2009-02-17 20:49:49 +0000 | [diff] [blame] | 335 | |
Dan Gohman | 2d1be87 | 2009-04-16 03:18:22 +0000 | [diff] [blame] | 336 | // If there are any floating-point recurrences, attempt to |
Dan Gohman | 60f8a63 | 2009-02-17 20:49:49 +0000 | [diff] [blame] | 337 | // transform them to use integer recurrences. |
| 338 | RewriteNonIntegerIVs(L); |
| 339 | |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 340 | BasicBlock *ExitingBlock = L->getExitingBlock(); // may be null |
Owen Anderson | 372b46c | 2009-06-22 21:39:50 +0000 | [diff] [blame] | 341 | const SCEV* BackedgeTakenCount = SE->getBackedgeTakenCount(L); |
Chris Lattner | 9caed54 | 2007-03-04 01:00:28 +0000 | [diff] [blame] | 342 | |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 343 | // Create a rewriter object which we'll use to transform the code with. |
| 344 | SCEVExpander Rewriter(*SE); |
| 345 | |
Chris Lattner | 40bf8b4 | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 346 | // Check to see if this loop has a computable loop-invariant execution count. |
| 347 | // If so, this means that we can compute the final value of any expressions |
| 348 | // that are recurrent in the loop, and substitute the exit values from the |
| 349 | // loop into any instructions outside of the loop that use the final values of |
| 350 | // the current expressions. |
Chris Lattner | 3dec1f2 | 2002-05-10 15:38:35 +0000 | [diff] [blame] | 351 | // |
Dan Gohman | 46bdfb0 | 2009-02-24 18:55:53 +0000 | [diff] [blame] | 352 | if (!isa<SCEVCouldNotCompute>(BackedgeTakenCount)) |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 353 | RewriteLoopExitValues(L, BackedgeTakenCount, Rewriter); |
Chris Lattner | 6148c02 | 2001-12-03 17:28:42 +0000 | [diff] [blame] | 354 | |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 355 | // Compute the type of the largest recurrence expression, and decide whether |
| 356 | // a canonical induction variable should be inserted. |
Dan Gohman | c2390b1 | 2009-02-12 22:19:27 +0000 | [diff] [blame] | 357 | const Type *LargestType = 0; |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 358 | bool NeedCannIV = false; |
Dan Gohman | 46bdfb0 | 2009-02-24 18:55:53 +0000 | [diff] [blame] | 359 | if (!isa<SCEVCouldNotCompute>(BackedgeTakenCount)) { |
| 360 | LargestType = BackedgeTakenCount->getType(); |
Dan Gohman | af79fb5 | 2009-04-21 01:07:12 +0000 | [diff] [blame] | 361 | LargestType = SE->getEffectiveSCEVType(LargestType); |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 362 | // If we have a known trip count and a single exit block, we'll be |
| 363 | // rewriting the loop exit test condition below, which requires a |
| 364 | // canonical induction variable. |
| 365 | if (ExitingBlock) |
| 366 | NeedCannIV = true; |
Chris Lattner | f50af08 | 2004-04-17 18:08:33 +0000 | [diff] [blame] | 367 | } |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 368 | for (unsigned i = 0, e = IU->StrideOrder.size(); i != e; ++i) { |
Owen Anderson | 372b46c | 2009-06-22 21:39:50 +0000 | [diff] [blame] | 369 | const SCEV* Stride = IU->StrideOrder[i]; |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 370 | const Type *Ty = SE->getEffectiveSCEVType(Stride->getType()); |
Dan Gohman | c2390b1 | 2009-02-12 22:19:27 +0000 | [diff] [blame] | 371 | if (!LargestType || |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 372 | SE->getTypeSizeInBits(Ty) > |
Dan Gohman | af79fb5 | 2009-04-21 01:07:12 +0000 | [diff] [blame] | 373 | SE->getTypeSizeInBits(LargestType)) |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 374 | LargestType = Ty; |
| 375 | |
Owen Anderson | 372b46c | 2009-06-22 21:39:50 +0000 | [diff] [blame] | 376 | std::map<const SCEV*, IVUsersOfOneStride *>::iterator SI = |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 377 | IU->IVUsesByStride.find(IU->StrideOrder[i]); |
| 378 | assert(SI != IU->IVUsesByStride.end() && "Stride doesn't exist!"); |
| 379 | |
| 380 | if (!SI->second->Users.empty()) |
| 381 | NeedCannIV = true; |
Chris Lattner | 6148c02 | 2001-12-03 17:28:42 +0000 | [diff] [blame] | 382 | } |
| 383 | |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 384 | // Now that we know the largest of of the induction variable expressions |
| 385 | // in this loop, insert a canonical induction variable of the largest size. |
Dan Gohman | c2390b1 | 2009-02-12 22:19:27 +0000 | [diff] [blame] | 386 | Value *IndVar = 0; |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 387 | if (NeedCannIV) { |
Dan Gohman | 4d8414f | 2009-06-13 16:25:49 +0000 | [diff] [blame] | 388 | // Check to see if the loop already has a canonical-looking induction |
| 389 | // variable. If one is present and it's wider than the planned canonical |
| 390 | // induction variable, temporarily remove it, so that the Rewriter |
| 391 | // doesn't attempt to reuse it. |
| 392 | PHINode *OldCannIV = L->getCanonicalInductionVariable(); |
| 393 | if (OldCannIV) { |
| 394 | if (SE->getTypeSizeInBits(OldCannIV->getType()) > |
| 395 | SE->getTypeSizeInBits(LargestType)) |
| 396 | OldCannIV->removeFromParent(); |
| 397 | else |
| 398 | OldCannIV = 0; |
| 399 | } |
| 400 | |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 401 | IndVar = Rewriter.getOrInsertCanonicalInductionVariable(L, LargestType); |
Dan Gohman | 4d8414f | 2009-06-13 16:25:49 +0000 | [diff] [blame] | 402 | |
Dan Gohman | c2390b1 | 2009-02-12 22:19:27 +0000 | [diff] [blame] | 403 | ++NumInserted; |
| 404 | Changed = true; |
| 405 | DOUT << "INDVARS: New CanIV: " << *IndVar; |
Dan Gohman | 4d8414f | 2009-06-13 16:25:49 +0000 | [diff] [blame] | 406 | |
| 407 | // Now that the official induction variable is established, reinsert |
| 408 | // the old canonical-looking variable after it so that the IR remains |
| 409 | // consistent. It will be deleted as part of the dead-PHI deletion at |
| 410 | // the end of the pass. |
| 411 | if (OldCannIV) |
| 412 | OldCannIV->insertAfter(cast<Instruction>(IndVar)); |
Dan Gohman | d19534a | 2007-06-15 14:38:12 +0000 | [diff] [blame] | 413 | } |
Chris Lattner | 15cad75 | 2003-12-23 07:47:09 +0000 | [diff] [blame] | 414 | |
Dan Gohman | c2390b1 | 2009-02-12 22:19:27 +0000 | [diff] [blame] | 415 | // If we have a trip count expression, rewrite the loop's exit condition |
| 416 | // using it. We can currently only handle loops with a single exit. |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 417 | ICmpInst *NewICmp = 0; |
| 418 | if (!isa<SCEVCouldNotCompute>(BackedgeTakenCount) && ExitingBlock) { |
| 419 | assert(NeedCannIV && |
| 420 | "LinearFunctionTestReplace requires a canonical induction variable"); |
Dan Gohman | c2390b1 | 2009-02-12 22:19:27 +0000 | [diff] [blame] | 421 | // Can't rewrite non-branch yet. |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 422 | if (BranchInst *BI = dyn_cast<BranchInst>(ExitingBlock->getTerminator())) |
| 423 | NewICmp = LinearFunctionTestReplace(L, BackedgeTakenCount, IndVar, |
| 424 | ExitingBlock, BI, Rewriter); |
Chris Lattner | fcb81f5 | 2004-04-22 14:59:40 +0000 | [diff] [blame] | 425 | } |
| 426 | |
Torok Edwin | 3d43138 | 2009-05-24 20:08:21 +0000 | [diff] [blame] | 427 | // Rewrite IV-derived expressions. Clears the rewriter cache. |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 428 | RewriteIVExpressions(L, LargestType, Rewriter); |
Dan Gohman | c2390b1 | 2009-02-12 22:19:27 +0000 | [diff] [blame] | 429 | |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 430 | // The Rewriter may not be used from this point on. |
Torok Edwin | 3d43138 | 2009-05-24 20:08:21 +0000 | [diff] [blame] | 431 | |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 432 | // Loop-invariant instructions in the preheader that aren't used in the |
| 433 | // loop may be sunk below the loop to reduce register pressure. |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 434 | SinkUnusedInvariants(L); |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 435 | |
| 436 | // For completeness, inform IVUsers of the IV use in the newly-created |
| 437 | // loop exit test instruction. |
| 438 | if (NewICmp) |
| 439 | IU->AddUsersIfInteresting(cast<Instruction>(NewICmp->getOperand(0))); |
| 440 | |
| 441 | // Clean up dead instructions. |
| 442 | DeleteDeadPHIs(L->getHeader()); |
| 443 | // Check a post-condition. |
| 444 | assert(L->isLCSSAForm() && "Indvars did not leave the loop in lcssa form!"); |
Devang Patel | 5ee9997 | 2007-03-07 06:39:01 +0000 | [diff] [blame] | 445 | return Changed; |
Chris Lattner | 6148c02 | 2001-12-03 17:28:42 +0000 | [diff] [blame] | 446 | } |
Devang Patel | d22a849 | 2008-09-09 21:41:07 +0000 | [diff] [blame] | 447 | |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 448 | void IndVarSimplify::RewriteIVExpressions(Loop *L, const Type *LargestType, |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 449 | SCEVExpander &Rewriter) { |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 450 | SmallVector<WeakVH, 16> DeadInsts; |
| 451 | |
| 452 | // Rewrite all induction variable expressions in terms of the canonical |
| 453 | // induction variable. |
| 454 | // |
| 455 | // If there were induction variables of other sizes or offsets, manually |
| 456 | // add the offsets to the primary induction variable and cast, avoiding |
| 457 | // the need for the code evaluation methods to insert induction variables |
| 458 | // of different sizes. |
| 459 | for (unsigned i = 0, e = IU->StrideOrder.size(); i != e; ++i) { |
Owen Anderson | 372b46c | 2009-06-22 21:39:50 +0000 | [diff] [blame] | 460 | const SCEV* Stride = IU->StrideOrder[i]; |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 461 | |
Owen Anderson | 372b46c | 2009-06-22 21:39:50 +0000 | [diff] [blame] | 462 | std::map<const SCEV*, IVUsersOfOneStride *>::iterator SI = |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 463 | IU->IVUsesByStride.find(IU->StrideOrder[i]); |
| 464 | assert(SI != IU->IVUsesByStride.end() && "Stride doesn't exist!"); |
| 465 | ilist<IVStrideUse> &List = SI->second->Users; |
| 466 | for (ilist<IVStrideUse>::iterator UI = List.begin(), |
| 467 | E = List.end(); UI != E; ++UI) { |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 468 | Value *Op = UI->getOperandValToReplace(); |
Dan Gohman | 4d8414f | 2009-06-13 16:25:49 +0000 | [diff] [blame] | 469 | const Type *UseTy = Op->getType(); |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 470 | Instruction *User = UI->getUser(); |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 471 | |
| 472 | // Compute the final addrec to expand into code. |
Owen Anderson | 372b46c | 2009-06-22 21:39:50 +0000 | [diff] [blame] | 473 | const SCEV* AR = IU->getReplacementExpr(*UI); |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 474 | |
Dan Gohman | 40a5a1b | 2009-06-24 01:18:18 +0000 | [diff] [blame] | 475 | // FIXME: It is an extremely bad idea to indvar substitute anything more |
| 476 | // complex than affine induction variables. Doing so will put expensive |
| 477 | // polynomial evaluations inside of the loop, and the str reduction pass |
| 478 | // currently can only reduce affine polynomials. For now just disable |
| 479 | // indvar subst on anything more complex than an affine addrec, unless |
| 480 | // it can be expanded to a trivial value. |
| 481 | if (!AR->isLoopInvariant(L) && !Stride->isLoopInvariant(L)) |
| 482 | continue; |
Dan Gohman | 68c9344 | 2009-06-03 19:11:31 +0000 | [diff] [blame] | 483 | |
Dan Gohman | de53dc0 | 2009-06-27 05:16:57 +0000 | [diff] [blame^] | 484 | // Determine the insertion point for this user. By default, insert |
| 485 | // immediately before the user. The SCEVExpander class will automatically |
| 486 | // hoist loop invariants out of the loop. For PHI nodes, there may be |
| 487 | // multiple uses, so compute the nearest common dominator for the |
| 488 | // incoming blocks. |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 489 | Instruction *InsertPt = User; |
| 490 | if (PHINode *PHI = dyn_cast<PHINode>(InsertPt)) |
Dan Gohman | de53dc0 | 2009-06-27 05:16:57 +0000 | [diff] [blame^] | 491 | for (unsigned i = 0, e = PHI->getNumIncomingValues(); i != e; ++i) |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 492 | if (PHI->getIncomingValue(i) == Op) { |
Dan Gohman | de53dc0 | 2009-06-27 05:16:57 +0000 | [diff] [blame^] | 493 | if (InsertPt == User) |
| 494 | InsertPt = PHI->getIncomingBlock(i)->getTerminator(); |
| 495 | else |
| 496 | InsertPt = |
| 497 | DT->findNearestCommonDominator(InsertPt->getParent(), |
| 498 | PHI->getIncomingBlock(i)) |
| 499 | ->getTerminator(); |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 500 | } |
| 501 | |
Dan Gohman | 40a5a1b | 2009-06-24 01:18:18 +0000 | [diff] [blame] | 502 | // Now expand it into actual Instructions and patch it into place. |
| 503 | Value *NewVal = Rewriter.expandCodeFor(AR, UseTy, InsertPt); |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 504 | |
| 505 | // Patch the new value into place. |
| 506 | if (Op->hasName()) |
| 507 | NewVal->takeName(Op); |
| 508 | User->replaceUsesOfWith(Op, NewVal); |
| 509 | UI->setOperandValToReplace(NewVal); |
| 510 | DOUT << "INDVARS: Rewrote IV '" << *AR << "' " << *Op |
| 511 | << " into = " << *NewVal << "\n"; |
| 512 | ++NumRemoved; |
| 513 | Changed = true; |
| 514 | |
| 515 | // The old value may be dead now. |
| 516 | DeadInsts.push_back(Op); |
| 517 | } |
| 518 | } |
| 519 | |
Torok Edwin | 3d43138 | 2009-05-24 20:08:21 +0000 | [diff] [blame] | 520 | // Clear the rewriter cache, because values that are in the rewriter's cache |
| 521 | // can be deleted in the loop below, causing the AssertingVH in the cache to |
| 522 | // trigger. |
| 523 | Rewriter.clear(); |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 524 | // Now that we're done iterating through lists, clean up any instructions |
| 525 | // which are now dead. |
| 526 | while (!DeadInsts.empty()) { |
| 527 | Instruction *Inst = dyn_cast_or_null<Instruction>(DeadInsts.pop_back_val()); |
| 528 | if (Inst) |
| 529 | RecursivelyDeleteTriviallyDeadInstructions(Inst); |
| 530 | } |
| 531 | } |
| 532 | |
| 533 | /// If there's a single exit block, sink any loop-invariant values that |
| 534 | /// were defined in the preheader but not used inside the loop into the |
| 535 | /// exit block to reduce register pressure in the loop. |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 536 | void IndVarSimplify::SinkUnusedInvariants(Loop *L) { |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 537 | BasicBlock *ExitBlock = L->getExitBlock(); |
| 538 | if (!ExitBlock) return; |
| 539 | |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 540 | Instruction *InsertPt = ExitBlock->getFirstNonPHI(); |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 541 | BasicBlock *Preheader = L->getLoopPreheader(); |
| 542 | BasicBlock::iterator I = Preheader->getTerminator(); |
| 543 | while (I != Preheader->begin()) { |
| 544 | --I; |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 545 | // New instructions were inserted at the end of the preheader. |
| 546 | if (isa<PHINode>(I)) |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 547 | break; |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 548 | if (I->isTrapping()) |
| 549 | continue; |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 550 | // Determine if there is a use in or before the loop (direct or |
| 551 | // otherwise). |
| 552 | bool UsedInLoop = false; |
| 553 | for (Value::use_iterator UI = I->use_begin(), UE = I->use_end(); |
| 554 | UI != UE; ++UI) { |
| 555 | BasicBlock *UseBB = cast<Instruction>(UI)->getParent(); |
| 556 | if (PHINode *P = dyn_cast<PHINode>(UI)) { |
| 557 | unsigned i = |
| 558 | PHINode::getIncomingValueNumForOperand(UI.getOperandNo()); |
| 559 | UseBB = P->getIncomingBlock(i); |
| 560 | } |
| 561 | if (UseBB == Preheader || L->contains(UseBB)) { |
| 562 | UsedInLoop = true; |
| 563 | break; |
| 564 | } |
| 565 | } |
| 566 | // If there is, the def must remain in the preheader. |
| 567 | if (UsedInLoop) |
| 568 | continue; |
| 569 | // Otherwise, sink it to the exit block. |
| 570 | Instruction *ToMove = I; |
| 571 | bool Done = false; |
| 572 | if (I != Preheader->begin()) |
| 573 | --I; |
| 574 | else |
| 575 | Done = true; |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 576 | ToMove->moveBefore(InsertPt); |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 577 | if (Done) |
| 578 | break; |
Dan Gohman | 667d787 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 579 | InsertPt = ToMove; |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 580 | } |
| 581 | } |
| 582 | |
Devang Patel | 13877bf | 2008-11-18 00:40:02 +0000 | [diff] [blame] | 583 | /// Return true if it is OK to use SIToFPInst for an inducation variable |
| 584 | /// with given inital and exit values. |
| 585 | static bool useSIToFPInst(ConstantFP &InitV, ConstantFP &ExitV, |
| 586 | uint64_t intIV, uint64_t intEV) { |
| 587 | |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 588 | if (InitV.getValueAPF().isNegative() || ExitV.getValueAPF().isNegative()) |
Devang Patel | 13877bf | 2008-11-18 00:40:02 +0000 | [diff] [blame] | 589 | return true; |
| 590 | |
| 591 | // If the iteration range can be handled by SIToFPInst then use it. |
| 592 | APInt Max = APInt::getSignedMaxValue(32); |
Dale Johannesen | bae7d6d | 2009-05-14 16:47:34 +0000 | [diff] [blame] | 593 | if (Max.getZExtValue() > static_cast<uint64_t>(abs64(intEV - intIV))) |
Devang Patel | 13877bf | 2008-11-18 00:40:02 +0000 | [diff] [blame] | 594 | return true; |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 595 | |
Devang Patel | 13877bf | 2008-11-18 00:40:02 +0000 | [diff] [blame] | 596 | return false; |
| 597 | } |
| 598 | |
| 599 | /// convertToInt - Convert APF to an integer, if possible. |
Devang Patel | cd40233 | 2008-11-17 23:27:13 +0000 | [diff] [blame] | 600 | static bool convertToInt(const APFloat &APF, uint64_t *intVal) { |
| 601 | |
| 602 | bool isExact = false; |
Evan Cheng | 794a7db | 2008-11-26 01:11:57 +0000 | [diff] [blame] | 603 | if (&APF.getSemantics() == &APFloat::PPCDoubleDouble) |
| 604 | return false; |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 605 | if (APF.convertToInteger(intVal, 32, APF.isNegative(), |
Devang Patel | cd40233 | 2008-11-17 23:27:13 +0000 | [diff] [blame] | 606 | APFloat::rmTowardZero, &isExact) |
| 607 | != APFloat::opOK) |
| 608 | return false; |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 609 | if (!isExact) |
Devang Patel | cd40233 | 2008-11-17 23:27:13 +0000 | [diff] [blame] | 610 | return false; |
| 611 | return true; |
| 612 | |
| 613 | } |
| 614 | |
Devang Patel | 58d43d4 | 2008-11-03 18:32:19 +0000 | [diff] [blame] | 615 | /// HandleFloatingPointIV - If the loop has floating induction variable |
| 616 | /// then insert corresponding integer induction variable if possible. |
Devang Patel | 84e3515 | 2008-11-17 21:32:02 +0000 | [diff] [blame] | 617 | /// For example, |
| 618 | /// for(double i = 0; i < 10000; ++i) |
| 619 | /// bar(i) |
| 620 | /// is converted into |
| 621 | /// for(int i = 0; i < 10000; ++i) |
| 622 | /// bar((double)i); |
| 623 | /// |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 624 | void IndVarSimplify::HandleFloatingPointIV(Loop *L, PHINode *PH) { |
Devang Patel | 58d43d4 | 2008-11-03 18:32:19 +0000 | [diff] [blame] | 625 | |
Devang Patel | 84e3515 | 2008-11-17 21:32:02 +0000 | [diff] [blame] | 626 | unsigned IncomingEdge = L->contains(PH->getIncomingBlock(0)); |
| 627 | unsigned BackEdge = IncomingEdge^1; |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 628 | |
Devang Patel | 84e3515 | 2008-11-17 21:32:02 +0000 | [diff] [blame] | 629 | // Check incoming value. |
Devang Patel | cd40233 | 2008-11-17 23:27:13 +0000 | [diff] [blame] | 630 | ConstantFP *InitValue = dyn_cast<ConstantFP>(PH->getIncomingValue(IncomingEdge)); |
| 631 | if (!InitValue) return; |
| 632 | uint64_t newInitValue = Type::Int32Ty->getPrimitiveSizeInBits(); |
| 633 | if (!convertToInt(InitValue->getValueAPF(), &newInitValue)) |
| 634 | return; |
| 635 | |
| 636 | // Check IV increment. Reject this PH if increement operation is not |
| 637 | // an add or increment value can not be represented by an integer. |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 638 | BinaryOperator *Incr = |
Devang Patel | 84e3515 | 2008-11-17 21:32:02 +0000 | [diff] [blame] | 639 | dyn_cast<BinaryOperator>(PH->getIncomingValue(BackEdge)); |
| 640 | if (!Incr) return; |
Dan Gohman | ae3a0be | 2009-06-04 22:49:04 +0000 | [diff] [blame] | 641 | if (Incr->getOpcode() != Instruction::FAdd) return; |
Devang Patel | 84e3515 | 2008-11-17 21:32:02 +0000 | [diff] [blame] | 642 | ConstantFP *IncrValue = NULL; |
| 643 | unsigned IncrVIndex = 1; |
| 644 | if (Incr->getOperand(1) == PH) |
| 645 | IncrVIndex = 0; |
| 646 | IncrValue = dyn_cast<ConstantFP>(Incr->getOperand(IncrVIndex)); |
| 647 | if (!IncrValue) return; |
Devang Patel | cd40233 | 2008-11-17 23:27:13 +0000 | [diff] [blame] | 648 | uint64_t newIncrValue = Type::Int32Ty->getPrimitiveSizeInBits(); |
| 649 | if (!convertToInt(IncrValue->getValueAPF(), &newIncrValue)) |
| 650 | return; |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 651 | |
Devang Patel | cd40233 | 2008-11-17 23:27:13 +0000 | [diff] [blame] | 652 | // Check Incr uses. One user is PH and the other users is exit condition used |
| 653 | // by the conditional terminator. |
Devang Patel | 84e3515 | 2008-11-17 21:32:02 +0000 | [diff] [blame] | 654 | Value::use_iterator IncrUse = Incr->use_begin(); |
| 655 | Instruction *U1 = cast<Instruction>(IncrUse++); |
| 656 | if (IncrUse == Incr->use_end()) return; |
| 657 | Instruction *U2 = cast<Instruction>(IncrUse++); |
| 658 | if (IncrUse != Incr->use_end()) return; |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 659 | |
Devang Patel | 84e3515 | 2008-11-17 21:32:02 +0000 | [diff] [blame] | 660 | // Find exit condition. |
| 661 | FCmpInst *EC = dyn_cast<FCmpInst>(U1); |
| 662 | if (!EC) |
| 663 | EC = dyn_cast<FCmpInst>(U2); |
| 664 | if (!EC) return; |
| 665 | |
| 666 | if (BranchInst *BI = dyn_cast<BranchInst>(EC->getParent()->getTerminator())) { |
| 667 | if (!BI->isConditional()) return; |
| 668 | if (BI->getCondition() != EC) return; |
Devang Patel | 58d43d4 | 2008-11-03 18:32:19 +0000 | [diff] [blame] | 669 | } |
Devang Patel | 58d43d4 | 2008-11-03 18:32:19 +0000 | [diff] [blame] | 670 | |
Devang Patel | cd40233 | 2008-11-17 23:27:13 +0000 | [diff] [blame] | 671 | // Find exit value. If exit value can not be represented as an interger then |
| 672 | // do not handle this floating point PH. |
Devang Patel | 84e3515 | 2008-11-17 21:32:02 +0000 | [diff] [blame] | 673 | ConstantFP *EV = NULL; |
| 674 | unsigned EVIndex = 1; |
| 675 | if (EC->getOperand(1) == Incr) |
| 676 | EVIndex = 0; |
| 677 | EV = dyn_cast<ConstantFP>(EC->getOperand(EVIndex)); |
| 678 | if (!EV) return; |
Devang Patel | 84e3515 | 2008-11-17 21:32:02 +0000 | [diff] [blame] | 679 | uint64_t intEV = Type::Int32Ty->getPrimitiveSizeInBits(); |
Devang Patel | cd40233 | 2008-11-17 23:27:13 +0000 | [diff] [blame] | 680 | if (!convertToInt(EV->getValueAPF(), &intEV)) |
Devang Patel | 84e3515 | 2008-11-17 21:32:02 +0000 | [diff] [blame] | 681 | return; |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 682 | |
Devang Patel | 84e3515 | 2008-11-17 21:32:02 +0000 | [diff] [blame] | 683 | // Find new predicate for integer comparison. |
| 684 | CmpInst::Predicate NewPred = CmpInst::BAD_ICMP_PREDICATE; |
| 685 | switch (EC->getPredicate()) { |
| 686 | case CmpInst::FCMP_OEQ: |
| 687 | case CmpInst::FCMP_UEQ: |
| 688 | NewPred = CmpInst::ICMP_EQ; |
| 689 | break; |
| 690 | case CmpInst::FCMP_OGT: |
| 691 | case CmpInst::FCMP_UGT: |
| 692 | NewPred = CmpInst::ICMP_UGT; |
| 693 | break; |
| 694 | case CmpInst::FCMP_OGE: |
| 695 | case CmpInst::FCMP_UGE: |
| 696 | NewPred = CmpInst::ICMP_UGE; |
| 697 | break; |
| 698 | case CmpInst::FCMP_OLT: |
| 699 | case CmpInst::FCMP_ULT: |
| 700 | NewPred = CmpInst::ICMP_ULT; |
| 701 | break; |
| 702 | case CmpInst::FCMP_OLE: |
| 703 | case CmpInst::FCMP_ULE: |
| 704 | NewPred = CmpInst::ICMP_ULE; |
| 705 | break; |
| 706 | default: |
| 707 | break; |
Devang Patel | 58d43d4 | 2008-11-03 18:32:19 +0000 | [diff] [blame] | 708 | } |
Devang Patel | 84e3515 | 2008-11-17 21:32:02 +0000 | [diff] [blame] | 709 | if (NewPred == CmpInst::BAD_ICMP_PREDICATE) return; |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 710 | |
Devang Patel | 84e3515 | 2008-11-17 21:32:02 +0000 | [diff] [blame] | 711 | // Insert new integer induction variable. |
| 712 | PHINode *NewPHI = PHINode::Create(Type::Int32Ty, |
| 713 | PH->getName()+".int", PH); |
Devang Patel | cd40233 | 2008-11-17 23:27:13 +0000 | [diff] [blame] | 714 | NewPHI->addIncoming(ConstantInt::get(Type::Int32Ty, newInitValue), |
Devang Patel | 84e3515 | 2008-11-17 21:32:02 +0000 | [diff] [blame] | 715 | PH->getIncomingBlock(IncomingEdge)); |
| 716 | |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 717 | Value *NewAdd = BinaryOperator::CreateAdd(NewPHI, |
| 718 | ConstantInt::get(Type::Int32Ty, |
Devang Patel | cd40233 | 2008-11-17 23:27:13 +0000 | [diff] [blame] | 719 | newIncrValue), |
Devang Patel | 84e3515 | 2008-11-17 21:32:02 +0000 | [diff] [blame] | 720 | Incr->getName()+".int", Incr); |
| 721 | NewPHI->addIncoming(NewAdd, PH->getIncomingBlock(BackEdge)); |
| 722 | |
Dale Johannesen | 617d108 | 2009-04-27 21:03:15 +0000 | [diff] [blame] | 723 | // The back edge is edge 1 of newPHI, whatever it may have been in the |
| 724 | // original PHI. |
Devang Patel | 84e3515 | 2008-11-17 21:32:02 +0000 | [diff] [blame] | 725 | ConstantInt *NewEV = ConstantInt::get(Type::Int32Ty, intEV); |
Dale Johannesen | 617d108 | 2009-04-27 21:03:15 +0000 | [diff] [blame] | 726 | Value *LHS = (EVIndex == 1 ? NewPHI->getIncomingValue(1) : NewEV); |
| 727 | Value *RHS = (EVIndex == 1 ? NewEV : NewPHI->getIncomingValue(1)); |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 728 | ICmpInst *NewEC = new ICmpInst(NewPred, LHS, RHS, EC->getNameStart(), |
Devang Patel | 84e3515 | 2008-11-17 21:32:02 +0000 | [diff] [blame] | 729 | EC->getParent()->getTerminator()); |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 730 | |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 731 | // In the following deltions, PH may become dead and may be deleted. |
| 732 | // Use a WeakVH to observe whether this happens. |
| 733 | WeakVH WeakPH = PH; |
| 734 | |
Devang Patel | 84e3515 | 2008-11-17 21:32:02 +0000 | [diff] [blame] | 735 | // Delete old, floating point, exit comparision instruction. |
Dan Gohman | 14fba29 | 2009-05-24 18:09:01 +0000 | [diff] [blame] | 736 | NewEC->takeName(EC); |
Devang Patel | 84e3515 | 2008-11-17 21:32:02 +0000 | [diff] [blame] | 737 | EC->replaceAllUsesWith(NewEC); |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 738 | RecursivelyDeleteTriviallyDeadInstructions(EC); |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 739 | |
Devang Patel | 84e3515 | 2008-11-17 21:32:02 +0000 | [diff] [blame] | 740 | // Delete old, floating point, increment instruction. |
| 741 | Incr->replaceAllUsesWith(UndefValue::get(Incr->getType())); |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 742 | RecursivelyDeleteTriviallyDeadInstructions(Incr); |
Dan Gohman | cafb813 | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 743 | |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 744 | // Replace floating induction variable, if it isn't already deleted. |
| 745 | // Give SIToFPInst preference over UIToFPInst because it is faster on |
| 746 | // platforms that are widely used. |
| 747 | if (WeakPH && !PH->use_empty()) { |
| 748 | if (useSIToFPInst(*InitValue, *EV, newInitValue, intEV)) { |
| 749 | SIToFPInst *Conv = new SIToFPInst(NewPHI, PH->getType(), "indvar.conv", |
| 750 | PH->getParent()->getFirstNonPHI()); |
| 751 | PH->replaceAllUsesWith(Conv); |
| 752 | } else { |
| 753 | UIToFPInst *Conv = new UIToFPInst(NewPHI, PH->getType(), "indvar.conv", |
| 754 | PH->getParent()->getFirstNonPHI()); |
| 755 | PH->replaceAllUsesWith(Conv); |
| 756 | } |
| 757 | RecursivelyDeleteTriviallyDeadInstructions(PH); |
Devang Patel | cd40233 | 2008-11-17 23:27:13 +0000 | [diff] [blame] | 758 | } |
Devang Patel | 58d43d4 | 2008-11-03 18:32:19 +0000 | [diff] [blame] | 759 | |
Dan Gohman | 81db61a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 760 | // Add a new IVUsers entry for the newly-created integer PHI. |
| 761 | IU->AddUsersIfInteresting(NewPHI); |
| 762 | } |