| Chris Lattner | 476e6df | 2001-12-03 17:28:42 +0000 | [diff] [blame] | 1 | //===- IndVarSimplify.cpp - Induction Variable Elimination ----------------===// |
| Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 2 | // |
| Chandler Carruth | 2946cd7 | 2019-01-19 08:50:56 +0000 | [diff] [blame] | 3 | // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. |
| 4 | // See https://llvm.org/LICENSE.txt for license information. |
| 5 | // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception |
| Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 6 | // |
| John Criswell | 482202a | 2003-10-20 19:43:21 +0000 | [diff] [blame] | 7 | //===----------------------------------------------------------------------===// |
| Chris Lattner | 476e6df | 2001-12-03 17:28:42 +0000 | [diff] [blame] | 8 | // |
| Chris Lattner | e61b67d | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 9 | // This transformation analyzes and transforms the induction variables (and |
| 10 | // computations derived from them) into simpler forms suitable for subsequent |
| 11 | // analysis and transformation. |
| 12 | // |
| Chris Lattner | e61b67d | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 13 | // If the trip count of a loop is computable, this pass also makes the following |
| 14 | // changes: |
| 15 | // 1. The exit condition for the loop is canonicalized to compare the |
| 16 | // induction value against the exit value. This turns loops like: |
| 17 | // 'for (i = 7; i*i < 1000; ++i)' into 'for (i = 0; i != 25; ++i)' |
| 18 | // 2. Any use outside of the loop of an expression derived from the indvar |
| 19 | // is changed to compute the derived value outside of the loop, eliminating |
| 20 | // the dependence on the exit value of the induction variable. If the only |
| 21 | // purpose of the loop is to compute the exit value of some derived |
| 22 | // expression, this transformation will make the loop dead. |
| 23 | // |
| Chris Lattner | 476e6df | 2001-12-03 17:28:42 +0000 | [diff] [blame] | 24 | //===----------------------------------------------------------------------===// |
| 25 | |
| Sanjoy Das | 4d4339d | 2016-06-05 18:01:19 +0000 | [diff] [blame] | 26 | #include "llvm/Transforms/Scalar/IndVarSimplify.h" |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 27 | #include "llvm/ADT/APFloat.h" |
| 28 | #include "llvm/ADT/APInt.h" |
| 29 | #include "llvm/ADT/ArrayRef.h" |
| 30 | #include "llvm/ADT/DenseMap.h" |
| 31 | #include "llvm/ADT/None.h" |
| 32 | #include "llvm/ADT/Optional.h" |
| 33 | #include "llvm/ADT/STLExtras.h" |
| 34 | #include "llvm/ADT/SmallPtrSet.h" |
| Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 35 | #include "llvm/ADT/SmallVector.h" |
| 36 | #include "llvm/ADT/Statistic.h" |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 37 | #include "llvm/ADT/iterator_range.h" |
| Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 38 | #include "llvm/Analysis/LoopInfo.h" |
| 39 | #include "llvm/Analysis/LoopPass.h" |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 40 | #include "llvm/Analysis/ScalarEvolution.h" |
| Chandler Carruth | 3bab7e1 | 2017-01-11 09:43:56 +0000 | [diff] [blame] | 41 | #include "llvm/Analysis/ScalarEvolutionExpander.h" |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 42 | #include "llvm/Analysis/ScalarEvolutionExpressions.h" |
| Benjamin Kramer | 799003b | 2015-03-23 19:32:43 +0000 | [diff] [blame] | 43 | #include "llvm/Analysis/TargetLibraryInfo.h" |
| Jingyue Wu | 8a12cea | 2014-11-12 18:09:15 +0000 | [diff] [blame] | 44 | #include "llvm/Analysis/TargetTransformInfo.h" |
| David Blaikie | 31b98d2 | 2018-06-04 21:23:21 +0000 | [diff] [blame] | 45 | #include "llvm/Transforms/Utils/Local.h" |
| Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 46 | #include "llvm/IR/BasicBlock.h" |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 47 | #include "llvm/IR/Constant.h" |
| 48 | #include "llvm/IR/ConstantRange.h" |
| Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 49 | #include "llvm/IR/Constants.h" |
| 50 | #include "llvm/IR/DataLayout.h" |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 51 | #include "llvm/IR/DerivedTypes.h" |
| Chandler Carruth | 5ad5f15 | 2014-01-13 09:26:24 +0000 | [diff] [blame] | 52 | #include "llvm/IR/Dominators.h" |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 53 | #include "llvm/IR/Function.h" |
| 54 | #include "llvm/IR/IRBuilder.h" |
| 55 | #include "llvm/IR/InstrTypes.h" |
| 56 | #include "llvm/IR/Instruction.h" |
| Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 57 | #include "llvm/IR/Instructions.h" |
| 58 | #include "llvm/IR/IntrinsicInst.h" |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 59 | #include "llvm/IR/Intrinsics.h" |
| 60 | #include "llvm/IR/Module.h" |
| 61 | #include "llvm/IR/Operator.h" |
| 62 | #include "llvm/IR/PassManager.h" |
| Sanjoy Das | 6f062c8 | 2015-07-09 18:46:12 +0000 | [diff] [blame] | 63 | #include "llvm/IR/PatternMatch.h" |
| Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 64 | #include "llvm/IR/Type.h" |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 65 | #include "llvm/IR/Use.h" |
| 66 | #include "llvm/IR/User.h" |
| 67 | #include "llvm/IR/Value.h" |
| 68 | #include "llvm/IR/ValueHandle.h" |
| 69 | #include "llvm/Pass.h" |
| 70 | #include "llvm/Support/Casting.h" |
| Andrew Trick | 56b315a | 2011-06-28 03:01:46 +0000 | [diff] [blame] | 71 | #include "llvm/Support/CommandLine.h" |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 72 | #include "llvm/Support/Compiler.h" |
| Chris Lattner | 0816559 | 2007-01-07 01:14:12 +0000 | [diff] [blame] | 73 | #include "llvm/Support/Debug.h" |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 74 | #include "llvm/Support/ErrorHandling.h" |
| 75 | #include "llvm/Support/MathExtras.h" |
| Chris Lattner | b25de3f | 2009-08-23 04:37:46 +0000 | [diff] [blame] | 76 | #include "llvm/Support/raw_ostream.h" |
| Chandler Carruth | 3bab7e1 | 2017-01-11 09:43:56 +0000 | [diff] [blame] | 77 | #include "llvm/Transforms/Scalar.h" |
| 78 | #include "llvm/Transforms/Scalar/LoopPassManager.h" |
| Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 79 | #include "llvm/Transforms/Utils/BasicBlockUtils.h" |
| Sanjoy Das | 683bf07 | 2015-12-08 00:13:21 +0000 | [diff] [blame] | 80 | #include "llvm/Transforms/Utils/LoopUtils.h" |
| Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 81 | #include "llvm/Transforms/Utils/SimplifyIndVar.h" |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 82 | #include <cassert> |
| 83 | #include <cstdint> |
| 84 | #include <utility> |
| 85 | |
| John Criswell | b22e9b4 | 2003-12-18 17:19:19 +0000 | [diff] [blame] | 86 | using namespace llvm; |
| Brian Gaeke | 960707c | 2003-11-11 22:41:34 +0000 | [diff] [blame] | 87 | |
| Chandler Carruth | 964daaa | 2014-04-22 02:55:47 +0000 | [diff] [blame] | 88 | #define DEBUG_TYPE "indvars" |
| 89 | |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 90 | STATISTIC(NumWidened , "Number of indvars widened"); |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 91 | STATISTIC(NumReplaced , "Number of exit values replaced"); |
| 92 | STATISTIC(NumLFTR , "Number of loop exit tests replaced"); |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 93 | STATISTIC(NumElimExt , "Number of IV sign/zero extends eliminated"); |
| Andrew Trick | 3239055 | 2011-07-06 20:50:43 +0000 | [diff] [blame] | 94 | STATISTIC(NumElimIV , "Number of congruent IVs eliminated"); |
| Chris Lattner | d3678bc | 2003-12-22 03:58:44 +0000 | [diff] [blame] | 95 | |
| Benjamin Kramer | 7ba71be | 2011-11-26 23:01:57 +0000 | [diff] [blame] | 96 | // Trip count verification can be enabled by default under NDEBUG if we |
| 97 | // implement a strong expression equivalence checker in SCEV. Until then, we |
| 98 | // use the verify-indvars flag, which may assert in some cases. |
| 99 | static cl::opt<bool> VerifyIndvars( |
| 100 | "verify-indvars", cl::Hidden, |
| 101 | cl::desc("Verify the ScalarEvolution result after running indvars")); |
| Andrew Trick | 1abe296 | 2011-05-04 02:10:13 +0000 | [diff] [blame] | 102 | |
| Wei Mi | e2538b5 | 2015-05-28 21:49:07 +0000 | [diff] [blame] | 103 | enum ReplaceExitVal { NeverRepl, OnlyCheapRepl, AlwaysRepl }; |
| 104 | |
| 105 | static cl::opt<ReplaceExitVal> ReplaceExitValue( |
| 106 | "replexitval", cl::Hidden, cl::init(OnlyCheapRepl), |
| 107 | cl::desc("Choose the strategy to replace exit value in IndVarSimplify"), |
| 108 | cl::values(clEnumValN(NeverRepl, "never", "never replace exit value"), |
| 109 | clEnumValN(OnlyCheapRepl, "cheap", |
| 110 | "only replace exit value when the cost is cheap"), |
| 111 | clEnumValN(AlwaysRepl, "always", |
| Mehdi Amini | 732afdd | 2016-10-08 19:41:06 +0000 | [diff] [blame] | 112 | "always replace exit value whenever possible"))); |
| Wei Mi | e2538b5 | 2015-05-28 21:49:07 +0000 | [diff] [blame] | 113 | |
| Artur Pilipenko | f2d5dc5 | 2016-10-19 18:59:03 +0000 | [diff] [blame] | 114 | static cl::opt<bool> UsePostIncrementRanges( |
| 115 | "indvars-post-increment-ranges", cl::Hidden, |
| 116 | cl::desc("Use post increment control-dependent ranges in IndVarSimplify"), |
| 117 | cl::init(true)); |
| 118 | |
| Serguei Katkov | 38414b5 | 2017-06-09 06:11:59 +0000 | [diff] [blame] | 119 | static cl::opt<bool> |
| 120 | DisableLFTR("disable-lftr", cl::Hidden, cl::init(false), |
| 121 | cl::desc("Disable Linear Function Test Replace optimization")); |
| 122 | |
| Wei Mi | e2538b5 | 2015-05-28 21:49:07 +0000 | [diff] [blame] | 123 | namespace { |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 124 | |
| Wei Mi | e2538b5 | 2015-05-28 21:49:07 +0000 | [diff] [blame] | 125 | struct RewritePhi; |
| Wei Mi | e2538b5 | 2015-05-28 21:49:07 +0000 | [diff] [blame] | 126 | |
| Sanjoy Das | 496f274 | 2016-05-29 21:42:00 +0000 | [diff] [blame] | 127 | class IndVarSimplify { |
| 128 | LoopInfo *LI; |
| 129 | ScalarEvolution *SE; |
| 130 | DominatorTree *DT; |
| 131 | const DataLayout &DL; |
| 132 | TargetLibraryInfo *TLI; |
| Sanjoy Das | e1e352d | 2015-09-20 18:42:50 +0000 | [diff] [blame] | 133 | const TargetTransformInfo *TTI; |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 134 | |
| Sanjoy Das | e6bca0e | 2017-05-01 17:07:49 +0000 | [diff] [blame] | 135 | SmallVector<WeakTrackingVH, 16> DeadInsts; |
| Andrew Trick | 3239055 | 2011-07-06 20:50:43 +0000 | [diff] [blame] | 136 | |
| Sanjoy Das | e1e352d | 2015-09-20 18:42:50 +0000 | [diff] [blame] | 137 | bool isValidRewrite(Value *FromVal, Value *ToVal); |
| Devang Patel | 2ac57e1 | 2007-03-07 06:39:01 +0000 | [diff] [blame] | 138 | |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 139 | bool handleFloatingPointIV(Loop *L, PHINode *PH); |
| 140 | bool rewriteNonIntegerIVs(Loop *L); |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 141 | |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 142 | bool simplifyAndExtend(Loop *L, SCEVExpander &Rewriter, LoopInfo *LI); |
| Andrew Trick | 6d45a01 | 2011-08-06 07:00:37 +0000 | [diff] [blame] | 143 | |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 144 | bool canLoopBeDeleted(Loop *L, SmallVector<RewritePhi, 8> &RewritePhiSet); |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 145 | bool rewriteLoopExitValues(Loop *L, SCEVExpander &Rewriter); |
| Max Kazantsev | fde8857 | 2018-09-10 06:50:16 +0000 | [diff] [blame] | 146 | bool rewriteFirstIterationLoopExitValues(Loop *L); |
| Max Kazantsev | 266c087 | 2018-11-08 11:54:35 +0000 | [diff] [blame] | 147 | bool hasHardUserWithinLoop(const Loop *L, const Instruction *I) const; |
| Andrew Trick | 3ec331e | 2011-08-10 03:46:27 +0000 | [diff] [blame] | 148 | |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 149 | bool linearFunctionTestReplace(Loop *L, const SCEV *BackedgeTakenCount, |
| 150 | PHINode *IndVar, SCEVExpander &Rewriter); |
| Dan Gohman | d76d71a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 151 | |
| Max Kazantsev | 4d10ba3 | 2018-09-10 06:32:00 +0000 | [diff] [blame] | 152 | bool sinkUnusedInvariants(Loop *L); |
| Sanjoy Das | 6f062c8 | 2015-07-09 18:46:12 +0000 | [diff] [blame] | 153 | |
| Sanjoy Das | 496f274 | 2016-05-29 21:42:00 +0000 | [diff] [blame] | 154 | public: |
| 155 | IndVarSimplify(LoopInfo *LI, ScalarEvolution *SE, DominatorTree *DT, |
| 156 | const DataLayout &DL, TargetLibraryInfo *TLI, |
| 157 | TargetTransformInfo *TTI) |
| 158 | : LI(LI), SE(SE), DT(DT), DL(DL), TLI(TLI), TTI(TTI) {} |
| 159 | |
| 160 | bool run(Loop *L); |
| Sanjoy Das | e1e352d | 2015-09-20 18:42:50 +0000 | [diff] [blame] | 161 | }; |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 162 | |
| 163 | } // end anonymous namespace |
| Chris Lattner | 91daaab | 2001-12-04 04:32:29 +0000 | [diff] [blame] | 164 | |
| Sanjoy Das | 9119bf4 | 2015-09-20 06:58:03 +0000 | [diff] [blame] | 165 | /// Return true if the SCEV expansion generated by the rewriter can replace the |
| 166 | /// original value. SCEV guarantees that it produces the same value, but the way |
| Max Kazantsev | f901540 | 2018-09-06 05:52:47 +0000 | [diff] [blame] | 167 | /// it is produced may be illegal IR. Ideally, this function will only be |
| 168 | /// called for verification. |
| Andrew Trick | 87716c9 | 2011-03-17 23:51:11 +0000 | [diff] [blame] | 169 | bool IndVarSimplify::isValidRewrite(Value *FromVal, Value *ToVal) { |
| 170 | // If an SCEV expression subsumed multiple pointers, its expansion could |
| 171 | // reassociate the GEP changing the base pointer. This is illegal because the |
| 172 | // final address produced by a GEP chain must be inbounds relative to its |
| 173 | // underlying object. Otherwise basic alias analysis, among other things, |
| Max Kazantsev | f901540 | 2018-09-06 05:52:47 +0000 | [diff] [blame] | 174 | // could fail in a dangerous way. Ultimately, SCEV will be improved to avoid |
| 175 | // producing an expression involving multiple pointers. Until then, we must |
| 176 | // bail out here. |
| Andrew Trick | 87716c9 | 2011-03-17 23:51:11 +0000 | [diff] [blame] | 177 | // |
| 178 | // Retrieve the pointer operand of the GEP. Don't use GetUnderlyingObject |
| 179 | // because it understands lcssa phis while SCEV does not. |
| 180 | Value *FromPtr = FromVal; |
| 181 | Value *ToPtr = ToVal; |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 182 | if (auto *GEP = dyn_cast<GEPOperator>(FromVal)) { |
| Andrew Trick | 87716c9 | 2011-03-17 23:51:11 +0000 | [diff] [blame] | 183 | FromPtr = GEP->getPointerOperand(); |
| 184 | } |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 185 | if (auto *GEP = dyn_cast<GEPOperator>(ToVal)) { |
| Andrew Trick | 87716c9 | 2011-03-17 23:51:11 +0000 | [diff] [blame] | 186 | ToPtr = GEP->getPointerOperand(); |
| 187 | } |
| 188 | if (FromPtr != FromVal || ToPtr != ToVal) { |
| 189 | // Quickly check the common case |
| 190 | if (FromPtr == ToPtr) |
| 191 | return true; |
| 192 | |
| 193 | // SCEV may have rewritten an expression that produces the GEP's pointer |
| 194 | // operand. That's ok as long as the pointer operand has the same base |
| 195 | // pointer. Unlike GetUnderlyingObject(), getPointerBase() will find the |
| 196 | // base of a recurrence. This handles the case in which SCEV expansion |
| 197 | // converts a pointer type recurrence into a nonrecurrent pointer base |
| 198 | // indexed by an integer recurrence. |
| Nadav Rotem | 3924cb0 | 2011-12-05 06:29:09 +0000 | [diff] [blame] | 199 | |
| 200 | // If the GEP base pointer is a vector of pointers, abort. |
| 201 | if (!FromPtr->getType()->isPointerTy() || !ToPtr->getType()->isPointerTy()) |
| 202 | return false; |
| 203 | |
| Andrew Trick | 87716c9 | 2011-03-17 23:51:11 +0000 | [diff] [blame] | 204 | const SCEV *FromBase = SE->getPointerBase(SE->getSCEV(FromPtr)); |
| 205 | const SCEV *ToBase = SE->getPointerBase(SE->getSCEV(ToPtr)); |
| 206 | if (FromBase == ToBase) |
| 207 | return true; |
| 208 | |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 209 | LLVM_DEBUG(dbgs() << "INDVARS: GEP rewrite bail out " << *FromBase |
| 210 | << " != " << *ToBase << "\n"); |
| Andrew Trick | 87716c9 | 2011-03-17 23:51:11 +0000 | [diff] [blame] | 211 | |
| 212 | return false; |
| 213 | } |
| 214 | return true; |
| 215 | } |
| 216 | |
| Andrew Trick | 638b355 | 2011-07-20 05:32:06 +0000 | [diff] [blame] | 217 | /// Determine the insertion point for this user. By default, insert immediately |
| 218 | /// before the user. SCEVExpander or LICM will hoist loop invariants out of the |
| 219 | /// loop. For PHI nodes, there may be multiple uses, so compute the nearest |
| Max Kazantsev | 2a184af | 2019-02-12 09:59:44 +0000 | [diff] [blame] | 220 | /// common dominator for the incoming blocks. A nullptr can be returned if no |
| 221 | /// viable location is found: it may happen if User is a PHI and Def only comes |
| 222 | /// to this PHI from unreachable blocks. |
| Andrew Trick | 638b355 | 2011-07-20 05:32:06 +0000 | [diff] [blame] | 223 | static Instruction *getInsertPointForUses(Instruction *User, Value *Def, |
| Sanjoy Das | 683bf07 | 2015-12-08 00:13:21 +0000 | [diff] [blame] | 224 | DominatorTree *DT, LoopInfo *LI) { |
| Andrew Trick | 638b355 | 2011-07-20 05:32:06 +0000 | [diff] [blame] | 225 | PHINode *PHI = dyn_cast<PHINode>(User); |
| 226 | if (!PHI) |
| 227 | return User; |
| 228 | |
| Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 229 | Instruction *InsertPt = nullptr; |
| Andrew Trick | 638b355 | 2011-07-20 05:32:06 +0000 | [diff] [blame] | 230 | for (unsigned i = 0, e = PHI->getNumIncomingValues(); i != e; ++i) { |
| 231 | if (PHI->getIncomingValue(i) != Def) |
| 232 | continue; |
| 233 | |
| 234 | BasicBlock *InsertBB = PHI->getIncomingBlock(i); |
| Max Kazantsev | 2a184af | 2019-02-12 09:59:44 +0000 | [diff] [blame] | 235 | |
| 236 | if (!DT->isReachableFromEntry(InsertBB)) |
| 237 | continue; |
| 238 | |
| Andrew Trick | 638b355 | 2011-07-20 05:32:06 +0000 | [diff] [blame] | 239 | if (!InsertPt) { |
| 240 | InsertPt = InsertBB->getTerminator(); |
| 241 | continue; |
| 242 | } |
| 243 | InsertBB = DT->findNearestCommonDominator(InsertPt->getParent(), InsertBB); |
| 244 | InsertPt = InsertBB->getTerminator(); |
| 245 | } |
| Max Kazantsev | 2a184af | 2019-02-12 09:59:44 +0000 | [diff] [blame] | 246 | |
| 247 | // If we have skipped all inputs, it means that Def only comes to Phi from |
| 248 | // unreachable blocks. |
| 249 | if (!InsertPt) |
| 250 | return nullptr; |
| Sanjoy Das | 683bf07 | 2015-12-08 00:13:21 +0000 | [diff] [blame] | 251 | |
| 252 | auto *DefI = dyn_cast<Instruction>(Def); |
| 253 | if (!DefI) |
| 254 | return InsertPt; |
| 255 | |
| 256 | assert(DT->dominates(DefI, InsertPt) && "def does not dominate all uses"); |
| 257 | |
| 258 | auto *L = LI->getLoopFor(DefI->getParent()); |
| 259 | assert(!L || L->contains(LI->getLoopFor(InsertPt->getParent()))); |
| 260 | |
| 261 | for (auto *DTN = (*DT)[InsertPt->getParent()]; DTN; DTN = DTN->getIDom()) |
| 262 | if (LI->getLoopFor(DTN->getBlock()) == L) |
| 263 | return DTN->getBlock()->getTerminator(); |
| 264 | |
| 265 | llvm_unreachable("DefI dominates InsertPt!"); |
| Andrew Trick | 638b355 | 2011-07-20 05:32:06 +0000 | [diff] [blame] | 266 | } |
| 267 | |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 268 | //===----------------------------------------------------------------------===// |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 269 | // rewriteNonIntegerIVs and helpers. Prefer integer IVs. |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 270 | //===----------------------------------------------------------------------===// |
| Andrew Trick | 38c4e34 | 2011-05-03 22:24:10 +0000 | [diff] [blame] | 271 | |
| Sanjoy Das | 9119bf4 | 2015-09-20 06:58:03 +0000 | [diff] [blame] | 272 | /// Convert APF to an integer, if possible. |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 273 | static bool ConvertToSInt(const APFloat &APF, int64_t &IntVal) { |
| 274 | bool isExact = false; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 275 | // See if we can convert this to an int64_t |
| 276 | uint64_t UIntVal; |
| Simon Pilgrim | 00b3499 | 2017-03-20 14:40:12 +0000 | [diff] [blame] | 277 | if (APF.convertToInteger(makeMutableArrayRef(UIntVal), 64, true, |
| 278 | APFloat::rmTowardZero, &isExact) != APFloat::opOK || |
| 279 | !isExact) |
| Andrew Trick | 38c4e34 | 2011-05-03 22:24:10 +0000 | [diff] [blame] | 280 | return false; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 281 | IntVal = UIntVal; |
| Andrew Trick | 38c4e34 | 2011-05-03 22:24:10 +0000 | [diff] [blame] | 282 | return true; |
| 283 | } |
| 284 | |
| Sanjoy Das | 9119bf4 | 2015-09-20 06:58:03 +0000 | [diff] [blame] | 285 | /// If the loop has floating induction variable then insert corresponding |
| 286 | /// integer induction variable if possible. |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 287 | /// For example, |
| 288 | /// for(double i = 0; i < 10000; ++i) |
| 289 | /// bar(i) |
| 290 | /// is converted into |
| 291 | /// for(int i = 0; i < 10000; ++i) |
| 292 | /// bar((double)i); |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 293 | bool IndVarSimplify::handleFloatingPointIV(Loop *L, PHINode *PN) { |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 294 | unsigned IncomingEdge = L->contains(PN->getIncomingBlock(0)); |
| 295 | unsigned BackEdge = IncomingEdge^1; |
| Andrew Trick | eb3c36e | 2011-05-25 04:42:22 +0000 | [diff] [blame] | 296 | |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 297 | // Check incoming value. |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 298 | auto *InitValueVal = dyn_cast<ConstantFP>(PN->getIncomingValue(IncomingEdge)); |
| Andrew Trick | eb3c36e | 2011-05-25 04:42:22 +0000 | [diff] [blame] | 299 | |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 300 | int64_t InitValue; |
| 301 | if (!InitValueVal || !ConvertToSInt(InitValueVal->getValueAPF(), InitValue)) |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 302 | return false; |
| Andrew Trick | eb3c36e | 2011-05-25 04:42:22 +0000 | [diff] [blame] | 303 | |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 304 | // Check IV increment. Reject this PN if increment operation is not |
| 305 | // an add or increment value can not be represented by an integer. |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 306 | auto *Incr = dyn_cast<BinaryOperator>(PN->getIncomingValue(BackEdge)); |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 307 | if (Incr == nullptr || Incr->getOpcode() != Instruction::FAdd) return false; |
| Andrew Trick | eb3c36e | 2011-05-25 04:42:22 +0000 | [diff] [blame] | 308 | |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 309 | // If this is not an add of the PHI with a constantfp, or if the constant fp |
| 310 | // is not an integer, bail out. |
| 311 | ConstantFP *IncValueVal = dyn_cast<ConstantFP>(Incr->getOperand(1)); |
| 312 | int64_t IncValue; |
| Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 313 | if (IncValueVal == nullptr || Incr->getOperand(0) != PN || |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 314 | !ConvertToSInt(IncValueVal->getValueAPF(), IncValue)) |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 315 | return false; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 316 | |
| 317 | // Check Incr uses. One user is PN and the other user is an exit condition |
| 318 | // used by the conditional terminator. |
| Chandler Carruth | cdf4788 | 2014-03-09 03:16:01 +0000 | [diff] [blame] | 319 | Value::user_iterator IncrUse = Incr->user_begin(); |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 320 | Instruction *U1 = cast<Instruction>(*IncrUse++); |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 321 | if (IncrUse == Incr->user_end()) return false; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 322 | Instruction *U2 = cast<Instruction>(*IncrUse++); |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 323 | if (IncrUse != Incr->user_end()) return false; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 324 | |
| 325 | // Find exit condition, which is an fcmp. If it doesn't exist, or if it isn't |
| 326 | // only used by a branch, we can't transform it. |
| 327 | FCmpInst *Compare = dyn_cast<FCmpInst>(U1); |
| 328 | if (!Compare) |
| 329 | Compare = dyn_cast<FCmpInst>(U2); |
| Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 330 | if (!Compare || !Compare->hasOneUse() || |
| Chandler Carruth | cdf4788 | 2014-03-09 03:16:01 +0000 | [diff] [blame] | 331 | !isa<BranchInst>(Compare->user_back())) |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 332 | return false; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 333 | |
| Chandler Carruth | cdf4788 | 2014-03-09 03:16:01 +0000 | [diff] [blame] | 334 | BranchInst *TheBr = cast<BranchInst>(Compare->user_back()); |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 335 | |
| 336 | // We need to verify that the branch actually controls the iteration count |
| 337 | // of the loop. If not, the new IV can overflow and no one will notice. |
| 338 | // The branch block must be in the loop and one of the successors must be out |
| 339 | // of the loop. |
| 340 | assert(TheBr->isConditional() && "Can't use fcmp if not conditional"); |
| 341 | if (!L->contains(TheBr->getParent()) || |
| 342 | (L->contains(TheBr->getSuccessor(0)) && |
| 343 | L->contains(TheBr->getSuccessor(1)))) |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 344 | return false; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 345 | |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 346 | // If it isn't a comparison with an integer-as-fp (the exit value), we can't |
| 347 | // transform it. |
| 348 | ConstantFP *ExitValueVal = dyn_cast<ConstantFP>(Compare->getOperand(1)); |
| 349 | int64_t ExitValue; |
| Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 350 | if (ExitValueVal == nullptr || |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 351 | !ConvertToSInt(ExitValueVal->getValueAPF(), ExitValue)) |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 352 | return false; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 353 | |
| 354 | // Find new predicate for integer comparison. |
| 355 | CmpInst::Predicate NewPred = CmpInst::BAD_ICMP_PREDICATE; |
| 356 | switch (Compare->getPredicate()) { |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 357 | default: return false; // Unknown comparison. |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 358 | case CmpInst::FCMP_OEQ: |
| 359 | case CmpInst::FCMP_UEQ: NewPred = CmpInst::ICMP_EQ; break; |
| 360 | case CmpInst::FCMP_ONE: |
| 361 | case CmpInst::FCMP_UNE: NewPred = CmpInst::ICMP_NE; break; |
| 362 | case CmpInst::FCMP_OGT: |
| 363 | case CmpInst::FCMP_UGT: NewPred = CmpInst::ICMP_SGT; break; |
| 364 | case CmpInst::FCMP_OGE: |
| 365 | case CmpInst::FCMP_UGE: NewPred = CmpInst::ICMP_SGE; break; |
| 366 | case CmpInst::FCMP_OLT: |
| 367 | case CmpInst::FCMP_ULT: NewPred = CmpInst::ICMP_SLT; break; |
| 368 | case CmpInst::FCMP_OLE: |
| 369 | case CmpInst::FCMP_ULE: NewPred = CmpInst::ICMP_SLE; break; |
| Andrew Trick | eb3c36e | 2011-05-25 04:42:22 +0000 | [diff] [blame] | 370 | } |
| Andrew Trick | eb3c36e | 2011-05-25 04:42:22 +0000 | [diff] [blame] | 371 | |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 372 | // We convert the floating point induction variable to a signed i32 value if |
| 373 | // we can. This is only safe if the comparison will not overflow in a way |
| 374 | // that won't be trapped by the integer equivalent operations. Check for this |
| 375 | // now. |
| 376 | // TODO: We could use i64 if it is native and the range requires it. |
| Dan Gohman | 4a645b8 | 2010-04-12 21:13:43 +0000 | [diff] [blame] | 377 | |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 378 | // The start/stride/exit values must all fit in signed i32. |
| 379 | if (!isInt<32>(InitValue) || !isInt<32>(IncValue) || !isInt<32>(ExitValue)) |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 380 | return false; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 381 | |
| 382 | // If not actually striding (add x, 0.0), avoid touching the code. |
| 383 | if (IncValue == 0) |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 384 | return false; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 385 | |
| 386 | // Positive and negative strides have different safety conditions. |
| 387 | if (IncValue > 0) { |
| 388 | // If we have a positive stride, we require the init to be less than the |
| Andrew Trick | 3de5b8e | 2011-09-13 01:59:32 +0000 | [diff] [blame] | 389 | // exit value. |
| 390 | if (InitValue >= ExitValue) |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 391 | return false; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 392 | |
| 393 | uint32_t Range = uint32_t(ExitValue-InitValue); |
| Andrew Trick | 3de5b8e | 2011-09-13 01:59:32 +0000 | [diff] [blame] | 394 | // Check for infinite loop, either: |
| 395 | // while (i <= Exit) or until (i > Exit) |
| 396 | if (NewPred == CmpInst::ICMP_SLE || NewPred == CmpInst::ICMP_SGT) { |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 397 | if (++Range == 0) return false; // Range overflows. |
| Dan Gohman | eb6be65 | 2009-02-12 22:19:27 +0000 | [diff] [blame] | 398 | } |
| Chris Lattner | 0cec5cb | 2004-04-15 15:21:43 +0000 | [diff] [blame] | 399 | |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 400 | unsigned Leftover = Range % uint32_t(IncValue); |
| 401 | |
| 402 | // If this is an equality comparison, we require that the strided value |
| 403 | // exactly land on the exit value, otherwise the IV condition will wrap |
| 404 | // around and do things the fp IV wouldn't. |
| 405 | if ((NewPred == CmpInst::ICMP_EQ || NewPred == CmpInst::ICMP_NE) && |
| 406 | Leftover != 0) |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 407 | return false; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 408 | |
| 409 | // If the stride would wrap around the i32 before exiting, we can't |
| 410 | // transform the IV. |
| 411 | if (Leftover != 0 && int32_t(ExitValue+IncValue) < ExitValue) |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 412 | return false; |
| Chris Lattner | d7a559e | 2004-04-15 20:26:22 +0000 | [diff] [blame] | 413 | } else { |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 414 | // If we have a negative stride, we require the init to be greater than the |
| Andrew Trick | 3de5b8e | 2011-09-13 01:59:32 +0000 | [diff] [blame] | 415 | // exit value. |
| 416 | if (InitValue <= ExitValue) |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 417 | return false; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 418 | |
| 419 | uint32_t Range = uint32_t(InitValue-ExitValue); |
| Andrew Trick | 3de5b8e | 2011-09-13 01:59:32 +0000 | [diff] [blame] | 420 | // Check for infinite loop, either: |
| 421 | // while (i >= Exit) or until (i < Exit) |
| 422 | if (NewPred == CmpInst::ICMP_SGE || NewPred == CmpInst::ICMP_SLT) { |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 423 | if (++Range == 0) return false; // Range overflows. |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 424 | } |
| 425 | |
| 426 | unsigned Leftover = Range % uint32_t(-IncValue); |
| 427 | |
| 428 | // If this is an equality comparison, we require that the strided value |
| 429 | // exactly land on the exit value, otherwise the IV condition will wrap |
| 430 | // around and do things the fp IV wouldn't. |
| 431 | if ((NewPred == CmpInst::ICMP_EQ || NewPred == CmpInst::ICMP_NE) && |
| 432 | Leftover != 0) |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 433 | return false; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 434 | |
| 435 | // If the stride would wrap around the i32 before exiting, we can't |
| 436 | // transform the IV. |
| 437 | if (Leftover != 0 && int32_t(ExitValue+IncValue) > ExitValue) |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 438 | return false; |
| Chris Lattner | d7a559e | 2004-04-15 20:26:22 +0000 | [diff] [blame] | 439 | } |
| Chris Lattner | 0cec5cb | 2004-04-15 15:21:43 +0000 | [diff] [blame] | 440 | |
| Chris Lattner | 229907c | 2011-07-18 04:54:35 +0000 | [diff] [blame] | 441 | IntegerType *Int32Ty = Type::getInt32Ty(PN->getContext()); |
| Chris Lattner | e61b67d | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 442 | |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 443 | // Insert new integer induction variable. |
| 444 | PHINode *NewPHI = PHINode::Create(Int32Ty, 2, PN->getName()+".int", PN); |
| 445 | NewPHI->addIncoming(ConstantInt::get(Int32Ty, InitValue), |
| 446 | PN->getIncomingBlock(IncomingEdge)); |
| Chris Lattner | e61b67d | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 447 | |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 448 | Value *NewAdd = |
| 449 | BinaryOperator::CreateAdd(NewPHI, ConstantInt::get(Int32Ty, IncValue), |
| 450 | Incr->getName()+".int", Incr); |
| 451 | NewPHI->addIncoming(NewAdd, PN->getIncomingBlock(BackEdge)); |
| Dan Gohman | eb6be65 | 2009-02-12 22:19:27 +0000 | [diff] [blame] | 452 | |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 453 | ICmpInst *NewCompare = new ICmpInst(TheBr, NewPred, NewAdd, |
| 454 | ConstantInt::get(Int32Ty, ExitValue), |
| 455 | Compare->getName()); |
| Dan Gohman | d76d71a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 456 | |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 457 | // In the following deletions, PN may become dead and may be deleted. |
| Sanjoy Das | e6bca0e | 2017-05-01 17:07:49 +0000 | [diff] [blame] | 458 | // Use a WeakTrackingVH to observe whether this happens. |
| 459 | WeakTrackingVH WeakPH = PN; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 460 | |
| 461 | // Delete the old floating point exit comparison. The branch starts using the |
| 462 | // new comparison. |
| 463 | NewCompare->takeName(Compare); |
| 464 | Compare->replaceAllUsesWith(NewCompare); |
| Benjamin Kramer | 8bcc971 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 465 | RecursivelyDeleteTriviallyDeadInstructions(Compare, TLI); |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 466 | |
| 467 | // Delete the old floating point increment. |
| 468 | Incr->replaceAllUsesWith(UndefValue::get(Incr->getType())); |
| Benjamin Kramer | 8bcc971 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 469 | RecursivelyDeleteTriviallyDeadInstructions(Incr, TLI); |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 470 | |
| 471 | // If the FP induction variable still has uses, this is because something else |
| 472 | // in the loop uses its value. In order to canonicalize the induction |
| 473 | // variable, we chose to eliminate the IV and rewrite it in terms of an |
| 474 | // int->fp cast. |
| 475 | // |
| 476 | // We give preference to sitofp over uitofp because it is faster on most |
| 477 | // platforms. |
| 478 | if (WeakPH) { |
| 479 | Value *Conv = new SIToFPInst(NewPHI, PN->getType(), "indvar.conv", |
| Duncan P. N. Exon Smith | 3a9c9e3 | 2015-10-13 18:26:00 +0000 | [diff] [blame] | 480 | &*PN->getParent()->getFirstInsertionPt()); |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 481 | PN->replaceAllUsesWith(Conv); |
| Benjamin Kramer | 8bcc971 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 482 | RecursivelyDeleteTriviallyDeadInstructions(PN, TLI); |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 483 | } |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 484 | return true; |
| Chris Lattner | e61b67d | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 485 | } |
| 486 | |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 487 | bool IndVarSimplify::rewriteNonIntegerIVs(Loop *L) { |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 488 | // First step. Check to see if there are any floating-point recurrences. |
| 489 | // If there are, change them into integer recurrences, permitting analysis by |
| 490 | // the SCEV routines. |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 491 | BasicBlock *Header = L->getHeader(); |
| 492 | |
| Sanjoy Das | e6bca0e | 2017-05-01 17:07:49 +0000 | [diff] [blame] | 493 | SmallVector<WeakTrackingVH, 8> PHIs; |
| Benjamin Kramer | c7fc81e | 2017-12-30 15:27:33 +0000 | [diff] [blame] | 494 | for (PHINode &PN : Header->phis()) |
| 495 | PHIs.push_back(&PN); |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 496 | |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 497 | bool Changed = false; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 498 | for (unsigned i = 0, e = PHIs.size(); i != e; ++i) |
| 499 | if (PHINode *PN = dyn_cast_or_null<PHINode>(&*PHIs[i])) |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 500 | Changed |= handleFloatingPointIV(L, PN); |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 501 | |
| 502 | // If the loop previously had floating-point IV, ScalarEvolution |
| 503 | // may not have been able to compute a trip count. Now that we've done some |
| 504 | // re-writing, the trip count may be computable. |
| 505 | if (Changed) |
| 506 | SE->forgetLoop(L); |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 507 | return Changed; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 508 | } |
| 509 | |
| Wei Mi | e2538b5 | 2015-05-28 21:49:07 +0000 | [diff] [blame] | 510 | namespace { |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 511 | |
| Wei Mi | e2538b5 | 2015-05-28 21:49:07 +0000 | [diff] [blame] | 512 | // Collect information about PHI nodes which can be transformed in |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 513 | // rewriteLoopExitValues. |
| Wei Mi | e2538b5 | 2015-05-28 21:49:07 +0000 | [diff] [blame] | 514 | struct RewritePhi { |
| 515 | PHINode *PN; |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 516 | |
| 517 | // Ith incoming value. |
| 518 | unsigned Ith; |
| 519 | |
| 520 | // Exit value after expansion. |
| 521 | Value *Val; |
| 522 | |
| 523 | // High Cost when expansion. |
| 524 | bool HighCost; |
| Wei Mi | e2538b5 | 2015-05-28 21:49:07 +0000 | [diff] [blame] | 525 | |
| Sanjoy Das | de47590 | 2016-01-17 18:12:52 +0000 | [diff] [blame] | 526 | RewritePhi(PHINode *P, unsigned I, Value *V, bool H) |
| 527 | : PN(P), Ith(I), Val(V), HighCost(H) {} |
| Wei Mi | e2538b5 | 2015-05-28 21:49:07 +0000 | [diff] [blame] | 528 | }; |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 529 | |
| 530 | } // end anonymous namespace |
| Wei Mi | e2538b5 | 2015-05-28 21:49:07 +0000 | [diff] [blame] | 531 | |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 532 | //===----------------------------------------------------------------------===// |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 533 | // rewriteLoopExitValues - Optimize IV users outside the loop. |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 534 | // As a side effect, reduces the amount of IV processing within the loop. |
| 535 | //===----------------------------------------------------------------------===// |
| 536 | |
| Max Kazantsev | 266c087 | 2018-11-08 11:54:35 +0000 | [diff] [blame] | 537 | bool IndVarSimplify::hasHardUserWithinLoop(const Loop *L, const Instruction *I) const { |
| 538 | SmallPtrSet<const Instruction *, 8> Visited; |
| 539 | SmallVector<const Instruction *, 8> WorkList; |
| 540 | Visited.insert(I); |
| 541 | WorkList.push_back(I); |
| 542 | while (!WorkList.empty()) { |
| 543 | const Instruction *Curr = WorkList.pop_back_val(); |
| 544 | // This use is outside the loop, nothing to do. |
| 545 | if (!L->contains(Curr)) |
| 546 | continue; |
| 547 | // Do we assume it is a "hard" use which will not be eliminated easily? |
| 548 | if (Curr->mayHaveSideEffects()) |
| 549 | return true; |
| 550 | // Otherwise, add all its users to worklist. |
| 551 | for (auto U : Curr->users()) { |
| 552 | auto *UI = cast<Instruction>(U); |
| 553 | if (Visited.insert(UI).second) |
| 554 | WorkList.push_back(UI); |
| 555 | } |
| 556 | } |
| 557 | return false; |
| 558 | } |
| 559 | |
| Sanjoy Das | 9119bf4 | 2015-09-20 06:58:03 +0000 | [diff] [blame] | 560 | /// Check to see if this loop has a computable loop-invariant execution count. |
| 561 | /// If so, this means that we can compute the final value of any expressions |
| 562 | /// that are recurrent in the loop, and substitute the exit values from the loop |
| 563 | /// into any instructions outside of the loop that use the final values of the |
| 564 | /// current expressions. |
| Dan Gohman | d76d71a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 565 | /// |
| 566 | /// This is mostly redundant with the regular IndVarSimplify activities that |
| 567 | /// happen later, except that it's more powerful in some cases, because it's |
| 568 | /// able to brute-force evaluate arbitrary instructions as long as they have |
| 569 | /// constant operands at the beginning of the loop. |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 570 | bool IndVarSimplify::rewriteLoopExitValues(Loop *L, SCEVExpander &Rewriter) { |
| Sanjoy Das | 683bf07 | 2015-12-08 00:13:21 +0000 | [diff] [blame] | 571 | // Check a pre-condition. |
| Igor Laevsky | 04423cf | 2016-10-11 13:37:22 +0000 | [diff] [blame] | 572 | assert(L->isRecursivelyLCSSAForm(*DT, *LI) && |
| 573 | "Indvars did not preserve LCSSA!"); |
| Dan Gohman | d76d71a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 574 | |
| Devang Patel | b5933bb | 2007-08-21 00:31:24 +0000 | [diff] [blame] | 575 | SmallVector<BasicBlock*, 8> ExitBlocks; |
| Chris Lattner | d7b4c92 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 576 | L->getUniqueExitBlocks(ExitBlocks); |
| Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 577 | |
| Wei Mi | e2538b5 | 2015-05-28 21:49:07 +0000 | [diff] [blame] | 578 | SmallVector<RewritePhi, 8> RewritePhiSet; |
| Chris Lattner | d7b4c92 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 579 | // Find all values that are computed inside the loop, but used outside of it. |
| 580 | // Because of LCSSA, these values will only occur in LCSSA PHI Nodes. Scan |
| 581 | // the exit blocks of the loop to find them. |
| Sanjoy Das | 8fdf87c | 2016-01-27 17:05:03 +0000 | [diff] [blame] | 582 | for (BasicBlock *ExitBB : ExitBlocks) { |
| Chris Lattner | d7b4c92 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 583 | // If there are no PHI nodes in this exit block, then no values defined |
| 584 | // inside the loop are used on this path, skip it. |
| 585 | PHINode *PN = dyn_cast<PHINode>(ExitBB->begin()); |
| 586 | if (!PN) continue; |
| Dan Gohman | f84d42f | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 587 | |
| Chris Lattner | d7b4c92 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 588 | unsigned NumPreds = PN->getNumIncomingValues(); |
| Dan Gohman | f84d42f | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 589 | |
| Chris Lattner | d7b4c92 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 590 | // Iterate over all of the PHI nodes. |
| 591 | BasicBlock::iterator BBI = ExitBB->begin(); |
| 592 | while ((PN = dyn_cast<PHINode>(BBI++))) { |
| Torok Edwin | 5349cf5 | 2009-05-24 19:36:09 +0000 | [diff] [blame] | 593 | if (PN->use_empty()) |
| 594 | continue; // dead use, don't replace it |
| Dan Gohman | c43d264 | 2010-02-18 21:34:02 +0000 | [diff] [blame] | 595 | |
| Sanjoy Das | 2f7a744 | 2016-01-27 17:05:06 +0000 | [diff] [blame] | 596 | if (!SE->isSCEVable(PN->getType())) |
| Dan Gohman | c43d264 | 2010-02-18 21:34:02 +0000 | [diff] [blame] | 597 | continue; |
| 598 | |
| Dale Johannesen | 1d6827a | 2010-02-19 07:14:22 +0000 | [diff] [blame] | 599 | // It's necessary to tell ScalarEvolution about this explicitly so that |
| 600 | // it can walk the def-use list and forget all SCEVs, as it may not be |
| 601 | // watching the PHI itself. Once the new exit value is in place, there |
| 602 | // may not be a def-use connection between the loop and every instruction |
| 603 | // which got a SCEVAddRecExpr for that loop. |
| 604 | SE->forgetValue(PN); |
| 605 | |
| Chris Lattner | d7b4c92 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 606 | // Iterate over all of the values in all the PHI nodes. |
| 607 | for (unsigned i = 0; i != NumPreds; ++i) { |
| 608 | // If the value being merged in is not integer or is not defined |
| 609 | // in the loop, skip it. |
| 610 | Value *InVal = PN->getIncomingValue(i); |
| Dan Gohman | c43d264 | 2010-02-18 21:34:02 +0000 | [diff] [blame] | 611 | if (!isa<Instruction>(InVal)) |
| Chris Lattner | d7b4c92 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 612 | continue; |
| Chris Lattner | e61b67d | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 613 | |
| Chris Lattner | d7b4c92 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 614 | // If this pred is for a subloop, not L itself, skip it. |
| Dan Gohman | f84d42f | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 615 | if (LI->getLoopFor(PN->getIncomingBlock(i)) != L) |
| Chris Lattner | d7b4c92 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 616 | continue; // The Block is in a subloop, skip it. |
| 617 | |
| 618 | // Check that InVal is defined in the loop. |
| 619 | Instruction *Inst = cast<Instruction>(InVal); |
| Dan Gohman | 18fa568 | 2009-12-18 01:24:09 +0000 | [diff] [blame] | 620 | if (!L->contains(Inst)) |
| Chris Lattner | d7b4c92 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 621 | continue; |
| Dan Gohman | f84d42f | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 622 | |
| Chris Lattner | d7b4c92 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 623 | // Okay, this instruction has a user outside of the current loop |
| 624 | // and varies predictably *inside* the loop. Evaluate the value it |
| 625 | // contains when the loop exits, if possible. |
| Dan Gohman | af75234 | 2009-07-07 17:06:11 +0000 | [diff] [blame] | 626 | const SCEV *ExitValue = SE->getSCEVAtScope(Inst, L->getParentLoop()); |
| Andrew Trick | 57243da | 2013-10-25 21:35:56 +0000 | [diff] [blame] | 627 | if (!SE->isLoopInvariant(ExitValue, L) || |
| 628 | !isSafeToExpand(ExitValue, *SE)) |
| Chris Lattner | d7b4c92 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 629 | continue; |
| Chris Lattner | 1f7648e | 2007-03-04 01:00:28 +0000 | [diff] [blame] | 630 | |
| Max Kazantsev | 541f824 | 2018-10-31 10:30:50 +0000 | [diff] [blame] | 631 | // Computing the value outside of the loop brings no benefit if it is |
| 632 | // definitely used inside the loop in a way which can not be optimized |
| 633 | // away. |
| Max Kazantsev | 266c087 | 2018-11-08 11:54:35 +0000 | [diff] [blame] | 634 | if (!isa<SCEVConstant>(ExitValue) && hasHardUserWithinLoop(L, Inst)) |
| 635 | continue; |
| Arnaud A. de Grandmaison | 87c473f | 2013-03-19 20:00:22 +0000 | [diff] [blame] | 636 | |
| Igor Laevsky | 4709c03 | 2015-08-10 18:23:58 +0000 | [diff] [blame] | 637 | bool HighCost = Rewriter.isHighCostExpansion(ExitValue, L, Inst); |
| Max Kazantsev | 2cbba56 | 2018-09-04 05:01:35 +0000 | [diff] [blame] | 638 | Value *ExitVal = Rewriter.expandCodeFor(ExitValue, PN->getType(), Inst); |
| Dan Gohman | f84d42f | 2009-02-17 19:13:57 +0000 | [diff] [blame] | 639 | |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 640 | LLVM_DEBUG(dbgs() << "INDVARS: RLEV: AfterLoopVal = " << *ExitVal |
| 641 | << '\n' |
| 642 | << " LoopVal = " << *Inst << "\n"); |
| Chris Lattner | d7b4c92 | 2007-03-04 03:43:23 +0000 | [diff] [blame] | 643 | |
| Max Kazantsev | f901540 | 2018-09-06 05:52:47 +0000 | [diff] [blame] | 644 | if (!isValidRewrite(Inst, ExitVal)) { |
| 645 | DeadInsts.push_back(ExitVal); |
| 646 | continue; |
| 647 | } |
| Andrew Trick | 87716c9 | 2011-03-17 23:51:11 +0000 | [diff] [blame] | 648 | |
| Max Kazantsev | f34115c | 2018-09-04 06:34:40 +0000 | [diff] [blame] | 649 | #ifndef NDEBUG |
| 650 | // If we reuse an instruction from a loop which is neither L nor one of |
| 651 | // its containing loops, we end up breaking LCSSA form for this loop by |
| 652 | // creating a new use of its instruction. |
| 653 | if (auto *ExitInsn = dyn_cast<Instruction>(ExitVal)) |
| 654 | if (auto *EVL = LI->getLoopFor(ExitInsn->getParent())) |
| 655 | if (EVL != L) |
| 656 | assert(EVL->contains(L) && "LCSSA breach detected!"); |
| 657 | #endif |
| 658 | |
| Wei Mi | e2538b5 | 2015-05-28 21:49:07 +0000 | [diff] [blame] | 659 | // Collect all the candidate PHINodes to be rewritten. |
| Sanjoy Das | de47590 | 2016-01-17 18:12:52 +0000 | [diff] [blame] | 660 | RewritePhiSet.emplace_back(PN, i, ExitVal, HighCost); |
| Chris Lattner | ed30abf | 2007-03-03 22:48:48 +0000 | [diff] [blame] | 661 | } |
| Chris Lattner | ed30abf | 2007-03-03 22:48:48 +0000 | [diff] [blame] | 662 | } |
| 663 | } |
| Dan Gohman | 1a2abe5 | 2010-03-20 03:53:53 +0000 | [diff] [blame] | 664 | |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 665 | bool LoopCanBeDel = canLoopBeDeleted(L, RewritePhiSet); |
| Wei Mi | e2538b5 | 2015-05-28 21:49:07 +0000 | [diff] [blame] | 666 | |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 667 | bool Changed = false; |
| Wei Mi | e2538b5 | 2015-05-28 21:49:07 +0000 | [diff] [blame] | 668 | // Transformation. |
| 669 | for (const RewritePhi &Phi : RewritePhiSet) { |
| 670 | PHINode *PN = Phi.PN; |
| 671 | Value *ExitVal = Phi.Val; |
| 672 | |
| 673 | // Only do the rewrite when the ExitValue can be expanded cheaply. |
| 674 | // If LoopCanBeDel is true, rewrite exit value aggressively. |
| 675 | if (ReplaceExitValue == OnlyCheapRepl && !LoopCanBeDel && Phi.HighCost) { |
| 676 | DeadInsts.push_back(ExitVal); |
| 677 | continue; |
| 678 | } |
| 679 | |
| 680 | Changed = true; |
| 681 | ++NumReplaced; |
| 682 | Instruction *Inst = cast<Instruction>(PN->getIncomingValue(Phi.Ith)); |
| 683 | PN->setIncomingValue(Phi.Ith, ExitVal); |
| 684 | |
| 685 | // If this instruction is dead now, delete it. Don't do it now to avoid |
| 686 | // invalidating iterators. |
| 687 | if (isInstructionTriviallyDead(Inst, TLI)) |
| 688 | DeadInsts.push_back(Inst); |
| 689 | |
| Sanjoy Das | de47590 | 2016-01-17 18:12:52 +0000 | [diff] [blame] | 690 | // Replace PN with ExitVal if that is legal and does not break LCSSA. |
| 691 | if (PN->getNumIncomingValues() == 1 && |
| 692 | LI->replacementPreservesLCSSAForm(PN, ExitVal)) { |
| Wei Mi | e2538b5 | 2015-05-28 21:49:07 +0000 | [diff] [blame] | 693 | PN->replaceAllUsesWith(ExitVal); |
| 694 | PN->eraseFromParent(); |
| 695 | } |
| 696 | } |
| 697 | |
| Dan Gohman | 1a2abe5 | 2010-03-20 03:53:53 +0000 | [diff] [blame] | 698 | // The insertion point instruction may have been deleted; clear it out |
| 699 | // so that the rewriter doesn't trip over it later. |
| 700 | Rewriter.clearInsertPoint(); |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 701 | return Changed; |
| Chris Lattner | e61b67d | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 702 | } |
| 703 | |
| Chen Li | 5cde838 | 2016-01-27 07:40:41 +0000 | [diff] [blame] | 704 | //===---------------------------------------------------------------------===// |
| 705 | // rewriteFirstIterationLoopExitValues: Rewrite loop exit values if we know |
| 706 | // they will exit at the first iteration. |
| 707 | //===---------------------------------------------------------------------===// |
| 708 | |
| 709 | /// Check to see if this loop has loop invariant conditions which lead to loop |
| 710 | /// exits. If so, we know that if the exit path is taken, it is at the first |
| 711 | /// loop iteration. This lets us predict exit values of PHI nodes that live in |
| 712 | /// loop header. |
| Max Kazantsev | fde8857 | 2018-09-10 06:50:16 +0000 | [diff] [blame] | 713 | bool IndVarSimplify::rewriteFirstIterationLoopExitValues(Loop *L) { |
| Chen Li | 5cde838 | 2016-01-27 07:40:41 +0000 | [diff] [blame] | 714 | // Verify the input to the pass is already in LCSSA form. |
| 715 | assert(L->isLCSSAForm(*DT)); |
| 716 | |
| 717 | SmallVector<BasicBlock *, 8> ExitBlocks; |
| 718 | L->getUniqueExitBlocks(ExitBlocks); |
| Chen Li | 5cde838 | 2016-01-27 07:40:41 +0000 | [diff] [blame] | 719 | |
| Max Kazantsev | fde8857 | 2018-09-10 06:50:16 +0000 | [diff] [blame] | 720 | bool MadeAnyChanges = false; |
| Chen Li | 5cde838 | 2016-01-27 07:40:41 +0000 | [diff] [blame] | 721 | for (auto *ExitBB : ExitBlocks) { |
| Chen Li | 5cde838 | 2016-01-27 07:40:41 +0000 | [diff] [blame] | 722 | // If there are no more PHI nodes in this exit block, then no more |
| 723 | // values defined inside the loop are used on this path. |
| Benjamin Kramer | c7fc81e | 2017-12-30 15:27:33 +0000 | [diff] [blame] | 724 | for (PHINode &PN : ExitBB->phis()) { |
| 725 | for (unsigned IncomingValIdx = 0, E = PN.getNumIncomingValues(); |
| 726 | IncomingValIdx != E; ++IncomingValIdx) { |
| 727 | auto *IncomingBB = PN.getIncomingBlock(IncomingValIdx); |
| Chen Li | 5cde838 | 2016-01-27 07:40:41 +0000 | [diff] [blame] | 728 | |
| Philip Reames | bd8d309 | 2019-05-14 17:20:10 +0000 | [diff] [blame] | 729 | // Can we prove that the exit must run on the first iteration if it |
| 730 | // runs at all? (i.e. early exits are fine for our purposes, but |
| 731 | // traces which lead to this exit being taken on the 2nd iteration |
| 732 | // aren't.) Note that this is about whether the exit branch is |
| 733 | // executed, not about whether it is taken. |
| 734 | if (!L->getLoopLatch() || |
| 735 | !DT->dominates(IncomingBB, L->getLoopLatch())) |
| Chen Li | 5cde838 | 2016-01-27 07:40:41 +0000 | [diff] [blame] | 736 | continue; |
| 737 | |
| 738 | // Get condition that leads to the exit path. |
| 739 | auto *TermInst = IncomingBB->getTerminator(); |
| 740 | |
| 741 | Value *Cond = nullptr; |
| 742 | if (auto *BI = dyn_cast<BranchInst>(TermInst)) { |
| 743 | // Must be a conditional branch, otherwise the block |
| 744 | // should not be in the loop. |
| 745 | Cond = BI->getCondition(); |
| 746 | } else if (auto *SI = dyn_cast<SwitchInst>(TermInst)) |
| 747 | Cond = SI->getCondition(); |
| 748 | else |
| 749 | continue; |
| 750 | |
| 751 | if (!L->isLoopInvariant(Cond)) |
| 752 | continue; |
| 753 | |
| Benjamin Kramer | c7fc81e | 2017-12-30 15:27:33 +0000 | [diff] [blame] | 754 | auto *ExitVal = dyn_cast<PHINode>(PN.getIncomingValue(IncomingValIdx)); |
| Chen Li | 5cde838 | 2016-01-27 07:40:41 +0000 | [diff] [blame] | 755 | |
| Philip Reames | bd8d309 | 2019-05-14 17:20:10 +0000 | [diff] [blame] | 756 | // Only deal with PHIs in the loop header. |
| 757 | if (!ExitVal || ExitVal->getParent() != L->getHeader()) |
| Chen Li | 5cde838 | 2016-01-27 07:40:41 +0000 | [diff] [blame] | 758 | continue; |
| 759 | |
| 760 | // If ExitVal is a PHI on the loop header, then we know its |
| 761 | // value along this exit because the exit can only be taken |
| 762 | // on the first iteration. |
| 763 | auto *LoopPreheader = L->getLoopPreheader(); |
| 764 | assert(LoopPreheader && "Invalid loop"); |
| 765 | int PreheaderIdx = ExitVal->getBasicBlockIndex(LoopPreheader); |
| 766 | if (PreheaderIdx != -1) { |
| Philip Reames | 75ad8c5 | 2019-05-14 17:50:06 +0000 | [diff] [blame] | 767 | assert(ExitVal->getParent() == L->getHeader() && |
| Chen Li | 5cde838 | 2016-01-27 07:40:41 +0000 | [diff] [blame] | 768 | "ExitVal must be in loop header"); |
| Max Kazantsev | fde8857 | 2018-09-10 06:50:16 +0000 | [diff] [blame] | 769 | MadeAnyChanges = true; |
| Benjamin Kramer | c7fc81e | 2017-12-30 15:27:33 +0000 | [diff] [blame] | 770 | PN.setIncomingValue(IncomingValIdx, |
| 771 | ExitVal->getIncomingValue(PreheaderIdx)); |
| Chen Li | 5cde838 | 2016-01-27 07:40:41 +0000 | [diff] [blame] | 772 | } |
| 773 | } |
| 774 | } |
| 775 | } |
| Max Kazantsev | fde8857 | 2018-09-10 06:50:16 +0000 | [diff] [blame] | 776 | return MadeAnyChanges; |
| Chen Li | 5cde838 | 2016-01-27 07:40:41 +0000 | [diff] [blame] | 777 | } |
| 778 | |
| Sanjoy Das | 9119bf4 | 2015-09-20 06:58:03 +0000 | [diff] [blame] | 779 | /// Check whether it is possible to delete the loop after rewriting exit |
| 780 | /// value. If it is possible, ignore ReplaceExitValue and do rewriting |
| 781 | /// aggressively. |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 782 | bool IndVarSimplify::canLoopBeDeleted( |
| Wei Mi | e2538b5 | 2015-05-28 21:49:07 +0000 | [diff] [blame] | 783 | Loop *L, SmallVector<RewritePhi, 8> &RewritePhiSet) { |
| Wei Mi | e2538b5 | 2015-05-28 21:49:07 +0000 | [diff] [blame] | 784 | BasicBlock *Preheader = L->getLoopPreheader(); |
| 785 | // If there is no preheader, the loop will not be deleted. |
| 786 | if (!Preheader) |
| 787 | return false; |
| 788 | |
| 789 | // In LoopDeletion pass Loop can be deleted when ExitingBlocks.size() > 1. |
| 790 | // We obviate multiple ExitingBlocks case for simplicity. |
| 791 | // TODO: If we see testcase with multiple ExitingBlocks can be deleted |
| 792 | // after exit value rewriting, we can enhance the logic here. |
| 793 | SmallVector<BasicBlock *, 4> ExitingBlocks; |
| 794 | L->getExitingBlocks(ExitingBlocks); |
| 795 | SmallVector<BasicBlock *, 8> ExitBlocks; |
| 796 | L->getUniqueExitBlocks(ExitBlocks); |
| 797 | if (ExitBlocks.size() > 1 || ExitingBlocks.size() > 1) |
| 798 | return false; |
| 799 | |
| 800 | BasicBlock *ExitBlock = ExitBlocks[0]; |
| 801 | BasicBlock::iterator BI = ExitBlock->begin(); |
| 802 | while (PHINode *P = dyn_cast<PHINode>(BI)) { |
| 803 | Value *Incoming = P->getIncomingValueForBlock(ExitingBlocks[0]); |
| 804 | |
| 805 | // If the Incoming value of P is found in RewritePhiSet, we know it |
| 806 | // could be rewritten to use a loop invariant value in transformation |
| 807 | // phase later. Skip it in the loop invariant check below. |
| 808 | bool found = false; |
| 809 | for (const RewritePhi &Phi : RewritePhiSet) { |
| 810 | unsigned i = Phi.Ith; |
| 811 | if (Phi.PN == P && (Phi.PN)->getIncomingValue(i) == Incoming) { |
| 812 | found = true; |
| 813 | break; |
| 814 | } |
| 815 | } |
| 816 | |
| 817 | Instruction *I; |
| 818 | if (!found && (I = dyn_cast<Instruction>(Incoming))) |
| 819 | if (!L->hasLoopInvariantOperands(I)) |
| 820 | return false; |
| 821 | |
| 822 | ++BI; |
| 823 | } |
| 824 | |
| Sanjoy Das | 42e551b | 2015-12-08 23:52:58 +0000 | [diff] [blame] | 825 | for (auto *BB : L->blocks()) |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 826 | if (llvm::any_of(*BB, [](Instruction &I) { |
| 827 | return I.mayHaveSideEffects(); |
| 828 | })) |
| Sanjoy Das | 42e551b | 2015-12-08 23:52:58 +0000 | [diff] [blame] | 829 | return false; |
| Wei Mi | e2538b5 | 2015-05-28 21:49:07 +0000 | [diff] [blame] | 830 | |
| 831 | return true; |
| 832 | } |
| 833 | |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 834 | //===----------------------------------------------------------------------===// |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 835 | // IV Widening - Extend the width of an IV to cover its widest uses. |
| 836 | //===----------------------------------------------------------------------===// |
| 837 | |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 838 | namespace { |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 839 | |
| Sanjoy Das | e1e352d | 2015-09-20 18:42:50 +0000 | [diff] [blame] | 840 | // Collect information about induction variables that are used by sign/zero |
| 841 | // extend operations. This information is recorded by CollectExtend and provides |
| 842 | // the input to WidenIV. |
| 843 | struct WideIVInfo { |
| Sanjoy Das | 7cc2cfe | 2015-09-20 18:42:53 +0000 | [diff] [blame] | 844 | PHINode *NarrowIV = nullptr; |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 845 | |
| 846 | // Widest integer type created [sz]ext |
| 847 | Type *WidestNativeType = nullptr; |
| 848 | |
| 849 | // Was a sext user seen before a zext? |
| 850 | bool IsSigned = false; |
| Sanjoy Das | e1e352d | 2015-09-20 18:42:50 +0000 | [diff] [blame] | 851 | }; |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 852 | |
| 853 | } // end anonymous namespace |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 854 | |
| Sanjoy Das | 9119bf4 | 2015-09-20 06:58:03 +0000 | [diff] [blame] | 855 | /// Update information about the induction variable that is extended by this |
| 856 | /// sign or zero extend operation. This is used to determine the final width of |
| 857 | /// the IV before actually widening it. |
| Andrew Trick | b6bc783 | 2014-01-02 21:12:11 +0000 | [diff] [blame] | 858 | static void visitIVCast(CastInst *Cast, WideIVInfo &WI, ScalarEvolution *SE, |
| Mehdi Amini | a28d91d | 2015-03-10 02:37:25 +0000 | [diff] [blame] | 859 | const TargetTransformInfo *TTI) { |
| Andrew Trick | 3ec331e | 2011-08-10 03:46:27 +0000 | [diff] [blame] | 860 | bool IsSigned = Cast->getOpcode() == Instruction::SExt; |
| 861 | if (!IsSigned && Cast->getOpcode() != Instruction::ZExt) |
| 862 | return; |
| 863 | |
| Chris Lattner | 229907c | 2011-07-18 04:54:35 +0000 | [diff] [blame] | 864 | Type *Ty = Cast->getType(); |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 865 | uint64_t Width = SE->getTypeSizeInBits(Ty); |
| Mehdi Amini | a28d91d | 2015-03-10 02:37:25 +0000 | [diff] [blame] | 866 | if (!Cast->getModule()->getDataLayout().isLegalInteger(Width)) |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 867 | return; |
| 868 | |
| Sanjoy Das | 3502511 | 2016-08-13 00:58:31 +0000 | [diff] [blame] | 869 | // Check that `Cast` actually extends the induction variable (we rely on this |
| 870 | // later). This takes care of cases where `Cast` is extending a truncation of |
| 871 | // the narrow induction variable, and thus can end up being narrower than the |
| 872 | // "narrow" induction variable. |
| 873 | uint64_t NarrowIVWidth = SE->getTypeSizeInBits(WI.NarrowIV->getType()); |
| 874 | if (NarrowIVWidth >= Width) |
| 875 | return; |
| 876 | |
| Jingyue Wu | 8a12cea | 2014-11-12 18:09:15 +0000 | [diff] [blame] | 877 | // Cast is either an sext or zext up to this point. |
| 878 | // We should not widen an indvar if arithmetics on the wider indvar are more |
| 879 | // expensive than those on the narrower indvar. We check only the cost of ADD |
| 880 | // because at least an ADD is required to increment the induction variable. We |
| 881 | // could compute more comprehensively the cost of all instructions on the |
| 882 | // induction variable when necessary. |
| 883 | if (TTI && |
| 884 | TTI->getArithmeticInstrCost(Instruction::Add, Ty) > |
| 885 | TTI->getArithmeticInstrCost(Instruction::Add, |
| 886 | Cast->getOperand(0)->getType())) { |
| 887 | return; |
| 888 | } |
| 889 | |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 890 | if (!WI.WidestNativeType) { |
| 891 | WI.WidestNativeType = SE->getEffectiveSCEVType(Ty); |
| 892 | WI.IsSigned = IsSigned; |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 893 | return; |
| 894 | } |
| 895 | |
| 896 | // We extend the IV to satisfy the sign of its first user, arbitrarily. |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 897 | if (WI.IsSigned != IsSigned) |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 898 | return; |
| 899 | |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 900 | if (Width > SE->getTypeSizeInBits(WI.WidestNativeType)) |
| 901 | WI.WidestNativeType = SE->getEffectiveSCEVType(Ty); |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 902 | } |
| 903 | |
| 904 | namespace { |
| Andrew Trick | 2210448 | 2011-07-20 04:39:24 +0000 | [diff] [blame] | 905 | |
| Sanjoy Das | 9119bf4 | 2015-09-20 06:58:03 +0000 | [diff] [blame] | 906 | /// Record a link in the Narrow IV def-use chain along with the WideIV that |
| 907 | /// computes the same value as the Narrow IV def. This avoids caching Use* |
| 908 | /// pointers. |
| Andrew Trick | 2210448 | 2011-07-20 04:39:24 +0000 | [diff] [blame] | 909 | struct NarrowIVDefUse { |
| Sanjoy Das | 7cc2cfe | 2015-09-20 18:42:53 +0000 | [diff] [blame] | 910 | Instruction *NarrowDef = nullptr; |
| 911 | Instruction *NarrowUse = nullptr; |
| 912 | Instruction *WideDef = nullptr; |
| Andrew Trick | 2210448 | 2011-07-20 04:39:24 +0000 | [diff] [blame] | 913 | |
| Sanjoy Das | 428db15 | 2015-09-20 01:52:18 +0000 | [diff] [blame] | 914 | // True if the narrow def is never negative. Tracking this information lets |
| 915 | // us use a sign extension instead of a zero extension or vice versa, when |
| 916 | // profitable and legal. |
| Sanjoy Das | 7cc2cfe | 2015-09-20 18:42:53 +0000 | [diff] [blame] | 917 | bool NeverNegative = false; |
| Sanjoy Das | 428db15 | 2015-09-20 01:52:18 +0000 | [diff] [blame] | 918 | |
| 919 | NarrowIVDefUse(Instruction *ND, Instruction *NU, Instruction *WD, |
| 920 | bool NeverNegative) |
| 921 | : NarrowDef(ND), NarrowUse(NU), WideDef(WD), |
| 922 | NeverNegative(NeverNegative) {} |
| Andrew Trick | 2210448 | 2011-07-20 04:39:24 +0000 | [diff] [blame] | 923 | }; |
| 924 | |
| Sanjoy Das | 9119bf4 | 2015-09-20 06:58:03 +0000 | [diff] [blame] | 925 | /// The goal of this transform is to remove sign and zero extends without |
| 926 | /// creating any new induction variables. To do this, it creates a new phi of |
| 927 | /// the wider type and redirects all users, either removing extends or inserting |
| 928 | /// truncs whenever we stop propagating the type. |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 929 | class WidenIV { |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 930 | // Parameters |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 931 | PHINode *OrigPhi; |
| Chris Lattner | 229907c | 2011-07-18 04:54:35 +0000 | [diff] [blame] | 932 | Type *WideType; |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 933 | |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 934 | // Context |
| 935 | LoopInfo *LI; |
| 936 | Loop *L; |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 937 | ScalarEvolution *SE; |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 938 | DominatorTree *DT; |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 939 | |
| Artur Pilipenko | 5c6ef75 | 2016-10-19 19:43:54 +0000 | [diff] [blame] | 940 | // Does the module have any calls to the llvm.experimental.guard intrinsic |
| 941 | // at all? If not we can avoid scanning instructions looking for guards. |
| 942 | bool HasGuards; |
| 943 | |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 944 | // Result |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 945 | PHINode *WidePhi = nullptr; |
| 946 | Instruction *WideInc = nullptr; |
| 947 | const SCEV *WideIncExpr = nullptr; |
| Sanjoy Das | e6bca0e | 2017-05-01 17:07:49 +0000 | [diff] [blame] | 948 | SmallVectorImpl<WeakTrackingVH> &DeadInsts; |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 949 | |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 950 | SmallPtrSet<Instruction *,16> Widened; |
| Andrew Trick | 2210448 | 2011-07-20 04:39:24 +0000 | [diff] [blame] | 951 | SmallVector<NarrowIVDefUse, 8> NarrowIVUsers; |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 952 | |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 953 | enum ExtendKind { ZeroExtended, SignExtended, Unknown }; |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 954 | |
| Simon Pilgrim | 610ad9b | 2017-03-20 13:55:35 +0000 | [diff] [blame] | 955 | // A map tracking the kind of extension used to widen each narrow IV |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 956 | // and narrow IV user. |
| 957 | // Key: pointer to a narrow IV or IV user. |
| 958 | // Value: the kind of extension used to widen this Instruction. |
| 959 | DenseMap<AssertingVH<Instruction>, ExtendKind> ExtendKindMap; |
| 960 | |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 961 | using DefUserPair = std::pair<AssertingVH<Value>, AssertingVH<Instruction>>; |
| 962 | |
| Artur Pilipenko | f2d5dc5 | 2016-10-19 18:59:03 +0000 | [diff] [blame] | 963 | // A map with control-dependent ranges for post increment IV uses. The key is |
| 964 | // a pair of IV def and a use of this def denoting the context. The value is |
| 965 | // a ConstantRange representing possible values of the def at the given |
| 966 | // context. |
| 967 | DenseMap<DefUserPair, ConstantRange> PostIncRangeInfos; |
| 968 | |
| 969 | Optional<ConstantRange> getPostIncRangeInfo(Value *Def, |
| 970 | Instruction *UseI) { |
| 971 | DefUserPair Key(Def, UseI); |
| 972 | auto It = PostIncRangeInfos.find(Key); |
| 973 | return It == PostIncRangeInfos.end() |
| 974 | ? Optional<ConstantRange>(None) |
| 975 | : Optional<ConstantRange>(It->second); |
| 976 | } |
| 977 | |
| 978 | void calculatePostIncRanges(PHINode *OrigPhi); |
| 979 | void calculatePostIncRange(Instruction *NarrowDef, Instruction *NarrowUser); |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 980 | |
| Artur Pilipenko | f2d5dc5 | 2016-10-19 18:59:03 +0000 | [diff] [blame] | 981 | void updatePostIncRangeInfo(Value *Def, Instruction *UseI, ConstantRange R) { |
| 982 | DefUserPair Key(Def, UseI); |
| 983 | auto It = PostIncRangeInfos.find(Key); |
| 984 | if (It == PostIncRangeInfos.end()) |
| 985 | PostIncRangeInfos.insert({Key, R}); |
| 986 | else |
| 987 | It->second = R.intersectWith(It->second); |
| 988 | } |
| 989 | |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 990 | public: |
| Sanjoy Das | e6bca0e | 2017-05-01 17:07:49 +0000 | [diff] [blame] | 991 | WidenIV(const WideIVInfo &WI, LoopInfo *LInfo, ScalarEvolution *SEv, |
| 992 | DominatorTree *DTree, SmallVectorImpl<WeakTrackingVH> &DI, |
| 993 | bool HasGuards) |
| 994 | : OrigPhi(WI.NarrowIV), WideType(WI.WidestNativeType), LI(LInfo), |
| 995 | L(LI->getLoopFor(OrigPhi->getParent())), SE(SEv), DT(DTree), |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 996 | HasGuards(HasGuards), DeadInsts(DI) { |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 997 | assert(L->getHeader() == OrigPhi->getParent() && "Phi must be an IV"); |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 998 | ExtendKindMap[OrigPhi] = WI.IsSigned ? SignExtended : ZeroExtended; |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 999 | } |
| 1000 | |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 1001 | PHINode *createWideIV(SCEVExpander &Rewriter); |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1002 | |
| 1003 | protected: |
| Sanjoy Das | 7360f30 | 2015-10-16 01:00:50 +0000 | [diff] [blame] | 1004 | Value *createExtendInst(Value *NarrowOper, Type *WideType, bool IsSigned, |
| 1005 | Instruction *Use); |
| Andrew Trick | e0e3053 | 2011-09-28 01:35:36 +0000 | [diff] [blame] | 1006 | |
| Sanjoy Das | 1fd184e | 2015-10-16 01:00:39 +0000 | [diff] [blame] | 1007 | Instruction *cloneIVUser(NarrowIVDefUse DU, const SCEVAddRecExpr *WideAR); |
| 1008 | Instruction *cloneArithmeticIVUser(NarrowIVDefUse DU, |
| 1009 | const SCEVAddRecExpr *WideAR); |
| 1010 | Instruction *cloneBitwiseIVUser(NarrowIVDefUse DU); |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1011 | |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1012 | ExtendKind getExtendKind(Instruction *I); |
| Andrew Trick | 92905a1 | 2011-07-05 18:19:39 +0000 | [diff] [blame] | 1013 | |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 1014 | using WidenedRecTy = std::pair<const SCEVAddRecExpr *, ExtendKind>; |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1015 | |
| 1016 | WidenedRecTy getWideRecurrence(NarrowIVDefUse DU); |
| 1017 | |
| 1018 | WidenedRecTy getExtendedOperandRecurrence(NarrowIVDefUse DU); |
| Andrew Trick | c7868bf0 | 2011-09-10 01:24:17 +0000 | [diff] [blame] | 1019 | |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 1020 | const SCEV *getSCEVByOpCode(const SCEV *LHS, const SCEV *RHS, |
| Zinovy Nis | 0a36cba | 2014-08-21 08:25:45 +0000 | [diff] [blame] | 1021 | unsigned OpCode) const; |
| 1022 | |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 1023 | Instruction *widenIVUse(NarrowIVDefUse DU, SCEVExpander &Rewriter); |
| Andrew Trick | 6d12309 | 2011-07-02 02:34:25 +0000 | [diff] [blame] | 1024 | |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 1025 | bool widenLoopCompare(NarrowIVDefUse DU); |
| Abderrazek Zaafrani | c30dfb2 | 2018-09-07 22:41:57 +0000 | [diff] [blame] | 1026 | bool widenWithVariantLoadUse(NarrowIVDefUse DU); |
| 1027 | void widenWithVariantLoadUseCodegen(NarrowIVDefUse DU); |
| Chad Rosier | bb99f40 | 2014-09-17 14:10:33 +0000 | [diff] [blame] | 1028 | |
| Andrew Trick | 6d12309 | 2011-07-02 02:34:25 +0000 | [diff] [blame] | 1029 | void pushNarrowIVUsers(Instruction *NarrowDef, Instruction *WideDef); |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1030 | }; |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 1031 | |
| 1032 | } // end anonymous namespace |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1033 | |
| Sanjoy Das | 7360f30 | 2015-10-16 01:00:50 +0000 | [diff] [blame] | 1034 | Value *WidenIV::createExtendInst(Value *NarrowOper, Type *WideType, |
| 1035 | bool IsSigned, Instruction *Use) { |
| Andrew Trick | e0e3053 | 2011-09-28 01:35:36 +0000 | [diff] [blame] | 1036 | // Set the debug location and conservative insertion point. |
| 1037 | IRBuilder<> Builder(Use); |
| 1038 | // Hoist the insertion point into loop preheaders as far as possible. |
| 1039 | for (const Loop *L = LI->getLoopFor(Use->getParent()); |
| Philip Reames | 45e7690 | 2019-05-17 02:09:03 +0000 | [diff] [blame^] | 1040 | L && L->getLoopPreheader() && L->isLoopInvariant(NarrowOper); |
| Andrew Trick | e0e3053 | 2011-09-28 01:35:36 +0000 | [diff] [blame] | 1041 | L = L->getParentLoop()) |
| 1042 | Builder.SetInsertPoint(L->getLoopPreheader()->getTerminator()); |
| 1043 | |
| Andrew Trick | eb3c36e | 2011-05-25 04:42:22 +0000 | [diff] [blame] | 1044 | return IsSigned ? Builder.CreateSExt(NarrowOper, WideType) : |
| 1045 | Builder.CreateZExt(NarrowOper, WideType); |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1046 | } |
| 1047 | |
| Sanjoy Das | 9119bf4 | 2015-09-20 06:58:03 +0000 | [diff] [blame] | 1048 | /// Instantiate a wide operation to replace a narrow operation. This only needs |
| 1049 | /// to handle operations that can evaluation to SCEVAddRec. It can safely return |
| 1050 | /// 0 for any operation we decide not to clone. |
| Sanjoy Das | 1fd184e | 2015-10-16 01:00:39 +0000 | [diff] [blame] | 1051 | Instruction *WidenIV::cloneIVUser(NarrowIVDefUse DU, |
| 1052 | const SCEVAddRecExpr *WideAR) { |
| Andrew Trick | 2210448 | 2011-07-20 04:39:24 +0000 | [diff] [blame] | 1053 | unsigned Opcode = DU.NarrowUse->getOpcode(); |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1054 | switch (Opcode) { |
| 1055 | default: |
| Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 1056 | return nullptr; |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1057 | case Instruction::Add: |
| 1058 | case Instruction::Mul: |
| 1059 | case Instruction::UDiv: |
| 1060 | case Instruction::Sub: |
| Sanjoy Das | 1fd184e | 2015-10-16 01:00:39 +0000 | [diff] [blame] | 1061 | return cloneArithmeticIVUser(DU, WideAR); |
| 1062 | |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1063 | case Instruction::And: |
| 1064 | case Instruction::Or: |
| 1065 | case Instruction::Xor: |
| 1066 | case Instruction::Shl: |
| 1067 | case Instruction::LShr: |
| 1068 | case Instruction::AShr: |
| Sanjoy Das | 1fd184e | 2015-10-16 01:00:39 +0000 | [diff] [blame] | 1069 | return cloneBitwiseIVUser(DU); |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1070 | } |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1071 | } |
| 1072 | |
| Sanjoy Das | 1fd184e | 2015-10-16 01:00:39 +0000 | [diff] [blame] | 1073 | Instruction *WidenIV::cloneBitwiseIVUser(NarrowIVDefUse DU) { |
| Sanjoy Das | 472840a | 2015-10-16 01:00:44 +0000 | [diff] [blame] | 1074 | Instruction *NarrowUse = DU.NarrowUse; |
| 1075 | Instruction *NarrowDef = DU.NarrowDef; |
| 1076 | Instruction *WideDef = DU.WideDef; |
| 1077 | |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 1078 | LLVM_DEBUG(dbgs() << "Cloning bitwise IVUser: " << *NarrowUse << "\n"); |
| Sanjoy Das | 1fd184e | 2015-10-16 01:00:39 +0000 | [diff] [blame] | 1079 | |
| 1080 | // Replace NarrowDef operands with WideDef. Otherwise, we don't know anything |
| 1081 | // about the narrow operand yet so must insert a [sz]ext. It is probably loop |
| 1082 | // invariant and will be folded or hoisted. If it actually comes from a |
| 1083 | // widened IV, it should be removed during a future call to widenIVUse. |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1084 | bool IsSigned = getExtendKind(NarrowDef) == SignExtended; |
| Sanjoy Das | 7360f30 | 2015-10-16 01:00:50 +0000 | [diff] [blame] | 1085 | Value *LHS = (NarrowUse->getOperand(0) == NarrowDef) |
| 1086 | ? WideDef |
| 1087 | : createExtendInst(NarrowUse->getOperand(0), WideType, |
| 1088 | IsSigned, NarrowUse); |
| 1089 | Value *RHS = (NarrowUse->getOperand(1) == NarrowDef) |
| 1090 | ? WideDef |
| 1091 | : createExtendInst(NarrowUse->getOperand(1), WideType, |
| 1092 | IsSigned, NarrowUse); |
| Sanjoy Das | 1fd184e | 2015-10-16 01:00:39 +0000 | [diff] [blame] | 1093 | |
| Sanjoy Das | 472840a | 2015-10-16 01:00:44 +0000 | [diff] [blame] | 1094 | auto *NarrowBO = cast<BinaryOperator>(NarrowUse); |
| Sanjoy Das | 1fd184e | 2015-10-16 01:00:39 +0000 | [diff] [blame] | 1095 | auto *WideBO = BinaryOperator::Create(NarrowBO->getOpcode(), LHS, RHS, |
| 1096 | NarrowBO->getName()); |
| Sanjoy Das | 472840a | 2015-10-16 01:00:44 +0000 | [diff] [blame] | 1097 | IRBuilder<> Builder(NarrowUse); |
| Sanjoy Das | 1fd184e | 2015-10-16 01:00:39 +0000 | [diff] [blame] | 1098 | Builder.Insert(WideBO); |
| Sanjay Patel | 739f2ce | 2015-11-24 17:16:33 +0000 | [diff] [blame] | 1099 | WideBO->copyIRFlags(NarrowBO); |
| Sanjoy Das | 1fd184e | 2015-10-16 01:00:39 +0000 | [diff] [blame] | 1100 | return WideBO; |
| 1101 | } |
| 1102 | |
| 1103 | Instruction *WidenIV::cloneArithmeticIVUser(NarrowIVDefUse DU, |
| 1104 | const SCEVAddRecExpr *WideAR) { |
| Sanjoy Das | 472840a | 2015-10-16 01:00:44 +0000 | [diff] [blame] | 1105 | Instruction *NarrowUse = DU.NarrowUse; |
| 1106 | Instruction *NarrowDef = DU.NarrowDef; |
| 1107 | Instruction *WideDef = DU.WideDef; |
| 1108 | |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 1109 | LLVM_DEBUG(dbgs() << "Cloning arithmetic IVUser: " << *NarrowUse << "\n"); |
| Sanjoy Das | 1fd184e | 2015-10-16 01:00:39 +0000 | [diff] [blame] | 1110 | |
| Sanjoy Das | 37e87c2 | 2015-10-16 01:00:47 +0000 | [diff] [blame] | 1111 | unsigned IVOpIdx = (NarrowUse->getOperand(0) == NarrowDef) ? 0 : 1; |
| 1112 | |
| 1113 | // We're trying to find X such that |
| 1114 | // |
| 1115 | // Widen(NarrowDef `op` NonIVNarrowDef) == WideAR == WideDef `op.wide` X |
| 1116 | // |
| 1117 | // We guess two solutions to X, sext(NonIVNarrowDef) and zext(NonIVNarrowDef), |
| 1118 | // and check using SCEV if any of them are correct. |
| 1119 | |
| 1120 | // Returns true if extending NonIVNarrowDef according to `SignExt` is a |
| 1121 | // correct solution to X. |
| 1122 | auto GuessNonIVOperand = [&](bool SignExt) { |
| 1123 | const SCEV *WideLHS; |
| 1124 | const SCEV *WideRHS; |
| 1125 | |
| 1126 | auto GetExtend = [this, SignExt](const SCEV *S, Type *Ty) { |
| 1127 | if (SignExt) |
| 1128 | return SE->getSignExtendExpr(S, Ty); |
| 1129 | return SE->getZeroExtendExpr(S, Ty); |
| 1130 | }; |
| 1131 | |
| 1132 | if (IVOpIdx == 0) { |
| 1133 | WideLHS = SE->getSCEV(WideDef); |
| 1134 | const SCEV *NarrowRHS = SE->getSCEV(NarrowUse->getOperand(1)); |
| 1135 | WideRHS = GetExtend(NarrowRHS, WideType); |
| 1136 | } else { |
| 1137 | const SCEV *NarrowLHS = SE->getSCEV(NarrowUse->getOperand(0)); |
| 1138 | WideLHS = GetExtend(NarrowLHS, WideType); |
| 1139 | WideRHS = SE->getSCEV(WideDef); |
| 1140 | } |
| 1141 | |
| 1142 | // WideUse is "WideDef `op.wide` X" as described in the comment. |
| 1143 | const SCEV *WideUse = nullptr; |
| 1144 | |
| 1145 | switch (NarrowUse->getOpcode()) { |
| 1146 | default: |
| 1147 | llvm_unreachable("No other possibility!"); |
| 1148 | |
| 1149 | case Instruction::Add: |
| 1150 | WideUse = SE->getAddExpr(WideLHS, WideRHS); |
| 1151 | break; |
| 1152 | |
| 1153 | case Instruction::Mul: |
| 1154 | WideUse = SE->getMulExpr(WideLHS, WideRHS); |
| 1155 | break; |
| 1156 | |
| 1157 | case Instruction::UDiv: |
| 1158 | WideUse = SE->getUDivExpr(WideLHS, WideRHS); |
| 1159 | break; |
| 1160 | |
| 1161 | case Instruction::Sub: |
| 1162 | WideUse = SE->getMinusSCEV(WideLHS, WideRHS); |
| 1163 | break; |
| 1164 | } |
| 1165 | |
| 1166 | return WideUse == WideAR; |
| 1167 | }; |
| 1168 | |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1169 | bool SignExtend = getExtendKind(NarrowDef) == SignExtended; |
| Sanjoy Das | 37e87c2 | 2015-10-16 01:00:47 +0000 | [diff] [blame] | 1170 | if (!GuessNonIVOperand(SignExtend)) { |
| 1171 | SignExtend = !SignExtend; |
| 1172 | if (!GuessNonIVOperand(SignExtend)) |
| 1173 | return nullptr; |
| 1174 | } |
| 1175 | |
| 1176 | Value *LHS = (NarrowUse->getOperand(0) == NarrowDef) |
| 1177 | ? WideDef |
| Sanjoy Das | 7360f30 | 2015-10-16 01:00:50 +0000 | [diff] [blame] | 1178 | : createExtendInst(NarrowUse->getOperand(0), WideType, |
| 1179 | SignExtend, NarrowUse); |
| Sanjoy Das | 37e87c2 | 2015-10-16 01:00:47 +0000 | [diff] [blame] | 1180 | Value *RHS = (NarrowUse->getOperand(1) == NarrowDef) |
| 1181 | ? WideDef |
| Sanjoy Das | 7360f30 | 2015-10-16 01:00:50 +0000 | [diff] [blame] | 1182 | : createExtendInst(NarrowUse->getOperand(1), WideType, |
| 1183 | SignExtend, NarrowUse); |
| Sanjoy Das | 1fd184e | 2015-10-16 01:00:39 +0000 | [diff] [blame] | 1184 | |
| Sanjoy Das | 472840a | 2015-10-16 01:00:44 +0000 | [diff] [blame] | 1185 | auto *NarrowBO = cast<BinaryOperator>(NarrowUse); |
| Sanjoy Das | 1fd184e | 2015-10-16 01:00:39 +0000 | [diff] [blame] | 1186 | auto *WideBO = BinaryOperator::Create(NarrowBO->getOpcode(), LHS, RHS, |
| 1187 | NarrowBO->getName()); |
| Sanjoy Das | 37e87c2 | 2015-10-16 01:00:47 +0000 | [diff] [blame] | 1188 | |
| Sanjoy Das | 472840a | 2015-10-16 01:00:44 +0000 | [diff] [blame] | 1189 | IRBuilder<> Builder(NarrowUse); |
| Sanjoy Das | 1fd184e | 2015-10-16 01:00:39 +0000 | [diff] [blame] | 1190 | Builder.Insert(WideBO); |
| Sanjay Patel | 739f2ce | 2015-11-24 17:16:33 +0000 | [diff] [blame] | 1191 | WideBO->copyIRFlags(NarrowBO); |
| Sanjoy Das | 1fd184e | 2015-10-16 01:00:39 +0000 | [diff] [blame] | 1192 | return WideBO; |
| 1193 | } |
| 1194 | |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1195 | WidenIV::ExtendKind WidenIV::getExtendKind(Instruction *I) { |
| 1196 | auto It = ExtendKindMap.find(I); |
| 1197 | assert(It != ExtendKindMap.end() && "Instruction not yet extended!"); |
| 1198 | return It->second; |
| 1199 | } |
| 1200 | |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 1201 | const SCEV *WidenIV::getSCEVByOpCode(const SCEV *LHS, const SCEV *RHS, |
| Zinovy Nis | 0a36cba | 2014-08-21 08:25:45 +0000 | [diff] [blame] | 1202 | unsigned OpCode) const { |
| 1203 | if (OpCode == Instruction::Add) |
| 1204 | return SE->getAddExpr(LHS, RHS); |
| 1205 | if (OpCode == Instruction::Sub) |
| 1206 | return SE->getMinusSCEV(LHS, RHS); |
| 1207 | if (OpCode == Instruction::Mul) |
| 1208 | return SE->getMulExpr(LHS, RHS); |
| 1209 | |
| 1210 | llvm_unreachable("Unsupported opcode."); |
| Zinovy Nis | 0a36cba | 2014-08-21 08:25:45 +0000 | [diff] [blame] | 1211 | } |
| 1212 | |
| Andrew Trick | c7868bf0 | 2011-09-10 01:24:17 +0000 | [diff] [blame] | 1213 | /// No-wrap operations can transfer sign extension of their result to their |
| 1214 | /// operands. Generate the SCEV value for the widened operation without |
| 1215 | /// actually modifying the IR yet. If the expression after extending the |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1216 | /// operands is an AddRec for this loop, return the AddRec and the kind of |
| 1217 | /// extension used. |
| 1218 | WidenIV::WidenedRecTy WidenIV::getExtendedOperandRecurrence(NarrowIVDefUse DU) { |
| Andrew Trick | c7868bf0 | 2011-09-10 01:24:17 +0000 | [diff] [blame] | 1219 | // Handle the common case of add<nsw/nuw> |
| Zinovy Nis | 0a36cba | 2014-08-21 08:25:45 +0000 | [diff] [blame] | 1220 | const unsigned OpCode = DU.NarrowUse->getOpcode(); |
| 1221 | // Only Add/Sub/Mul instructions supported yet. |
| 1222 | if (OpCode != Instruction::Add && OpCode != Instruction::Sub && |
| 1223 | OpCode != Instruction::Mul) |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1224 | return {nullptr, Unknown}; |
| Andrew Trick | c7868bf0 | 2011-09-10 01:24:17 +0000 | [diff] [blame] | 1225 | |
| 1226 | // One operand (NarrowDef) has already been extended to WideDef. Now determine |
| 1227 | // if extending the other will lead to a recurrence. |
| Zinovy Nis | ccc3e37 | 2014-10-02 13:01:15 +0000 | [diff] [blame] | 1228 | const unsigned ExtendOperIdx = |
| 1229 | DU.NarrowUse->getOperand(0) == DU.NarrowDef ? 1 : 0; |
| Andrew Trick | c7868bf0 | 2011-09-10 01:24:17 +0000 | [diff] [blame] | 1230 | assert(DU.NarrowUse->getOperand(1-ExtendOperIdx) == DU.NarrowDef && "bad DU"); |
| 1231 | |
| Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 1232 | const SCEV *ExtendOperExpr = nullptr; |
| Andrew Trick | c7868bf0 | 2011-09-10 01:24:17 +0000 | [diff] [blame] | 1233 | const OverflowingBinaryOperator *OBO = |
| 1234 | cast<OverflowingBinaryOperator>(DU.NarrowUse); |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1235 | ExtendKind ExtKind = getExtendKind(DU.NarrowDef); |
| 1236 | if (ExtKind == SignExtended && OBO->hasNoSignedWrap()) |
| Andrew Trick | c7868bf0 | 2011-09-10 01:24:17 +0000 | [diff] [blame] | 1237 | ExtendOperExpr = SE->getSignExtendExpr( |
| 1238 | SE->getSCEV(DU.NarrowUse->getOperand(ExtendOperIdx)), WideType); |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1239 | else if(ExtKind == ZeroExtended && OBO->hasNoUnsignedWrap()) |
| Andrew Trick | c7868bf0 | 2011-09-10 01:24:17 +0000 | [diff] [blame] | 1240 | ExtendOperExpr = SE->getZeroExtendExpr( |
| 1241 | SE->getSCEV(DU.NarrowUse->getOperand(ExtendOperIdx)), WideType); |
| 1242 | else |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1243 | return {nullptr, Unknown}; |
| Andrew Trick | c7868bf0 | 2011-09-10 01:24:17 +0000 | [diff] [blame] | 1244 | |
| Zinovy Nis | 0a36cba | 2014-08-21 08:25:45 +0000 | [diff] [blame] | 1245 | // When creating this SCEV expr, don't apply the current operations NSW or NUW |
| Andrew Trick | d25089f | 2011-11-29 02:16:38 +0000 | [diff] [blame] | 1246 | // flags. This instruction may be guarded by control flow that the no-wrap |
| 1247 | // behavior depends on. Non-control-equivalent instructions can be mapped to |
| 1248 | // the same SCEV expression, and it would be incorrect to transfer NSW/NUW |
| 1249 | // semantics to those operations. |
| Zinovy Nis | ccc3e37 | 2014-10-02 13:01:15 +0000 | [diff] [blame] | 1250 | const SCEV *lhs = SE->getSCEV(DU.WideDef); |
| 1251 | const SCEV *rhs = ExtendOperExpr; |
| 1252 | |
| 1253 | // Let's swap operands to the initial order for the case of non-commutative |
| 1254 | // operations, like SUB. See PR21014. |
| 1255 | if (ExtendOperIdx == 0) |
| 1256 | std::swap(lhs, rhs); |
| 1257 | const SCEVAddRecExpr *AddRec = |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 1258 | dyn_cast<SCEVAddRecExpr>(getSCEVByOpCode(lhs, rhs, OpCode)); |
| Zinovy Nis | ccc3e37 | 2014-10-02 13:01:15 +0000 | [diff] [blame] | 1259 | |
| Andrew Trick | c7868bf0 | 2011-09-10 01:24:17 +0000 | [diff] [blame] | 1260 | if (!AddRec || AddRec->getLoop() != L) |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1261 | return {nullptr, Unknown}; |
| 1262 | |
| 1263 | return {AddRec, ExtKind}; |
| Andrew Trick | c7868bf0 | 2011-09-10 01:24:17 +0000 | [diff] [blame] | 1264 | } |
| 1265 | |
| Sanjoy Das | 9119bf4 | 2015-09-20 06:58:03 +0000 | [diff] [blame] | 1266 | /// Is this instruction potentially interesting for further simplification after |
| 1267 | /// widening it's type? In other words, can the extend be safely hoisted out of |
| 1268 | /// the loop with SCEV reducing the value to a recurrence on the same loop. If |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1269 | /// so, return the extended recurrence and the kind of extension used. Otherwise |
| 1270 | /// return {nullptr, Unknown}. |
| 1271 | WidenIV::WidenedRecTy WidenIV::getWideRecurrence(NarrowIVDefUse DU) { |
| 1272 | if (!SE->isSCEVable(DU.NarrowUse->getType())) |
| 1273 | return {nullptr, Unknown}; |
| Andrew Trick | 92905a1 | 2011-07-05 18:19:39 +0000 | [diff] [blame] | 1274 | |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1275 | const SCEV *NarrowExpr = SE->getSCEV(DU.NarrowUse); |
| Sanjoy Das | ff9eea2 | 2016-07-21 18:58:01 +0000 | [diff] [blame] | 1276 | if (SE->getTypeSizeInBits(NarrowExpr->getType()) >= |
| 1277 | SE->getTypeSizeInBits(WideType)) { |
| Andrew Trick | 92905a1 | 2011-07-05 18:19:39 +0000 | [diff] [blame] | 1278 | // NarrowUse implicitly widens its operand. e.g. a gep with a narrow |
| 1279 | // index. So don't follow this use. |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1280 | return {nullptr, Unknown}; |
| Andrew Trick | 92905a1 | 2011-07-05 18:19:39 +0000 | [diff] [blame] | 1281 | } |
| 1282 | |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1283 | const SCEV *WideExpr; |
| 1284 | ExtendKind ExtKind; |
| 1285 | if (DU.NeverNegative) { |
| 1286 | WideExpr = SE->getSignExtendExpr(NarrowExpr, WideType); |
| 1287 | if (isa<SCEVAddRecExpr>(WideExpr)) |
| 1288 | ExtKind = SignExtended; |
| 1289 | else { |
| 1290 | WideExpr = SE->getZeroExtendExpr(NarrowExpr, WideType); |
| 1291 | ExtKind = ZeroExtended; |
| 1292 | } |
| 1293 | } else if (getExtendKind(DU.NarrowDef) == SignExtended) { |
| 1294 | WideExpr = SE->getSignExtendExpr(NarrowExpr, WideType); |
| 1295 | ExtKind = SignExtended; |
| 1296 | } else { |
| 1297 | WideExpr = SE->getZeroExtendExpr(NarrowExpr, WideType); |
| 1298 | ExtKind = ZeroExtended; |
| 1299 | } |
| Andrew Trick | 92905a1 | 2011-07-05 18:19:39 +0000 | [diff] [blame] | 1300 | const SCEVAddRecExpr *AddRec = dyn_cast<SCEVAddRecExpr>(WideExpr); |
| 1301 | if (!AddRec || AddRec->getLoop() != L) |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1302 | return {nullptr, Unknown}; |
| 1303 | return {AddRec, ExtKind}; |
| Andrew Trick | 92905a1 | 2011-07-05 18:19:39 +0000 | [diff] [blame] | 1304 | } |
| 1305 | |
| Andrew Trick | 020dd89 | 2014-01-02 19:29:38 +0000 | [diff] [blame] | 1306 | /// This IV user cannot be widen. Replace this use of the original narrow IV |
| 1307 | /// with a truncation of the new wide IV to isolate and eliminate the narrow IV. |
| Sanjoy Das | 683bf07 | 2015-12-08 00:13:21 +0000 | [diff] [blame] | 1308 | static void truncateIVUse(NarrowIVDefUse DU, DominatorTree *DT, LoopInfo *LI) { |
| Max Kazantsev | 2a184af | 2019-02-12 09:59:44 +0000 | [diff] [blame] | 1309 | auto *InsertPt = getInsertPointForUses(DU.NarrowUse, DU.NarrowDef, DT, LI); |
| 1310 | if (!InsertPt) |
| 1311 | return; |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 1312 | LLVM_DEBUG(dbgs() << "INDVARS: Truncate IV " << *DU.WideDef << " for user " |
| 1313 | << *DU.NarrowUse << "\n"); |
| Max Kazantsev | 2a184af | 2019-02-12 09:59:44 +0000 | [diff] [blame] | 1314 | IRBuilder<> Builder(InsertPt); |
| Andrew Trick | 020dd89 | 2014-01-02 19:29:38 +0000 | [diff] [blame] | 1315 | Value *Trunc = Builder.CreateTrunc(DU.WideDef, DU.NarrowDef->getType()); |
| 1316 | DU.NarrowUse->replaceUsesOfWith(DU.NarrowDef, Trunc); |
| 1317 | } |
| 1318 | |
| Chad Rosier | bb99f40 | 2014-09-17 14:10:33 +0000 | [diff] [blame] | 1319 | /// If the narrow use is a compare instruction, then widen the compare |
| 1320 | // (and possibly the other operand). The extend operation is hoisted into the |
| 1321 | // loop preheader as far as possible. |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 1322 | bool WidenIV::widenLoopCompare(NarrowIVDefUse DU) { |
| Chad Rosier | bb99f40 | 2014-09-17 14:10:33 +0000 | [diff] [blame] | 1323 | ICmpInst *Cmp = dyn_cast<ICmpInst>(DU.NarrowUse); |
| 1324 | if (!Cmp) |
| 1325 | return false; |
| 1326 | |
| Sanjoy Das | f69d0e3 | 2015-09-18 21:21:02 +0000 | [diff] [blame] | 1327 | // We can legally widen the comparison in the following two cases: |
| 1328 | // |
| 1329 | // - The signedness of the IV extension and comparison match |
| 1330 | // |
| 1331 | // - The narrow IV is always positive (and thus its sign extension is equal |
| 1332 | // to its zero extension). For instance, let's say we're zero extending |
| 1333 | // %narrow for the following use |
| 1334 | // |
| 1335 | // icmp slt i32 %narrow, %val ... (A) |
| 1336 | // |
| 1337 | // and %narrow is always positive. Then |
| 1338 | // |
| 1339 | // (A) == icmp slt i32 sext(%narrow), sext(%val) |
| 1340 | // == icmp slt i32 zext(%narrow), sext(%val) |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1341 | bool IsSigned = getExtendKind(DU.NarrowDef) == SignExtended; |
| Sanjoy Das | 428db15 | 2015-09-20 01:52:18 +0000 | [diff] [blame] | 1342 | if (!(DU.NeverNegative || IsSigned == Cmp->isSigned())) |
| Chad Rosier | 307b50b | 2014-09-17 16:35:09 +0000 | [diff] [blame] | 1343 | return false; |
| 1344 | |
| Chad Rosier | bb99f40 | 2014-09-17 14:10:33 +0000 | [diff] [blame] | 1345 | Value *Op = Cmp->getOperand(Cmp->getOperand(0) == DU.NarrowDef ? 1 : 0); |
| 1346 | unsigned CastWidth = SE->getTypeSizeInBits(Op->getType()); |
| 1347 | unsigned IVWidth = SE->getTypeSizeInBits(WideType); |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 1348 | assert(CastWidth <= IVWidth && "Unexpected width while widening compare."); |
| Chad Rosier | bb99f40 | 2014-09-17 14:10:33 +0000 | [diff] [blame] | 1349 | |
| 1350 | // Widen the compare instruction. |
| Max Kazantsev | 2a184af | 2019-02-12 09:59:44 +0000 | [diff] [blame] | 1351 | auto *InsertPt = getInsertPointForUses(DU.NarrowUse, DU.NarrowDef, DT, LI); |
| 1352 | if (!InsertPt) |
| 1353 | return false; |
| 1354 | IRBuilder<> Builder(InsertPt); |
| Chad Rosier | bb99f40 | 2014-09-17 14:10:33 +0000 | [diff] [blame] | 1355 | DU.NarrowUse->replaceUsesOfWith(DU.NarrowDef, DU.WideDef); |
| 1356 | |
| 1357 | // Widen the other operand of the compare, if necessary. |
| 1358 | if (CastWidth < IVWidth) { |
| Sanjoy Das | 7360f30 | 2015-10-16 01:00:50 +0000 | [diff] [blame] | 1359 | Value *ExtOp = createExtendInst(Op, WideType, Cmp->isSigned(), Cmp); |
| Chad Rosier | bb99f40 | 2014-09-17 14:10:33 +0000 | [diff] [blame] | 1360 | DU.NarrowUse->replaceUsesOfWith(Op, ExtOp); |
| 1361 | } |
| 1362 | return true; |
| 1363 | } |
| 1364 | |
| Abderrazek Zaafrani | c30dfb2 | 2018-09-07 22:41:57 +0000 | [diff] [blame] | 1365 | /// If the narrow use is an instruction whose two operands are the defining |
| 1366 | /// instruction of DU and a load instruction, then we have the following: |
| 1367 | /// if the load is hoisted outside the loop, then we do not reach this function |
| 1368 | /// as scalar evolution analysis works fine in widenIVUse with variables |
| 1369 | /// hoisted outside the loop and efficient code is subsequently generated by |
| 1370 | /// not emitting truncate instructions. But when the load is not hoisted |
| 1371 | /// (whether due to limitation in alias analysis or due to a true legality), |
| 1372 | /// then scalar evolution can not proceed with loop variant values and |
| 1373 | /// inefficient code is generated. This function handles the non-hoisted load |
| 1374 | /// special case by making the optimization generate the same type of code for |
| 1375 | /// hoisted and non-hoisted load (widen use and eliminate sign extend |
| 1376 | /// instruction). This special case is important especially when the induction |
| 1377 | /// variables are affecting addressing mode in code generation. |
| 1378 | bool WidenIV::widenWithVariantLoadUse(NarrowIVDefUse DU) { |
| 1379 | Instruction *NarrowUse = DU.NarrowUse; |
| 1380 | Instruction *NarrowDef = DU.NarrowDef; |
| 1381 | Instruction *WideDef = DU.WideDef; |
| 1382 | |
| 1383 | // Handle the common case of add<nsw/nuw> |
| 1384 | const unsigned OpCode = NarrowUse->getOpcode(); |
| 1385 | // Only Add/Sub/Mul instructions are supported. |
| 1386 | if (OpCode != Instruction::Add && OpCode != Instruction::Sub && |
| 1387 | OpCode != Instruction::Mul) |
| 1388 | return false; |
| 1389 | |
| 1390 | // The operand that is not defined by NarrowDef of DU. Let's call it the |
| 1391 | // other operand. |
| 1392 | unsigned ExtendOperIdx = DU.NarrowUse->getOperand(0) == NarrowDef ? 1 : 0; |
| 1393 | assert(DU.NarrowUse->getOperand(1 - ExtendOperIdx) == DU.NarrowDef && |
| 1394 | "bad DU"); |
| 1395 | |
| 1396 | const SCEV *ExtendOperExpr = nullptr; |
| 1397 | const OverflowingBinaryOperator *OBO = |
| 1398 | cast<OverflowingBinaryOperator>(NarrowUse); |
| 1399 | ExtendKind ExtKind = getExtendKind(NarrowDef); |
| 1400 | if (ExtKind == SignExtended && OBO->hasNoSignedWrap()) |
| 1401 | ExtendOperExpr = SE->getSignExtendExpr( |
| 1402 | SE->getSCEV(NarrowUse->getOperand(ExtendOperIdx)), WideType); |
| 1403 | else if (ExtKind == ZeroExtended && OBO->hasNoUnsignedWrap()) |
| 1404 | ExtendOperExpr = SE->getZeroExtendExpr( |
| 1405 | SE->getSCEV(NarrowUse->getOperand(ExtendOperIdx)), WideType); |
| 1406 | else |
| 1407 | return false; |
| 1408 | |
| 1409 | // We are interested in the other operand being a load instruction. |
| 1410 | // But, we should look into relaxing this restriction later on. |
| 1411 | auto *I = dyn_cast<Instruction>(NarrowUse->getOperand(ExtendOperIdx)); |
| 1412 | if (I && I->getOpcode() != Instruction::Load) |
| 1413 | return false; |
| 1414 | |
| 1415 | // Verifying that Defining operand is an AddRec |
| 1416 | const SCEV *Op1 = SE->getSCEV(WideDef); |
| 1417 | const SCEVAddRecExpr *AddRecOp1 = dyn_cast<SCEVAddRecExpr>(Op1); |
| 1418 | if (!AddRecOp1 || AddRecOp1->getLoop() != L) |
| 1419 | return false; |
| 1420 | // Verifying that other operand is an Extend. |
| 1421 | if (ExtKind == SignExtended) { |
| 1422 | if (!isa<SCEVSignExtendExpr>(ExtendOperExpr)) |
| 1423 | return false; |
| 1424 | } else { |
| 1425 | if (!isa<SCEVZeroExtendExpr>(ExtendOperExpr)) |
| 1426 | return false; |
| 1427 | } |
| 1428 | |
| 1429 | if (ExtKind == SignExtended) { |
| 1430 | for (Use &U : NarrowUse->uses()) { |
| 1431 | SExtInst *User = dyn_cast<SExtInst>(U.getUser()); |
| 1432 | if (!User || User->getType() != WideType) |
| 1433 | return false; |
| 1434 | } |
| 1435 | } else { // ExtKind == ZeroExtended |
| 1436 | for (Use &U : NarrowUse->uses()) { |
| 1437 | ZExtInst *User = dyn_cast<ZExtInst>(U.getUser()); |
| 1438 | if (!User || User->getType() != WideType) |
| 1439 | return false; |
| 1440 | } |
| 1441 | } |
| 1442 | |
| 1443 | return true; |
| 1444 | } |
| 1445 | |
| 1446 | /// Special Case for widening with variant Loads (see |
| 1447 | /// WidenIV::widenWithVariantLoadUse). This is the code generation part. |
| 1448 | void WidenIV::widenWithVariantLoadUseCodegen(NarrowIVDefUse DU) { |
| 1449 | Instruction *NarrowUse = DU.NarrowUse; |
| 1450 | Instruction *NarrowDef = DU.NarrowDef; |
| 1451 | Instruction *WideDef = DU.WideDef; |
| 1452 | |
| 1453 | ExtendKind ExtKind = getExtendKind(NarrowDef); |
| 1454 | |
| 1455 | LLVM_DEBUG(dbgs() << "Cloning arithmetic IVUser: " << *NarrowUse << "\n"); |
| 1456 | |
| 1457 | // Generating a widening use instruction. |
| 1458 | Value *LHS = (NarrowUse->getOperand(0) == NarrowDef) |
| 1459 | ? WideDef |
| 1460 | : createExtendInst(NarrowUse->getOperand(0), WideType, |
| 1461 | ExtKind, NarrowUse); |
| 1462 | Value *RHS = (NarrowUse->getOperand(1) == NarrowDef) |
| 1463 | ? WideDef |
| 1464 | : createExtendInst(NarrowUse->getOperand(1), WideType, |
| 1465 | ExtKind, NarrowUse); |
| 1466 | |
| 1467 | auto *NarrowBO = cast<BinaryOperator>(NarrowUse); |
| 1468 | auto *WideBO = BinaryOperator::Create(NarrowBO->getOpcode(), LHS, RHS, |
| 1469 | NarrowBO->getName()); |
| 1470 | IRBuilder<> Builder(NarrowUse); |
| 1471 | Builder.Insert(WideBO); |
| 1472 | WideBO->copyIRFlags(NarrowBO); |
| 1473 | |
| 1474 | if (ExtKind == SignExtended) |
| 1475 | ExtendKindMap[NarrowUse] = SignExtended; |
| 1476 | else |
| 1477 | ExtendKindMap[NarrowUse] = ZeroExtended; |
| 1478 | |
| 1479 | // Update the Use. |
| 1480 | if (ExtKind == SignExtended) { |
| 1481 | for (Use &U : NarrowUse->uses()) { |
| 1482 | SExtInst *User = dyn_cast<SExtInst>(U.getUser()); |
| 1483 | if (User && User->getType() == WideType) { |
| 1484 | LLVM_DEBUG(dbgs() << "INDVARS: eliminating " << *User << " replaced by " |
| 1485 | << *WideBO << "\n"); |
| 1486 | ++NumElimExt; |
| 1487 | User->replaceAllUsesWith(WideBO); |
| 1488 | DeadInsts.emplace_back(User); |
| 1489 | } |
| 1490 | } |
| 1491 | } else { // ExtKind == ZeroExtended |
| 1492 | for (Use &U : NarrowUse->uses()) { |
| 1493 | ZExtInst *User = dyn_cast<ZExtInst>(U.getUser()); |
| 1494 | if (User && User->getType() == WideType) { |
| 1495 | LLVM_DEBUG(dbgs() << "INDVARS: eliminating " << *User << " replaced by " |
| 1496 | << *WideBO << "\n"); |
| 1497 | ++NumElimExt; |
| 1498 | User->replaceAllUsesWith(WideBO); |
| 1499 | DeadInsts.emplace_back(User); |
| 1500 | } |
| 1501 | } |
| 1502 | } |
| 1503 | } |
| 1504 | |
| Sanjoy Das | 9119bf4 | 2015-09-20 06:58:03 +0000 | [diff] [blame] | 1505 | /// Determine whether an individual user of the narrow IV can be widened. If so, |
| 1506 | /// return the wide clone of the user. |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 1507 | Instruction *WidenIV::widenIVUse(NarrowIVDefUse DU, SCEVExpander &Rewriter) { |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1508 | assert(ExtendKindMap.count(DU.NarrowDef) && |
| 1509 | "Should already know the kind of extension used to widen NarrowDef"); |
| Andrew Trick | ecdd6e4 | 2011-06-29 23:03:57 +0000 | [diff] [blame] | 1510 | |
| Andrew Trick | 6d12309 | 2011-07-02 02:34:25 +0000 | [diff] [blame] | 1511 | // Stop traversing the def-use chain at inner-loop phis or post-loop phis. |
| Andrew Trick | e4a1860 | 2014-01-07 06:59:12 +0000 | [diff] [blame] | 1512 | if (PHINode *UsePhi = dyn_cast<PHINode>(DU.NarrowUse)) { |
| 1513 | if (LI->getLoopFor(UsePhi->getParent()) != L) { |
| 1514 | // For LCSSA phis, sink the truncate outside the loop. |
| 1515 | // After SimplifyCFG most loop exit targets have a single predecessor. |
| 1516 | // Otherwise fall back to a truncate within the loop. |
| 1517 | if (UsePhi->getNumOperands() != 1) |
| Sanjoy Das | 683bf07 | 2015-12-08 00:13:21 +0000 | [diff] [blame] | 1518 | truncateIVUse(DU, DT, LI); |
| Andrew Trick | e4a1860 | 2014-01-07 06:59:12 +0000 | [diff] [blame] | 1519 | else { |
| David Majnemer | 5d51838 | 2016-03-30 21:12:06 +0000 | [diff] [blame] | 1520 | // Widening the PHI requires us to insert a trunc. The logical place |
| 1521 | // for this trunc is in the same BB as the PHI. This is not possible if |
| 1522 | // the BB is terminated by a catchswitch. |
| 1523 | if (isa<CatchSwitchInst>(UsePhi->getParent()->getTerminator())) |
| 1524 | return nullptr; |
| 1525 | |
| Andrew Trick | e4a1860 | 2014-01-07 06:59:12 +0000 | [diff] [blame] | 1526 | PHINode *WidePhi = |
| 1527 | PHINode::Create(DU.WideDef->getType(), 1, UsePhi->getName() + ".wide", |
| 1528 | UsePhi); |
| 1529 | WidePhi->addIncoming(DU.WideDef, UsePhi->getIncomingBlock(0)); |
| Duncan P. N. Exon Smith | 3a9c9e3 | 2015-10-13 18:26:00 +0000 | [diff] [blame] | 1530 | IRBuilder<> Builder(&*WidePhi->getParent()->getFirstInsertionPt()); |
| Andrew Trick | e4a1860 | 2014-01-07 06:59:12 +0000 | [diff] [blame] | 1531 | Value *Trunc = Builder.CreateTrunc(WidePhi, DU.NarrowDef->getType()); |
| 1532 | UsePhi->replaceAllUsesWith(Trunc); |
| Benjamin Kramer | f5e2fc4 | 2015-05-29 19:43:39 +0000 | [diff] [blame] | 1533 | DeadInsts.emplace_back(UsePhi); |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 1534 | LLVM_DEBUG(dbgs() << "INDVARS: Widen lcssa phi " << *UsePhi << " to " |
| 1535 | << *WidePhi << "\n"); |
| Andrew Trick | e4a1860 | 2014-01-07 06:59:12 +0000 | [diff] [blame] | 1536 | } |
| Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 1537 | return nullptr; |
| Andrew Trick | e4a1860 | 2014-01-07 06:59:12 +0000 | [diff] [blame] | 1538 | } |
| Andrew Trick | 020dd89 | 2014-01-02 19:29:38 +0000 | [diff] [blame] | 1539 | } |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1540 | |
| 1541 | // This narrow use can be widened by a sext if it's non-negative or its narrow |
| 1542 | // def was widended by a sext. Same for zext. |
| 1543 | auto canWidenBySExt = [&]() { |
| 1544 | return DU.NeverNegative || getExtendKind(DU.NarrowDef) == SignExtended; |
| 1545 | }; |
| 1546 | auto canWidenByZExt = [&]() { |
| 1547 | return DU.NeverNegative || getExtendKind(DU.NarrowDef) == ZeroExtended; |
| 1548 | }; |
| 1549 | |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1550 | // Our raison d'etre! Eliminate sign and zero extension. |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1551 | if ((isa<SExtInst>(DU.NarrowUse) && canWidenBySExt()) || |
| 1552 | (isa<ZExtInst>(DU.NarrowUse) && canWidenByZExt())) { |
| Andrew Trick | 2210448 | 2011-07-20 04:39:24 +0000 | [diff] [blame] | 1553 | Value *NewDef = DU.WideDef; |
| 1554 | if (DU.NarrowUse->getType() != WideType) { |
| 1555 | unsigned CastWidth = SE->getTypeSizeInBits(DU.NarrowUse->getType()); |
| Andrew Trick | eb3c36e | 2011-05-25 04:42:22 +0000 | [diff] [blame] | 1556 | unsigned IVWidth = SE->getTypeSizeInBits(WideType); |
| 1557 | if (CastWidth < IVWidth) { |
| 1558 | // The cast isn't as wide as the IV, so insert a Trunc. |
| Andrew Trick | 2210448 | 2011-07-20 04:39:24 +0000 | [diff] [blame] | 1559 | IRBuilder<> Builder(DU.NarrowUse); |
| 1560 | NewDef = Builder.CreateTrunc(DU.WideDef, DU.NarrowUse->getType()); |
| Andrew Trick | eb3c36e | 2011-05-25 04:42:22 +0000 | [diff] [blame] | 1561 | } |
| 1562 | else { |
| 1563 | // A wider extend was hidden behind a narrower one. This may induce |
| 1564 | // another round of IV widening in which the intermediate IV becomes |
| 1565 | // dead. It should be very rare. |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 1566 | LLVM_DEBUG(dbgs() << "INDVARS: New IV " << *WidePhi |
| 1567 | << " not wide enough to subsume " << *DU.NarrowUse |
| 1568 | << "\n"); |
| Andrew Trick | 2210448 | 2011-07-20 04:39:24 +0000 | [diff] [blame] | 1569 | DU.NarrowUse->replaceUsesOfWith(DU.NarrowDef, DU.WideDef); |
| 1570 | NewDef = DU.NarrowUse; |
| Andrew Trick | eb3c36e | 2011-05-25 04:42:22 +0000 | [diff] [blame] | 1571 | } |
| 1572 | } |
| Andrew Trick | 2210448 | 2011-07-20 04:39:24 +0000 | [diff] [blame] | 1573 | if (NewDef != DU.NarrowUse) { |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 1574 | LLVM_DEBUG(dbgs() << "INDVARS: eliminating " << *DU.NarrowUse |
| 1575 | << " replaced by " << *DU.WideDef << "\n"); |
| Andrew Trick | eb3c36e | 2011-05-25 04:42:22 +0000 | [diff] [blame] | 1576 | ++NumElimExt; |
| Andrew Trick | 2210448 | 2011-07-20 04:39:24 +0000 | [diff] [blame] | 1577 | DU.NarrowUse->replaceAllUsesWith(NewDef); |
| Benjamin Kramer | f5e2fc4 | 2015-05-29 19:43:39 +0000 | [diff] [blame] | 1578 | DeadInsts.emplace_back(DU.NarrowUse); |
| Andrew Trick | eb3c36e | 2011-05-25 04:42:22 +0000 | [diff] [blame] | 1579 | } |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 1580 | // Now that the extend is gone, we want to expose it's uses for potential |
| 1581 | // further simplification. We don't need to directly inform SimplifyIVUsers |
| 1582 | // of the new users, because their parent IV will be processed later as a |
| 1583 | // new loop phi. If we preserved IVUsers analysis, we would also want to |
| 1584 | // push the uses of WideDef here. |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1585 | |
| 1586 | // No further widening is needed. The deceased [sz]ext had done it for us. |
| Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 1587 | return nullptr; |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1588 | } |
| Andrew Trick | 6d12309 | 2011-07-02 02:34:25 +0000 | [diff] [blame] | 1589 | |
| 1590 | // Does this user itself evaluate to a recurrence after widening? |
| Wei Mi | d2948ce | 2016-11-15 17:34:52 +0000 | [diff] [blame] | 1591 | WidenedRecTy WideAddRec = getExtendedOperandRecurrence(DU); |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1592 | if (!WideAddRec.first) |
| Wei Mi | d2948ce | 2016-11-15 17:34:52 +0000 | [diff] [blame] | 1593 | WideAddRec = getWideRecurrence(DU); |
| Chad Rosier | bb99f40 | 2014-09-17 14:10:33 +0000 | [diff] [blame] | 1594 | |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1595 | assert((WideAddRec.first == nullptr) == (WideAddRec.second == Unknown)); |
| 1596 | if (!WideAddRec.first) { |
| Chad Rosier | bb99f40 | 2014-09-17 14:10:33 +0000 | [diff] [blame] | 1597 | // If use is a loop condition, try to promote the condition instead of |
| 1598 | // truncating the IV first. |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 1599 | if (widenLoopCompare(DU)) |
| Chad Rosier | bb99f40 | 2014-09-17 14:10:33 +0000 | [diff] [blame] | 1600 | return nullptr; |
| 1601 | |
| Abderrazek Zaafrani | c30dfb2 | 2018-09-07 22:41:57 +0000 | [diff] [blame] | 1602 | // We are here about to generate a truncate instruction that may hurt |
| 1603 | // performance because the scalar evolution expression computed earlier |
| 1604 | // in WideAddRec.first does not indicate a polynomial induction expression. |
| 1605 | // In that case, look at the operands of the use instruction to determine |
| 1606 | // if we can still widen the use instead of truncating its operand. |
| 1607 | if (widenWithVariantLoadUse(DU)) { |
| 1608 | widenWithVariantLoadUseCodegen(DU); |
| 1609 | return nullptr; |
| 1610 | } |
| 1611 | |
| Xin Tong | ee5cb65 | 2017-01-07 04:30:58 +0000 | [diff] [blame] | 1612 | // This user does not evaluate to a recurrence after widening, so don't |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1613 | // follow it. Instead insert a Trunc to kill off the original use, |
| 1614 | // eventually isolating the original narrow IV so it can be removed. |
| Sanjoy Das | 683bf07 | 2015-12-08 00:13:21 +0000 | [diff] [blame] | 1615 | truncateIVUse(DU, DT, LI); |
| Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 1616 | return nullptr; |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1617 | } |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 1618 | // Assume block terminators cannot evaluate to a recurrence. We can't to |
| Andrew Trick | 6d12309 | 2011-07-02 02:34:25 +0000 | [diff] [blame] | 1619 | // insert a Trunc after a terminator if there happens to be a critical edge. |
| Andrew Trick | 2210448 | 2011-07-20 04:39:24 +0000 | [diff] [blame] | 1620 | assert(DU.NarrowUse != DU.NarrowUse->getParent()->getTerminator() && |
| Andrew Trick | 6d12309 | 2011-07-02 02:34:25 +0000 | [diff] [blame] | 1621 | "SCEV is not expected to evaluate a block terminator"); |
| Andrew Trick | ecdd6e4 | 2011-06-29 23:03:57 +0000 | [diff] [blame] | 1622 | |
| Andrew Trick | 7fac79e | 2011-05-26 00:46:11 +0000 | [diff] [blame] | 1623 | // Reuse the IV increment that SCEVExpander created as long as it dominates |
| 1624 | // NarrowUse. |
| Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 1625 | Instruction *WideUse = nullptr; |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1626 | if (WideAddRec.first == WideIncExpr && |
| 1627 | Rewriter.hoistIVInc(WideInc, DU.NarrowUse)) |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1628 | WideUse = WideInc; |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1629 | else { |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1630 | WideUse = cloneIVUser(DU, WideAddRec.first); |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1631 | if (!WideUse) |
| Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 1632 | return nullptr; |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1633 | } |
| Andrew Trick | 6d12309 | 2011-07-02 02:34:25 +0000 | [diff] [blame] | 1634 | // Evaluation of WideAddRec ensured that the narrow expression could be |
| 1635 | // extended outside the loop without overflow. This suggests that the wide use |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1636 | // evaluates to the same expression as the extended narrow use, but doesn't |
| 1637 | // absolutely guarantee it. Hence the following failsafe check. In rare cases |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 1638 | // where it fails, we simply throw away the newly created wide use. |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1639 | if (WideAddRec.first != SE->getSCEV(WideUse)) { |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 1640 | LLVM_DEBUG(dbgs() << "Wide use expression mismatch: " << *WideUse << ": " |
| 1641 | << *SE->getSCEV(WideUse) << " != " << *WideAddRec.first |
| 1642 | << "\n"); |
| Benjamin Kramer | f5e2fc4 | 2015-05-29 19:43:39 +0000 | [diff] [blame] | 1643 | DeadInsts.emplace_back(WideUse); |
| Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 1644 | return nullptr; |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1645 | } |
| 1646 | |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1647 | ExtendKindMap[DU.NarrowUse] = WideAddRec.second; |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1648 | // Returning WideUse pushes it on the worklist. |
| 1649 | return WideUse; |
| 1650 | } |
| 1651 | |
| Sanjoy Das | 9119bf4 | 2015-09-20 06:58:03 +0000 | [diff] [blame] | 1652 | /// Add eligible users of NarrowDef to NarrowIVUsers. |
| Andrew Trick | 6d12309 | 2011-07-02 02:34:25 +0000 | [diff] [blame] | 1653 | void WidenIV::pushNarrowIVUsers(Instruction *NarrowDef, Instruction *WideDef) { |
| Sanjoy Das | 428db15 | 2015-09-20 01:52:18 +0000 | [diff] [blame] | 1654 | const SCEV *NarrowSCEV = SE->getSCEV(NarrowDef); |
| Artur Pilipenko | f2d5dc5 | 2016-10-19 18:59:03 +0000 | [diff] [blame] | 1655 | bool NonNegativeDef = |
| Sanjoy Das | 428db15 | 2015-09-20 01:52:18 +0000 | [diff] [blame] | 1656 | SE->isKnownPredicate(ICmpInst::ICMP_SGE, NarrowSCEV, |
| Artur Pilipenko | b78ad9d | 2016-08-22 13:12:07 +0000 | [diff] [blame] | 1657 | SE->getConstant(NarrowSCEV->getType(), 0)); |
| Chandler Carruth | cdf4788 | 2014-03-09 03:16:01 +0000 | [diff] [blame] | 1658 | for (User *U : NarrowDef->users()) { |
| 1659 | Instruction *NarrowUser = cast<Instruction>(U); |
| Andrew Trick | 6d12309 | 2011-07-02 02:34:25 +0000 | [diff] [blame] | 1660 | |
| 1661 | // Handle data flow merges and bizarre phi cycles. |
| David Blaikie | 70573dc | 2014-11-19 07:49:26 +0000 | [diff] [blame] | 1662 | if (!Widened.insert(NarrowUser).second) |
| Andrew Trick | 6d12309 | 2011-07-02 02:34:25 +0000 | [diff] [blame] | 1663 | continue; |
| 1664 | |
| Artur Pilipenko | f2d5dc5 | 2016-10-19 18:59:03 +0000 | [diff] [blame] | 1665 | bool NonNegativeUse = false; |
| 1666 | if (!NonNegativeDef) { |
| 1667 | // We might have a control-dependent range information for this context. |
| 1668 | if (auto RangeInfo = getPostIncRangeInfo(NarrowDef, NarrowUser)) |
| 1669 | NonNegativeUse = RangeInfo->getSignedMin().isNonNegative(); |
| 1670 | } |
| 1671 | |
| 1672 | NarrowIVUsers.emplace_back(NarrowDef, NarrowUser, WideDef, |
| 1673 | NonNegativeDef || NonNegativeUse); |
| Andrew Trick | 6d12309 | 2011-07-02 02:34:25 +0000 | [diff] [blame] | 1674 | } |
| 1675 | } |
| 1676 | |
| Sanjoy Das | 9119bf4 | 2015-09-20 06:58:03 +0000 | [diff] [blame] | 1677 | /// Process a single induction variable. First use the SCEVExpander to create a |
| 1678 | /// wide induction variable that evaluates to the same recurrence as the |
| 1679 | /// original narrow IV. Then use a worklist to forward traverse the narrow IV's |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 1680 | /// def-use chain. After widenIVUse has processed all interesting IV users, the |
| Sanjoy Das | 9119bf4 | 2015-09-20 06:58:03 +0000 | [diff] [blame] | 1681 | /// narrow IV will be isolated for removal by DeleteDeadPHIs. |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1682 | /// |
| 1683 | /// It would be simpler to delete uses as they are processed, but we must avoid |
| 1684 | /// invalidating SCEV expressions. |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 1685 | PHINode *WidenIV::createWideIV(SCEVExpander &Rewriter) { |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1686 | // Is this phi an induction variable? |
| 1687 | const SCEVAddRecExpr *AddRec = dyn_cast<SCEVAddRecExpr>(SE->getSCEV(OrigPhi)); |
| 1688 | if (!AddRec) |
| Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 1689 | return nullptr; |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1690 | |
| 1691 | // Widen the induction variable expression. |
| Evgeny Stupachenko | dc8a254 | 2016-09-28 23:39:39 +0000 | [diff] [blame] | 1692 | const SCEV *WideIVExpr = getExtendKind(OrigPhi) == SignExtended |
| 1693 | ? SE->getSignExtendExpr(AddRec, WideType) |
| 1694 | : SE->getZeroExtendExpr(AddRec, WideType); |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1695 | |
| 1696 | assert(SE->getEffectiveSCEVType(WideIVExpr->getType()) == WideType && |
| 1697 | "Expect the new IV expression to preserve its type"); |
| 1698 | |
| 1699 | // Can the IV be extended outside the loop without overflow? |
| 1700 | AddRec = dyn_cast<SCEVAddRecExpr>(WideIVExpr); |
| 1701 | if (!AddRec || AddRec->getLoop() != L) |
| Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 1702 | return nullptr; |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1703 | |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 1704 | // An AddRec must have loop-invariant operands. Since this AddRec is |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1705 | // materialized by a loop header phi, the expression cannot have any post-loop |
| 1706 | // operands, so they must dominate the loop header. |
| Sanjoy Das | 91e6ba6 | 2016-06-24 21:23:32 +0000 | [diff] [blame] | 1707 | assert( |
| 1708 | SE->properlyDominates(AddRec->getStart(), L->getHeader()) && |
| 1709 | SE->properlyDominates(AddRec->getStepRecurrence(*SE), L->getHeader()) && |
| 1710 | "Loop header phi recurrence inputs do not dominate the loop"); |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1711 | |
| Artur Pilipenko | f2d5dc5 | 2016-10-19 18:59:03 +0000 | [diff] [blame] | 1712 | // Iterate over IV uses (including transitive ones) looking for IV increments |
| 1713 | // of the form 'add nsw %iv, <const>'. For each increment and each use of |
| 1714 | // the increment calculate control-dependent range information basing on |
| 1715 | // dominating conditions inside of the loop (e.g. a range check inside of the |
| 1716 | // loop). Calculated ranges are stored in PostIncRangeInfos map. |
| 1717 | // |
| 1718 | // Control-dependent range information is later used to prove that a narrow |
| 1719 | // definition is not negative (see pushNarrowIVUsers). It's difficult to do |
| 1720 | // this on demand because when pushNarrowIVUsers needs this information some |
| 1721 | // of the dominating conditions might be already widened. |
| 1722 | if (UsePostIncrementRanges) |
| 1723 | calculatePostIncRanges(OrigPhi); |
| 1724 | |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1725 | // The rewriter provides a value for the desired IV expression. This may |
| 1726 | // either find an existing phi or materialize a new one. Either way, we |
| 1727 | // expect a well-formed cyclic phi-with-increments. i.e. any operand not part |
| 1728 | // of the phi-SCC dominates the loop entry. |
| Duncan P. N. Exon Smith | 3a9c9e3 | 2015-10-13 18:26:00 +0000 | [diff] [blame] | 1729 | Instruction *InsertPt = &L->getHeader()->front(); |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1730 | WidePhi = cast<PHINode>(Rewriter.expandCodeFor(AddRec, WideType, InsertPt)); |
| 1731 | |
| 1732 | // Remembering the WideIV increment generated by SCEVExpander allows |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 1733 | // widenIVUse to reuse it when widening the narrow IV's increment. We don't |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1734 | // employ a general reuse mechanism because the call above is the only call to |
| 1735 | // SCEVExpander. Henceforth, we produce 1-to-1 narrow to wide uses. |
| Andrew Trick | 7fac79e | 2011-05-26 00:46:11 +0000 | [diff] [blame] | 1736 | if (BasicBlock *LatchBlock = L->getLoopLatch()) { |
| 1737 | WideInc = |
| 1738 | cast<Instruction>(WidePhi->getIncomingValueForBlock(LatchBlock)); |
| 1739 | WideIncExpr = SE->getSCEV(WideInc); |
| Andrea Di Biagio | 824cabd | 2016-10-25 16:45:17 +0000 | [diff] [blame] | 1740 | // Propagate the debug location associated with the original loop increment |
| 1741 | // to the new (widened) increment. |
| 1742 | auto *OrigInc = |
| 1743 | cast<Instruction>(OrigPhi->getIncomingValueForBlock(LatchBlock)); |
| 1744 | WideInc->setDebugLoc(OrigInc->getDebugLoc()); |
| Andrew Trick | 7fac79e | 2011-05-26 00:46:11 +0000 | [diff] [blame] | 1745 | } |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1746 | |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 1747 | LLVM_DEBUG(dbgs() << "Wide IV: " << *WidePhi << "\n"); |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1748 | ++NumWidened; |
| 1749 | |
| 1750 | // Traverse the def-use chain using a worklist starting at the original IV. |
| Andrew Trick | 6d12309 | 2011-07-02 02:34:25 +0000 | [diff] [blame] | 1751 | assert(Widened.empty() && NarrowIVUsers.empty() && "expect initial state" ); |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1752 | |
| Andrew Trick | 6d12309 | 2011-07-02 02:34:25 +0000 | [diff] [blame] | 1753 | Widened.insert(OrigPhi); |
| 1754 | pushNarrowIVUsers(OrigPhi, WidePhi); |
| 1755 | |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1756 | while (!NarrowIVUsers.empty()) { |
| Andrew Trick | 2210448 | 2011-07-20 04:39:24 +0000 | [diff] [blame] | 1757 | NarrowIVDefUse DU = NarrowIVUsers.pop_back_val(); |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1758 | |
| Andrew Trick | 7fac79e | 2011-05-26 00:46:11 +0000 | [diff] [blame] | 1759 | // Process a def-use edge. This may replace the use, so don't hold a |
| 1760 | // use_iterator across it. |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 1761 | Instruction *WideUse = widenIVUse(DU, Rewriter); |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1762 | |
| Andrew Trick | 7fac79e | 2011-05-26 00:46:11 +0000 | [diff] [blame] | 1763 | // Follow all def-use edges from the previous narrow use. |
| Andrew Trick | 6d12309 | 2011-07-02 02:34:25 +0000 | [diff] [blame] | 1764 | if (WideUse) |
| Andrew Trick | 2210448 | 2011-07-20 04:39:24 +0000 | [diff] [blame] | 1765 | pushNarrowIVUsers(DU.NarrowUse, WideUse); |
| Andrew Trick | 6d12309 | 2011-07-02 02:34:25 +0000 | [diff] [blame] | 1766 | |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 1767 | // widenIVUse may have removed the def-use edge. |
| Andrew Trick | 2210448 | 2011-07-20 04:39:24 +0000 | [diff] [blame] | 1768 | if (DU.NarrowDef->use_empty()) |
| Benjamin Kramer | f5e2fc4 | 2015-05-29 19:43:39 +0000 | [diff] [blame] | 1769 | DeadInsts.emplace_back(DU.NarrowDef); |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1770 | } |
| Adrian Prantl | fbb6fbf | 2017-11-02 23:17:06 +0000 | [diff] [blame] | 1771 | |
| 1772 | // Attach any debug information to the new PHI. Since OrigPhi and WidePHI |
| 1773 | // evaluate the same recurrence, we can just copy the debug info over. |
| 1774 | SmallVector<DbgValueInst *, 1> DbgValues; |
| 1775 | llvm::findDbgValues(DbgValues, OrigPhi); |
| 1776 | auto *MDPhi = MetadataAsValue::get(WidePhi->getContext(), |
| 1777 | ValueAsMetadata::get(WidePhi)); |
| 1778 | for (auto &DbgValue : DbgValues) |
| 1779 | DbgValue->setOperand(0, MDPhi); |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 1780 | return WidePhi; |
| Andrew Trick | f44aadf | 2011-05-20 18:25:42 +0000 | [diff] [blame] | 1781 | } |
| 1782 | |
| Artur Pilipenko | f2d5dc5 | 2016-10-19 18:59:03 +0000 | [diff] [blame] | 1783 | /// Calculates control-dependent range for the given def at the given context |
| 1784 | /// by looking at dominating conditions inside of the loop |
| 1785 | void WidenIV::calculatePostIncRange(Instruction *NarrowDef, |
| 1786 | Instruction *NarrowUser) { |
| 1787 | using namespace llvm::PatternMatch; |
| 1788 | |
| 1789 | Value *NarrowDefLHS; |
| 1790 | const APInt *NarrowDefRHS; |
| 1791 | if (!match(NarrowDef, m_NSWAdd(m_Value(NarrowDefLHS), |
| 1792 | m_APInt(NarrowDefRHS))) || |
| 1793 | !NarrowDefRHS->isNonNegative()) |
| 1794 | return; |
| 1795 | |
| 1796 | auto UpdateRangeFromCondition = [&] (Value *Condition, |
| 1797 | bool TrueDest) { |
| 1798 | CmpInst::Predicate Pred; |
| 1799 | Value *CmpRHS; |
| 1800 | if (!match(Condition, m_ICmp(Pred, m_Specific(NarrowDefLHS), |
| 1801 | m_Value(CmpRHS)))) |
| 1802 | return; |
| 1803 | |
| 1804 | CmpInst::Predicate P = |
| Simon Pilgrim | 610ad9b | 2017-03-20 13:55:35 +0000 | [diff] [blame] | 1805 | TrueDest ? Pred : CmpInst::getInversePredicate(Pred); |
| Artur Pilipenko | f2d5dc5 | 2016-10-19 18:59:03 +0000 | [diff] [blame] | 1806 | |
| 1807 | auto CmpRHSRange = SE->getSignedRange(SE->getSCEV(CmpRHS)); |
| 1808 | auto CmpConstrainedLHSRange = |
| 1809 | ConstantRange::makeAllowedICmpRegion(P, CmpRHSRange); |
| 1810 | auto NarrowDefRange = |
| 1811 | CmpConstrainedLHSRange.addWithNoSignedWrap(*NarrowDefRHS); |
| 1812 | |
| 1813 | updatePostIncRangeInfo(NarrowDef, NarrowUser, NarrowDefRange); |
| 1814 | }; |
| 1815 | |
| Artur Pilipenko | 5c6ef75 | 2016-10-19 19:43:54 +0000 | [diff] [blame] | 1816 | auto UpdateRangeFromGuards = [&](Instruction *Ctx) { |
| 1817 | if (!HasGuards) |
| 1818 | return; |
| 1819 | |
| 1820 | for (Instruction &I : make_range(Ctx->getIterator().getReverse(), |
| 1821 | Ctx->getParent()->rend())) { |
| 1822 | Value *C = nullptr; |
| 1823 | if (match(&I, m_Intrinsic<Intrinsic::experimental_guard>(m_Value(C)))) |
| 1824 | UpdateRangeFromCondition(C, /*TrueDest=*/true); |
| 1825 | } |
| 1826 | }; |
| 1827 | |
| 1828 | UpdateRangeFromGuards(NarrowUser); |
| 1829 | |
| Artur Pilipenko | f2d5dc5 | 2016-10-19 18:59:03 +0000 | [diff] [blame] | 1830 | BasicBlock *NarrowUserBB = NarrowUser->getParent(); |
| Simon Pilgrim | 610ad9b | 2017-03-20 13:55:35 +0000 | [diff] [blame] | 1831 | // If NarrowUserBB is statically unreachable asking dominator queries may |
| Simon Pilgrim | 7d18a70 | 2016-11-20 13:19:49 +0000 | [diff] [blame] | 1832 | // yield surprising results. (e.g. the block may not have a dom tree node) |
| Artur Pilipenko | f2d5dc5 | 2016-10-19 18:59:03 +0000 | [diff] [blame] | 1833 | if (!DT->isReachableFromEntry(NarrowUserBB)) |
| 1834 | return; |
| 1835 | |
| 1836 | for (auto *DTB = (*DT)[NarrowUserBB]->getIDom(); |
| 1837 | L->contains(DTB->getBlock()); |
| 1838 | DTB = DTB->getIDom()) { |
| 1839 | auto *BB = DTB->getBlock(); |
| 1840 | auto *TI = BB->getTerminator(); |
| Artur Pilipenko | 5c6ef75 | 2016-10-19 19:43:54 +0000 | [diff] [blame] | 1841 | UpdateRangeFromGuards(TI); |
| Artur Pilipenko | f2d5dc5 | 2016-10-19 18:59:03 +0000 | [diff] [blame] | 1842 | |
| 1843 | auto *BI = dyn_cast<BranchInst>(TI); |
| 1844 | if (!BI || !BI->isConditional()) |
| 1845 | continue; |
| 1846 | |
| 1847 | auto *TrueSuccessor = BI->getSuccessor(0); |
| 1848 | auto *FalseSuccessor = BI->getSuccessor(1); |
| 1849 | |
| 1850 | auto DominatesNarrowUser = [this, NarrowUser] (BasicBlockEdge BBE) { |
| 1851 | return BBE.isSingleEdge() && |
| 1852 | DT->dominates(BBE, NarrowUser->getParent()); |
| 1853 | }; |
| 1854 | |
| 1855 | if (DominatesNarrowUser(BasicBlockEdge(BB, TrueSuccessor))) |
| 1856 | UpdateRangeFromCondition(BI->getCondition(), /*TrueDest=*/true); |
| 1857 | |
| 1858 | if (DominatesNarrowUser(BasicBlockEdge(BB, FalseSuccessor))) |
| 1859 | UpdateRangeFromCondition(BI->getCondition(), /*TrueDest=*/false); |
| 1860 | } |
| 1861 | } |
| 1862 | |
| 1863 | /// Calculates PostIncRangeInfos map for the given IV |
| 1864 | void WidenIV::calculatePostIncRanges(PHINode *OrigPhi) { |
| 1865 | SmallPtrSet<Instruction *, 16> Visited; |
| 1866 | SmallVector<Instruction *, 6> Worklist; |
| 1867 | Worklist.push_back(OrigPhi); |
| 1868 | Visited.insert(OrigPhi); |
| 1869 | |
| 1870 | while (!Worklist.empty()) { |
| 1871 | Instruction *NarrowDef = Worklist.pop_back_val(); |
| 1872 | |
| 1873 | for (Use &U : NarrowDef->uses()) { |
| 1874 | auto *NarrowUser = cast<Instruction>(U.getUser()); |
| 1875 | |
| 1876 | // Don't go looking outside the current loop. |
| 1877 | auto *NarrowUserLoop = (*LI)[NarrowUser->getParent()]; |
| 1878 | if (!NarrowUserLoop || !L->contains(NarrowUserLoop)) |
| 1879 | continue; |
| 1880 | |
| 1881 | if (!Visited.insert(NarrowUser).second) |
| 1882 | continue; |
| 1883 | |
| 1884 | Worklist.push_back(NarrowUser); |
| 1885 | |
| 1886 | calculatePostIncRange(NarrowDef, NarrowUser); |
| 1887 | } |
| 1888 | } |
| 1889 | } |
| 1890 | |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 1891 | //===----------------------------------------------------------------------===// |
| Andrew Trick | b6bc783 | 2014-01-02 21:12:11 +0000 | [diff] [blame] | 1892 | // Live IV Reduction - Minimize IVs live across the loop. |
| 1893 | //===----------------------------------------------------------------------===// |
| 1894 | |
| Andrew Trick | b6bc783 | 2014-01-02 21:12:11 +0000 | [diff] [blame] | 1895 | //===----------------------------------------------------------------------===// |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 1896 | // Simplification of IV users based on SCEV evaluation. |
| 1897 | //===----------------------------------------------------------------------===// |
| 1898 | |
| Andrew Trick | b6bc783 | 2014-01-02 21:12:11 +0000 | [diff] [blame] | 1899 | namespace { |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 1900 | |
| Sanjoy Das | e1e352d | 2015-09-20 18:42:50 +0000 | [diff] [blame] | 1901 | class IndVarSimplifyVisitor : public IVVisitor { |
| 1902 | ScalarEvolution *SE; |
| 1903 | const TargetTransformInfo *TTI; |
| 1904 | PHINode *IVPhi; |
| Andrew Trick | b6bc783 | 2014-01-02 21:12:11 +0000 | [diff] [blame] | 1905 | |
| Sanjoy Das | e1e352d | 2015-09-20 18:42:50 +0000 | [diff] [blame] | 1906 | public: |
| 1907 | WideIVInfo WI; |
| Andrew Trick | b6bc783 | 2014-01-02 21:12:11 +0000 | [diff] [blame] | 1908 | |
| Sanjoy Das | e1e352d | 2015-09-20 18:42:50 +0000 | [diff] [blame] | 1909 | IndVarSimplifyVisitor(PHINode *IV, ScalarEvolution *SCEV, |
| 1910 | const TargetTransformInfo *TTI, |
| 1911 | const DominatorTree *DTree) |
| 1912 | : SE(SCEV), TTI(TTI), IVPhi(IV) { |
| 1913 | DT = DTree; |
| 1914 | WI.NarrowIV = IVPhi; |
| Sanjoy Das | e1e352d | 2015-09-20 18:42:50 +0000 | [diff] [blame] | 1915 | } |
| Andrew Trick | b6bc783 | 2014-01-02 21:12:11 +0000 | [diff] [blame] | 1916 | |
| Sanjoy Das | e1e352d | 2015-09-20 18:42:50 +0000 | [diff] [blame] | 1917 | // Implement the interface used by simplifyUsersOfIV. |
| 1918 | void visitCast(CastInst *Cast) override { visitIVCast(Cast, WI, SE, TTI); } |
| 1919 | }; |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 1920 | |
| 1921 | } // end anonymous namespace |
| Andrew Trick | 81683ed | 2011-05-12 00:04:28 +0000 | [diff] [blame] | 1922 | |
| Sanjoy Das | 9119bf4 | 2015-09-20 06:58:03 +0000 | [diff] [blame] | 1923 | /// Iteratively perform simplification on a worklist of IV users. Each |
| 1924 | /// successive simplification may push more users which may themselves be |
| 1925 | /// candidates for simplification. |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 1926 | /// |
| Andrew Trick | 3ec331e | 2011-08-10 03:46:27 +0000 | [diff] [blame] | 1927 | /// Sign/Zero extend elimination is interleaved with IV simplification. |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 1928 | bool IndVarSimplify::simplifyAndExtend(Loop *L, |
| Andrew Trick | 3ec331e | 2011-08-10 03:46:27 +0000 | [diff] [blame] | 1929 | SCEVExpander &Rewriter, |
| Justin Bogner | 843fb20 | 2015-12-15 19:40:57 +0000 | [diff] [blame] | 1930 | LoopInfo *LI) { |
| Andrew Trick | d50861c | 2011-10-15 01:38:14 +0000 | [diff] [blame] | 1931 | SmallVector<WideIVInfo, 8> WideIVs; |
| Andrew Trick | 8a3c39c | 2011-06-28 02:49:20 +0000 | [diff] [blame] | 1932 | |
| Artur Pilipenko | 5c6ef75 | 2016-10-19 19:43:54 +0000 | [diff] [blame] | 1933 | auto *GuardDecl = L->getBlocks()[0]->getModule()->getFunction( |
| 1934 | Intrinsic::getName(Intrinsic::experimental_guard)); |
| 1935 | bool HasGuards = GuardDecl && !GuardDecl->use_empty(); |
| 1936 | |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 1937 | SmallVector<PHINode*, 8> LoopPhis; |
| 1938 | for (BasicBlock::iterator I = L->getHeader()->begin(); isa<PHINode>(I); ++I) { |
| 1939 | LoopPhis.push_back(cast<PHINode>(I)); |
| 1940 | } |
| Andrew Trick | 8a3c39c | 2011-06-28 02:49:20 +0000 | [diff] [blame] | 1941 | // Each round of simplification iterates through the SimplifyIVUsers worklist |
| 1942 | // for all current phis, then determines whether any IVs can be |
| 1943 | // widened. Widening adds new phis to LoopPhis, inducing another round of |
| 1944 | // simplification on the wide IVs. |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 1945 | bool Changed = false; |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 1946 | while (!LoopPhis.empty()) { |
| Andrew Trick | 8a3c39c | 2011-06-28 02:49:20 +0000 | [diff] [blame] | 1947 | // Evaluate as many IV expressions as possible before widening any IVs. This |
| Andrew Trick | 4426f5b | 2011-06-28 16:45:04 +0000 | [diff] [blame] | 1948 | // forces SCEV to set no-wrap flags before evaluating sign/zero |
| Andrew Trick | 8a3c39c | 2011-06-28 02:49:20 +0000 | [diff] [blame] | 1949 | // extension. The first time SCEV attempts to normalize sign/zero extension, |
| 1950 | // the result becomes final. So for the most predictable results, we delay |
| 1951 | // evaluation of sign/zero extend evaluation until needed, and avoid running |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 1952 | // other SCEV based analysis prior to simplifyAndExtend. |
| Andrew Trick | 8a3c39c | 2011-06-28 02:49:20 +0000 | [diff] [blame] | 1953 | do { |
| 1954 | PHINode *CurrIV = LoopPhis.pop_back_val(); |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 1955 | |
| Andrew Trick | 8a3c39c | 2011-06-28 02:49:20 +0000 | [diff] [blame] | 1956 | // Information about sign/zero extensions of CurrIV. |
| Mehdi Amini | a28d91d | 2015-03-10 02:37:25 +0000 | [diff] [blame] | 1957 | IndVarSimplifyVisitor Visitor(CurrIV, SE, TTI, DT); |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 1958 | |
| Hongbin Zheng | d36f2030 | 2017-10-12 02:54:11 +0000 | [diff] [blame] | 1959 | Changed |= |
| 1960 | simplifyUsersOfIV(CurrIV, SE, DT, LI, DeadInsts, Rewriter, &Visitor); |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 1961 | |
| Andrew Trick | b6bc783 | 2014-01-02 21:12:11 +0000 | [diff] [blame] | 1962 | if (Visitor.WI.WidestNativeType) { |
| 1963 | WideIVs.push_back(Visitor.WI); |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 1964 | } |
| Andrew Trick | 8a3c39c | 2011-06-28 02:49:20 +0000 | [diff] [blame] | 1965 | } while(!LoopPhis.empty()); |
| 1966 | |
| Andrew Trick | d50861c | 2011-10-15 01:38:14 +0000 | [diff] [blame] | 1967 | for (; !WideIVs.empty(); WideIVs.pop_back()) { |
| Artur Pilipenko | 5c6ef75 | 2016-10-19 19:43:54 +0000 | [diff] [blame] | 1968 | WidenIV Widener(WideIVs.back(), LI, SE, DT, DeadInsts, HasGuards); |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 1969 | if (PHINode *WidePhi = Widener.createWideIV(Rewriter)) { |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 1970 | Changed = true; |
| 1971 | LoopPhis.push_back(WidePhi); |
| 1972 | } |
| 1973 | } |
| 1974 | } |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 1975 | return Changed; |
| Andrew Trick | 69d4452 | 2011-06-21 03:22:38 +0000 | [diff] [blame] | 1976 | } |
| 1977 | |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 1978 | //===----------------------------------------------------------------------===// |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 1979 | // linearFunctionTestReplace and its kin. Rewrite the loop exit condition. |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 1980 | //===----------------------------------------------------------------------===// |
| 1981 | |
| Sanjoy Das | 9119bf4 | 2015-09-20 06:58:03 +0000 | [diff] [blame] | 1982 | /// Return true if this loop's backedge taken count expression can be safely and |
| 1983 | /// cheaply expanded into an instruction sequence that can be used by |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 1984 | /// linearFunctionTestReplace. |
| Sanjoy Das | 2e6bb3b | 2015-04-14 03:20:28 +0000 | [diff] [blame] | 1985 | static bool canExpandBackedgeTakenCount(Loop *L, ScalarEvolution *SE, |
| 1986 | SCEVExpander &Rewriter) { |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 1987 | const SCEV *BackedgeTakenCount = SE->getBackedgeTakenCount(L); |
| Philip Reames | 9283f18 | 2019-05-17 01:12:02 +0000 | [diff] [blame] | 1988 | if (isa<SCEVCouldNotCompute>(BackedgeTakenCount)) |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 1989 | return false; |
| 1990 | |
| Philip Reames | 9283f18 | 2019-05-17 01:12:02 +0000 | [diff] [blame] | 1991 | // Better to break the backedge |
| 1992 | if (BackedgeTakenCount->isZero()) |
| 1993 | return false; |
| 1994 | |
| 1995 | // Loops with multiple exits are not currently suported by lftr |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 1996 | if (!L->getExitingBlock()) |
| 1997 | return false; |
| 1998 | |
| 1999 | // Can't rewrite non-branch yet. |
| Sanjoy Das | 2e6bb3b | 2015-04-14 03:20:28 +0000 | [diff] [blame] | 2000 | if (!isa<BranchInst>(L->getExitingBlock()->getTerminator())) |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2001 | return false; |
| 2002 | |
| Sanjoy Das | 2e6bb3b | 2015-04-14 03:20:28 +0000 | [diff] [blame] | 2003 | if (Rewriter.isHighCostExpansion(BackedgeTakenCount, L)) |
| Andrew Trick | a27d8b1 | 2011-07-18 18:21:35 +0000 | [diff] [blame] | 2004 | return false; |
| 2005 | |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2006 | return true; |
| 2007 | } |
| 2008 | |
| Philip Reames | 9283f18 | 2019-05-17 01:12:02 +0000 | [diff] [blame] | 2009 | /// Given an Value which is hoped to be part of an add recurance in the given |
| 2010 | /// loop, return the associated Phi node if so. Otherwise, return null. Note |
| 2011 | /// that this is less general than SCEVs AddRec checking. |
| Philip Reames | 45e7690 | 2019-05-17 02:09:03 +0000 | [diff] [blame^] | 2012 | static PHINode *getLoopPhiForCounter(Value *IncV, Loop *L) { |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2013 | Instruction *IncI = dyn_cast<Instruction>(IncV); |
| 2014 | if (!IncI) |
| Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 2015 | return nullptr; |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2016 | |
| 2017 | switch (IncI->getOpcode()) { |
| 2018 | case Instruction::Add: |
| 2019 | case Instruction::Sub: |
| 2020 | break; |
| 2021 | case Instruction::GetElementPtr: |
| 2022 | // An IV counter must preserve its type. |
| 2023 | if (IncI->getNumOperands() == 2) |
| 2024 | break; |
| Galina Kistanova | 55344ab | 2017-06-03 05:19:10 +0000 | [diff] [blame] | 2025 | LLVM_FALLTHROUGH; |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2026 | default: |
| Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 2027 | return nullptr; |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2028 | } |
| 2029 | |
| 2030 | PHINode *Phi = dyn_cast<PHINode>(IncI->getOperand(0)); |
| 2031 | if (Phi && Phi->getParent() == L->getHeader()) { |
| Philip Reames | 45e7690 | 2019-05-17 02:09:03 +0000 | [diff] [blame^] | 2032 | if (L->isLoopInvariant(IncI->getOperand(1))) |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2033 | return Phi; |
| Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 2034 | return nullptr; |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2035 | } |
| 2036 | if (IncI->getOpcode() == Instruction::GetElementPtr) |
| Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 2037 | return nullptr; |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2038 | |
| 2039 | // Allow add/sub to be commuted. |
| 2040 | Phi = dyn_cast<PHINode>(IncI->getOperand(1)); |
| 2041 | if (Phi && Phi->getParent() == L->getHeader()) { |
| Philip Reames | 45e7690 | 2019-05-17 02:09:03 +0000 | [diff] [blame^] | 2042 | if (L->isLoopInvariant(IncI->getOperand(0))) |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2043 | return Phi; |
| 2044 | } |
| Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 2045 | return nullptr; |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2046 | } |
| 2047 | |
| Philip Reames | 9283f18 | 2019-05-17 01:12:02 +0000 | [diff] [blame] | 2048 | /// Given a loop with one backedge and one exit, return the ICmpInst |
| 2049 | /// controlling the sole loop exit. There is no guarantee that the exiting |
| 2050 | /// block is also the latch. |
| Andrew Trick | c087266 | 2012-07-18 04:35:10 +0000 | [diff] [blame] | 2051 | static ICmpInst *getLoopTest(Loop *L) { |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2052 | assert(L->getExitingBlock() && "expected loop exit"); |
| 2053 | |
| 2054 | BasicBlock *LatchBlock = L->getLoopLatch(); |
| 2055 | // Don't bother with LFTR if the loop is not properly simplified. |
| 2056 | if (!LatchBlock) |
| Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 2057 | return nullptr; |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2058 | |
| 2059 | BranchInst *BI = dyn_cast<BranchInst>(L->getExitingBlock()->getTerminator()); |
| 2060 | assert(BI && "expected exit branch"); |
| 2061 | |
| Andrew Trick | c087266 | 2012-07-18 04:35:10 +0000 | [diff] [blame] | 2062 | return dyn_cast<ICmpInst>(BI->getCondition()); |
| 2063 | } |
| 2064 | |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 2065 | /// linearFunctionTestReplace policy. Return true unless we can show that the |
| Sanjoy Das | 9119bf4 | 2015-09-20 06:58:03 +0000 | [diff] [blame] | 2066 | /// current exit test is already sufficiently canonical. |
| Philip Reames | 45e7690 | 2019-05-17 02:09:03 +0000 | [diff] [blame^] | 2067 | static bool needsLFTR(Loop *L) { |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2068 | // Do LFTR to simplify the exit condition to an ICMP. |
| Andrew Trick | c087266 | 2012-07-18 04:35:10 +0000 | [diff] [blame] | 2069 | ICmpInst *Cond = getLoopTest(L); |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2070 | if (!Cond) |
| 2071 | return true; |
| 2072 | |
| 2073 | // Do LFTR to simplify the exit ICMP to EQ/NE |
| 2074 | ICmpInst::Predicate Pred = Cond->getPredicate(); |
| 2075 | if (Pred != ICmpInst::ICMP_NE && Pred != ICmpInst::ICMP_EQ) |
| 2076 | return true; |
| 2077 | |
| 2078 | // Look for a loop invariant RHS |
| 2079 | Value *LHS = Cond->getOperand(0); |
| 2080 | Value *RHS = Cond->getOperand(1); |
| Philip Reames | 45e7690 | 2019-05-17 02:09:03 +0000 | [diff] [blame^] | 2081 | if (!L->isLoopInvariant(RHS)) { |
| 2082 | if (!L->isLoopInvariant(LHS)) |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2083 | return true; |
| 2084 | std::swap(LHS, RHS); |
| 2085 | } |
| 2086 | // Look for a simple IV counter LHS |
| 2087 | PHINode *Phi = dyn_cast<PHINode>(LHS); |
| 2088 | if (!Phi) |
| Philip Reames | 45e7690 | 2019-05-17 02:09:03 +0000 | [diff] [blame^] | 2089 | Phi = getLoopPhiForCounter(LHS, L); |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2090 | |
| 2091 | if (!Phi) |
| 2092 | return true; |
| 2093 | |
| Jakub Staszak | e076cac | 2012-10-04 19:08:30 +0000 | [diff] [blame] | 2094 | // Do LFTR if PHI node is defined in the loop, but is *not* a counter. |
| Jakub Staszak | f8a8129 | 2012-10-03 23:59:47 +0000 | [diff] [blame] | 2095 | int Idx = Phi->getBasicBlockIndex(L->getLoopLatch()); |
| 2096 | if (Idx < 0) |
| 2097 | return true; |
| Jakub Staszak | e076cac | 2012-10-04 19:08:30 +0000 | [diff] [blame] | 2098 | |
| 2099 | // Do LFTR if the exit condition's IV is *not* a simple counter. |
| Jakub Staszak | f8a8129 | 2012-10-03 23:59:47 +0000 | [diff] [blame] | 2100 | Value *IncV = Phi->getIncomingValue(Idx); |
| Philip Reames | 45e7690 | 2019-05-17 02:09:03 +0000 | [diff] [blame^] | 2101 | return Phi != getLoopPhiForCounter(IncV, L); |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2102 | } |
| 2103 | |
| Andrew Trick | c087266 | 2012-07-18 04:35:10 +0000 | [diff] [blame] | 2104 | /// Recursive helper for hasConcreteDef(). Unfortunately, this currently boils |
| 2105 | /// down to checking that all operands are constant and listing instructions |
| 2106 | /// that may hide undef. |
| Craig Topper | 71b7b68 | 2014-08-21 05:55:13 +0000 | [diff] [blame] | 2107 | static bool hasConcreteDefImpl(Value *V, SmallPtrSetImpl<Value*> &Visited, |
| Andrew Trick | c087266 | 2012-07-18 04:35:10 +0000 | [diff] [blame] | 2108 | unsigned Depth) { |
| 2109 | if (isa<Constant>(V)) |
| 2110 | return !isa<UndefValue>(V); |
| 2111 | |
| 2112 | if (Depth >= 6) |
| 2113 | return false; |
| 2114 | |
| 2115 | // Conservatively handle non-constant non-instructions. For example, Arguments |
| 2116 | // may be undef. |
| 2117 | Instruction *I = dyn_cast<Instruction>(V); |
| 2118 | if (!I) |
| 2119 | return false; |
| 2120 | |
| 2121 | // Load and return values may be undef. |
| 2122 | if(I->mayReadFromMemory() || isa<CallInst>(I) || isa<InvokeInst>(I)) |
| 2123 | return false; |
| 2124 | |
| 2125 | // Optimistically handle other instructions. |
| Sanjoy Das | 42e551b | 2015-12-08 23:52:58 +0000 | [diff] [blame] | 2126 | for (Value *Op : I->operands()) { |
| 2127 | if (!Visited.insert(Op).second) |
| Andrew Trick | c087266 | 2012-07-18 04:35:10 +0000 | [diff] [blame] | 2128 | continue; |
| Sanjoy Das | 42e551b | 2015-12-08 23:52:58 +0000 | [diff] [blame] | 2129 | if (!hasConcreteDefImpl(Op, Visited, Depth+1)) |
| Andrew Trick | c087266 | 2012-07-18 04:35:10 +0000 | [diff] [blame] | 2130 | return false; |
| 2131 | } |
| 2132 | return true; |
| 2133 | } |
| 2134 | |
| 2135 | /// Return true if the given value is concrete. We must prove that undef can |
| 2136 | /// never reach it. |
| 2137 | /// |
| 2138 | /// TODO: If we decide that this is a good approach to checking for undef, we |
| 2139 | /// may factor it into a common location. |
| 2140 | static bool hasConcreteDef(Value *V) { |
| 2141 | SmallPtrSet<Value*, 8> Visited; |
| 2142 | Visited.insert(V); |
| 2143 | return hasConcreteDefImpl(V, Visited, 0); |
| 2144 | } |
| 2145 | |
| Sanjoy Das | 9119bf4 | 2015-09-20 06:58:03 +0000 | [diff] [blame] | 2146 | /// Return true if this IV has any uses other than the (soon to be rewritten) |
| 2147 | /// loop exit test. |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2148 | static bool AlmostDeadIV(PHINode *Phi, BasicBlock *LatchBlock, Value *Cond) { |
| 2149 | int LatchIdx = Phi->getBasicBlockIndex(LatchBlock); |
| 2150 | Value *IncV = Phi->getIncomingValue(LatchIdx); |
| 2151 | |
| Chandler Carruth | cdf4788 | 2014-03-09 03:16:01 +0000 | [diff] [blame] | 2152 | for (User *U : Phi->users()) |
| 2153 | if (U != Cond && U != IncV) return false; |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2154 | |
| Chandler Carruth | cdf4788 | 2014-03-09 03:16:01 +0000 | [diff] [blame] | 2155 | for (User *U : IncV->users()) |
| 2156 | if (U != Cond && U != Phi) return false; |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2157 | return true; |
| 2158 | } |
| 2159 | |
| Philip Reames | 8e169cd | 2019-05-17 01:39:58 +0000 | [diff] [blame] | 2160 | /// Return true if the given phi is a "counter" in L. A counter is an |
| 2161 | /// add recurance (of integer or pointer type) with an arbitrary start, and a |
| 2162 | /// step of 1. Note that L must have exactly one latch. |
| 2163 | static bool isLoopCounter(PHINode* Phi, Loop *L, |
| Philip Reames | 45e7690 | 2019-05-17 02:09:03 +0000 | [diff] [blame^] | 2164 | ScalarEvolution *SE) { |
| Philip Reames | 8e169cd | 2019-05-17 01:39:58 +0000 | [diff] [blame] | 2165 | assert(Phi->getParent() == L->getHeader()); |
| 2166 | assert(L->getLoopLatch()); |
| 2167 | |
| 2168 | if (!SE->isSCEVable(Phi->getType())) |
| 2169 | return false; |
| 2170 | |
| 2171 | const SCEVAddRecExpr *AR = dyn_cast<SCEVAddRecExpr>(SE->getSCEV(Phi)); |
| 2172 | if (!AR || AR->getLoop() != L || !AR->isAffine()) |
| 2173 | return false; |
| 2174 | |
| 2175 | const SCEV *Step = dyn_cast<SCEVConstant>(AR->getStepRecurrence(*SE)); |
| 2176 | if (!Step || !Step->isOne()) |
| 2177 | return false; |
| 2178 | |
| 2179 | int LatchIdx = Phi->getBasicBlockIndex(L->getLoopLatch()); |
| 2180 | Value *IncV = Phi->getIncomingValue(LatchIdx); |
| Philip Reames | 45e7690 | 2019-05-17 02:09:03 +0000 | [diff] [blame^] | 2181 | return (getLoopPhiForCounter(IncV, L) == Phi); |
| Philip Reames | 8e169cd | 2019-05-17 01:39:58 +0000 | [diff] [blame] | 2182 | } |
| 2183 | |
| 2184 | /// Search the loop header for a loop counter (anadd rec w/step of one) |
| 2185 | /// suitable for use by LFTR. If multiple counters are available, select the |
| 2186 | /// "best" one based profitable heuristics. |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2187 | /// |
| Andrew Trick | c2c79c9 | 2011-11-02 17:19:57 +0000 | [diff] [blame] | 2188 | /// BECount may be an i8* pointer type. The pointer difference is already |
| 2189 | /// valid count without scaling the address stride, so it remains a pointer |
| 2190 | /// expression as far as SCEV is concerned. |
| Mehdi Amini | a28d91d | 2015-03-10 02:37:25 +0000 | [diff] [blame] | 2191 | static PHINode *FindLoopCounter(Loop *L, const SCEV *BECount, |
| Philip Reames | 45e7690 | 2019-05-17 02:09:03 +0000 | [diff] [blame^] | 2192 | ScalarEvolution *SE) { |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2193 | uint64_t BCWidth = SE->getTypeSizeInBits(BECount->getType()); |
| 2194 | |
| 2195 | Value *Cond = |
| 2196 | cast<BranchInst>(L->getExitingBlock()->getTerminator())->getCondition(); |
| 2197 | |
| 2198 | // Loop over all of the PHI nodes, looking for a simple counter. |
| Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 2199 | PHINode *BestPhi = nullptr; |
| 2200 | const SCEV *BestInit = nullptr; |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2201 | BasicBlock *LatchBlock = L->getLoopLatch(); |
| 2202 | assert(LatchBlock && "needsLFTR should guarantee a loop latch"); |
| Sanjoy Das | cddde58 | 2016-01-27 17:05:09 +0000 | [diff] [blame] | 2203 | const DataLayout &DL = L->getHeader()->getModule()->getDataLayout(); |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2204 | |
| 2205 | for (BasicBlock::iterator I = L->getHeader()->begin(); isa<PHINode>(I); ++I) { |
| 2206 | PHINode *Phi = cast<PHINode>(I); |
| Philip Reames | 45e7690 | 2019-05-17 02:09:03 +0000 | [diff] [blame^] | 2207 | if (!isLoopCounter(Phi, L, SE)) |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2208 | continue; |
| 2209 | |
| Andrew Trick | c2c79c9 | 2011-11-02 17:19:57 +0000 | [diff] [blame] | 2210 | // Avoid comparing an integer IV against a pointer Limit. |
| 2211 | if (BECount->getType()->isPointerTy() && !Phi->getType()->isPointerTy()) |
| 2212 | continue; |
| 2213 | |
| Philip Reames | 8e169cd | 2019-05-17 01:39:58 +0000 | [diff] [blame] | 2214 | const auto *AR = dyn_cast<SCEVAddRecExpr>(SE->getSCEV(Phi)); |
| 2215 | |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2216 | // AR may be a pointer type, while BECount is an integer type. |
| 2217 | // AR may be wider than BECount. With eq/ne tests overflow is immaterial. |
| 2218 | // AR may not be a narrower type, or we may never exit. |
| 2219 | uint64_t PhiWidth = SE->getTypeSizeInBits(AR->getType()); |
| Sanjoy Das | cddde58 | 2016-01-27 17:05:09 +0000 | [diff] [blame] | 2220 | if (PhiWidth < BCWidth || !DL.isLegalInteger(PhiWidth)) |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2221 | continue; |
| 2222 | |
| Andrew Trick | c087266 | 2012-07-18 04:35:10 +0000 | [diff] [blame] | 2223 | // Avoid reusing a potentially undef value to compute other values that may |
| 2224 | // have originally had a concrete definition. |
| 2225 | if (!hasConcreteDef(Phi)) { |
| 2226 | // We explicitly allow unknown phis as long as they are already used by |
| 2227 | // the loop test. In this case we assume that performing LFTR could not |
| 2228 | // increase the number of undef users. |
| 2229 | if (ICmpInst *Cond = getLoopTest(L)) { |
| Philip Reames | 45e7690 | 2019-05-17 02:09:03 +0000 | [diff] [blame^] | 2230 | if (Phi != getLoopPhiForCounter(Cond->getOperand(0), L) && |
| 2231 | Phi != getLoopPhiForCounter(Cond->getOperand(1), L)) { |
| Andrew Trick | c087266 | 2012-07-18 04:35:10 +0000 | [diff] [blame] | 2232 | continue; |
| 2233 | } |
| 2234 | } |
| 2235 | } |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2236 | const SCEV *Init = AR->getStart(); |
| 2237 | |
| 2238 | if (BestPhi && !AlmostDeadIV(BestPhi, LatchBlock, Cond)) { |
| 2239 | // Don't force a live loop counter if another IV can be used. |
| 2240 | if (AlmostDeadIV(Phi, LatchBlock, Cond)) |
| 2241 | continue; |
| 2242 | |
| 2243 | // Prefer to count-from-zero. This is a more "canonical" counter form. It |
| 2244 | // also prefers integer to pointer IVs. |
| 2245 | if (BestInit->isZero() != Init->isZero()) { |
| 2246 | if (BestInit->isZero()) |
| 2247 | continue; |
| 2248 | } |
| 2249 | // If two IVs both count from zero or both count from nonzero then the |
| 2250 | // narrower is likely a dead phi that has been widened. Use the wider phi |
| 2251 | // to allow the other to be eliminated. |
| Andrew Trick | 0d07dfc | 2012-07-18 04:35:13 +0000 | [diff] [blame] | 2252 | else if (PhiWidth <= SE->getTypeSizeInBits(BestPhi->getType())) |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2253 | continue; |
| 2254 | } |
| 2255 | BestPhi = Phi; |
| 2256 | BestInit = Init; |
| 2257 | } |
| 2258 | return BestPhi; |
| 2259 | } |
| 2260 | |
| Philip Reames | 9283f18 | 2019-05-17 01:12:02 +0000 | [diff] [blame] | 2261 | /// Insert an IR expression which computes the value held by the IV IndVar |
| Philip Reames | 8e169cd | 2019-05-17 01:39:58 +0000 | [diff] [blame] | 2262 | /// (which must be an loop counter w/unit stride) after the backedge of loop L |
| Philip Reames | 9283f18 | 2019-05-17 01:12:02 +0000 | [diff] [blame] | 2263 | /// is taken IVCount times. |
| Andrew Trick | c2c79c9 | 2011-11-02 17:19:57 +0000 | [diff] [blame] | 2264 | static Value *genLoopLimit(PHINode *IndVar, const SCEV *IVCount, Loop *L, |
| Chandler Carruth | 7ec5085 | 2012-11-01 08:07:29 +0000 | [diff] [blame] | 2265 | SCEVExpander &Rewriter, ScalarEvolution *SE) { |
| Andrew Trick | c2c79c9 | 2011-11-02 17:19:57 +0000 | [diff] [blame] | 2266 | const SCEVAddRecExpr *AR = dyn_cast<SCEVAddRecExpr>(SE->getSCEV(IndVar)); |
| 2267 | assert(AR && AR->getLoop() == L && AR->isAffine() && "bad loop counter"); |
| 2268 | const SCEV *IVInit = AR->getStart(); |
| 2269 | |
| 2270 | // IVInit may be a pointer while IVCount is an integer when FindLoopCounter |
| 2271 | // finds a valid pointer IV. Sign extend BECount in order to materialize a |
| 2272 | // GEP. Avoid running SCEVExpander on a new pointer value, instead reusing |
| 2273 | // the existing GEPs whenever possible. |
| Sanjoy Das | 91e6ba6 | 2016-06-24 21:23:32 +0000 | [diff] [blame] | 2274 | if (IndVar->getType()->isPointerTy() && !IVCount->getType()->isPointerTy()) { |
| Juergen Ributzka | d04d096 | 2013-10-24 05:29:56 +0000 | [diff] [blame] | 2275 | // IVOffset will be the new GEP offset that is interpreted by GEP as a |
| 2276 | // signed value. IVCount on the other hand represents the loop trip count, |
| 2277 | // which is an unsigned value. FindLoopCounter only allows induction |
| 2278 | // variables that have a positive unit stride of one. This means we don't |
| 2279 | // have to handle the case of negative offsets (yet) and just need to zero |
| 2280 | // extend IVCount. |
| Andrew Trick | c2c79c9 | 2011-11-02 17:19:57 +0000 | [diff] [blame] | 2281 | Type *OfsTy = SE->getEffectiveSCEVType(IVInit->getType()); |
| Juergen Ributzka | d04d096 | 2013-10-24 05:29:56 +0000 | [diff] [blame] | 2282 | const SCEV *IVOffset = SE->getTruncateOrZeroExtend(IVCount, OfsTy); |
| Andrew Trick | c2c79c9 | 2011-11-02 17:19:57 +0000 | [diff] [blame] | 2283 | |
| 2284 | // Expand the code for the iteration count. |
| 2285 | assert(SE->isLoopInvariant(IVOffset, L) && |
| 2286 | "Computed iteration count is not loop invariant!"); |
| 2287 | BranchInst *BI = cast<BranchInst>(L->getExitingBlock()->getTerminator()); |
| 2288 | Value *GEPOffset = Rewriter.expandCodeFor(IVOffset, OfsTy, BI); |
| 2289 | |
| 2290 | Value *GEPBase = IndVar->getIncomingValueForBlock(L->getLoopPreheader()); |
| 2291 | assert(AR->getStart() == SE->getSCEV(GEPBase) && "bad loop counter"); |
| 2292 | // We could handle pointer IVs other than i8*, but we need to compensate for |
| 2293 | // gep index scaling. See canExpandBackedgeTakenCount comments. |
| Matt Arsenault | a90a18e | 2013-09-10 19:55:24 +0000 | [diff] [blame] | 2294 | assert(SE->getSizeOfExpr(IntegerType::getInt64Ty(IndVar->getContext()), |
| Sanjoy Das | 91e6ba6 | 2016-06-24 21:23:32 +0000 | [diff] [blame] | 2295 | cast<PointerType>(GEPBase->getType()) |
| 2296 | ->getElementType())->isOne() && |
| 2297 | "unit stride pointer IV must be i8*"); |
| Andrew Trick | c2c79c9 | 2011-11-02 17:19:57 +0000 | [diff] [blame] | 2298 | |
| 2299 | IRBuilder<> Builder(L->getLoopPreheader()->getTerminator()); |
| James Y Knight | 7716075 | 2019-02-01 20:44:47 +0000 | [diff] [blame] | 2300 | return Builder.CreateGEP(GEPBase->getType()->getPointerElementType(), |
| 2301 | GEPBase, GEPOffset, "lftr.limit"); |
| Sanjoy Das | 91e6ba6 | 2016-06-24 21:23:32 +0000 | [diff] [blame] | 2302 | } else { |
| Andrew Trick | c2c79c9 | 2011-11-02 17:19:57 +0000 | [diff] [blame] | 2303 | // In any other case, convert both IVInit and IVCount to integers before |
| Xin Tong | 02b1397 | 2017-01-10 03:13:52 +0000 | [diff] [blame] | 2304 | // comparing. This may result in SCEV expansion of pointers, but in practice |
| Andrew Trick | c2c79c9 | 2011-11-02 17:19:57 +0000 | [diff] [blame] | 2305 | // SCEV will fold the pointer arithmetic away as such: |
| 2306 | // BECount = (IVEnd - IVInit - 1) => IVLimit = IVInit (postinc). |
| 2307 | // |
| 2308 | // Valid Cases: (1) both integers is most common; (2) both may be pointers |
| Andrew Trick | ada2356 | 2013-10-24 00:43:38 +0000 | [diff] [blame] | 2309 | // for simple memset-style loops. |
| 2310 | // |
| 2311 | // IVInit integer and IVCount pointer would only occur if a canonical IV |
| 2312 | // were generated on top of case #2, which is not expected. |
| Andrew Trick | c2c79c9 | 2011-11-02 17:19:57 +0000 | [diff] [blame] | 2313 | |
| Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 2314 | const SCEV *IVLimit = nullptr; |
| Andrew Trick | c2c79c9 | 2011-11-02 17:19:57 +0000 | [diff] [blame] | 2315 | // For unit stride, IVCount = Start + BECount with 2's complement overflow. |
| 2316 | // For non-zero Start, compute IVCount here. |
| 2317 | if (AR->getStart()->isZero()) |
| 2318 | IVLimit = IVCount; |
| 2319 | else { |
| 2320 | assert(AR->getStepRecurrence(*SE)->isOne() && "only handles unit stride"); |
| 2321 | const SCEV *IVInit = AR->getStart(); |
| 2322 | |
| 2323 | // For integer IVs, truncate the IV before computing IVInit + BECount. |
| 2324 | if (SE->getTypeSizeInBits(IVInit->getType()) |
| 2325 | > SE->getTypeSizeInBits(IVCount->getType())) |
| 2326 | IVInit = SE->getTruncateExpr(IVInit, IVCount->getType()); |
| 2327 | |
| 2328 | IVLimit = SE->getAddExpr(IVInit, IVCount); |
| 2329 | } |
| 2330 | // Expand the code for the iteration count. |
| 2331 | BranchInst *BI = cast<BranchInst>(L->getExitingBlock()->getTerminator()); |
| 2332 | IRBuilder<> Builder(BI); |
| 2333 | assert(SE->isLoopInvariant(IVLimit, L) && |
| 2334 | "Computed iteration count is not loop invariant!"); |
| 2335 | // Ensure that we generate the same type as IndVar, or a smaller integer |
| 2336 | // type. In the presence of null pointer values, we have an integer type |
| 2337 | // SCEV expression (IVInit) for a pointer type IV value (IndVar). |
| 2338 | Type *LimitTy = IVCount->getType()->isPointerTy() ? |
| 2339 | IndVar->getType() : IVCount->getType(); |
| 2340 | return Rewriter.expandCodeFor(IVLimit, LimitTy, BI); |
| 2341 | } |
| 2342 | } |
| 2343 | |
| Sanjoy Das | 9119bf4 | 2015-09-20 06:58:03 +0000 | [diff] [blame] | 2344 | /// This method rewrites the exit condition of the loop to be a canonical != |
| 2345 | /// comparison against the incremented loop induction variable. This pass is |
| 2346 | /// able to rewrite the exit tests of any loop where the SCEV analysis can |
| 2347 | /// determine a loop-invariant trip count of the loop, which is actually a much |
| 2348 | /// broader range than just linear tests. |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 2349 | bool IndVarSimplify:: |
| 2350 | linearFunctionTestReplace(Loop *L, const SCEV *BackedgeTakenCount, |
| 2351 | PHINode *IndVar, SCEVExpander &Rewriter) { |
| Sanjoy Das | 2e6bb3b | 2015-04-14 03:20:28 +0000 | [diff] [blame] | 2352 | assert(canExpandBackedgeTakenCount(L, SE, Rewriter) && "precondition"); |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2353 | |
| Andrew Trick | 2b71848 | 2013-07-12 22:08:44 +0000 | [diff] [blame] | 2354 | // Initialize CmpIndVar and IVCount to their preincremented values. |
| 2355 | Value *CmpIndVar = IndVar; |
| 2356 | const SCEV *IVCount = BackedgeTakenCount; |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2357 | |
| Sanjoy Das | 85cd132 | 2017-02-20 23:37:11 +0000 | [diff] [blame] | 2358 | assert(L->getLoopLatch() && "Loop no longer in simplified form?"); |
| 2359 | |
| Andrew Trick | c2c79c9 | 2011-11-02 17:19:57 +0000 | [diff] [blame] | 2360 | // If the exiting block is the same as the backedge block, we prefer to |
| 2361 | // compare against the post-incremented value, otherwise we must compare |
| 2362 | // against the preincremented value. |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2363 | if (L->getExitingBlock() == L->getLoopLatch()) { |
| Sanjoy Das | 2d38031 | 2015-03-02 21:41:07 +0000 | [diff] [blame] | 2364 | // Add one to the "backedge-taken" count to get the trip count. |
| 2365 | // This addition may overflow, which is valid as long as the comparison is |
| 2366 | // truncated to BackedgeTakenCount->getType(). |
| 2367 | IVCount = SE->getAddExpr(BackedgeTakenCount, |
| Sanjoy Das | 2aacc0e | 2015-09-23 01:59:04 +0000 | [diff] [blame] | 2368 | SE->getOne(BackedgeTakenCount->getType())); |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2369 | // The BackedgeTaken expression contains the number of times that the |
| 2370 | // backedge branches to the loop header. This is one less than the |
| 2371 | // number of times the loop executes, so use the incremented indvar. |
| Sanjoy Das | 2d38031 | 2015-03-02 21:41:07 +0000 | [diff] [blame] | 2372 | CmpIndVar = IndVar->getIncomingValueForBlock(L->getExitingBlock()); |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2373 | } |
| 2374 | |
| Chandler Carruth | 7ec5085 | 2012-11-01 08:07:29 +0000 | [diff] [blame] | 2375 | Value *ExitCnt = genLoopLimit(IndVar, IVCount, L, Rewriter, SE); |
| Sanjoy Das | 91e6ba6 | 2016-06-24 21:23:32 +0000 | [diff] [blame] | 2376 | assert(ExitCnt->getType()->isPointerTy() == |
| 2377 | IndVar->getType()->isPointerTy() && |
| 2378 | "genLoopLimit missed a cast"); |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2379 | |
| 2380 | // Insert a new icmp_ne or icmp_eq instruction before the branch. |
| Andrew Trick | c2c79c9 | 2011-11-02 17:19:57 +0000 | [diff] [blame] | 2381 | BranchInst *BI = cast<BranchInst>(L->getExitingBlock()->getTerminator()); |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2382 | ICmpInst::Predicate P; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2383 | if (L->contains(BI->getSuccessor(0))) |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2384 | P = ICmpInst::ICMP_NE; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2385 | else |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2386 | P = ICmpInst::ICMP_EQ; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2387 | |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 2388 | LLVM_DEBUG(dbgs() << "INDVARS: Rewriting loop exit condition to:\n" |
| 2389 | << " LHS:" << *CmpIndVar << '\n' |
| 2390 | << " op:\t" << (P == ICmpInst::ICMP_NE ? "!=" : "==") |
| 2391 | << "\n" |
| 2392 | << " RHS:\t" << *ExitCnt << "\n" |
| 2393 | << " IVCount:\t" << *IVCount << "\n"); |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2394 | |
| Andrew Trick | a1e4118 | 2013-07-12 22:08:48 +0000 | [diff] [blame] | 2395 | IRBuilder<> Builder(BI); |
| 2396 | |
| Andrea Di Biagio | 9bcb064 | 2016-10-26 10:28:32 +0000 | [diff] [blame] | 2397 | // The new loop exit condition should reuse the debug location of the |
| 2398 | // original loop exit condition. |
| 2399 | if (auto *Cond = dyn_cast<Instruction>(BI->getCondition())) |
| 2400 | Builder.SetCurrentDebugLocation(Cond->getDebugLoc()); |
| 2401 | |
| Andrew Trick | 2b71848 | 2013-07-12 22:08:44 +0000 | [diff] [blame] | 2402 | // LFTR can ignore IV overflow and truncate to the width of |
| 2403 | // BECount. This avoids materializing the add(zext(add)) expression. |
| Andrew Trick | a1e4118 | 2013-07-12 22:08:48 +0000 | [diff] [blame] | 2404 | unsigned CmpIndVarSize = SE->getTypeSizeInBits(CmpIndVar->getType()); |
| 2405 | unsigned ExitCntSize = SE->getTypeSizeInBits(ExitCnt->getType()); |
| 2406 | if (CmpIndVarSize > ExitCntSize) { |
| 2407 | const SCEVAddRecExpr *AR = cast<SCEVAddRecExpr>(SE->getSCEV(IndVar)); |
| 2408 | const SCEV *ARStart = AR->getStart(); |
| 2409 | const SCEV *ARStep = AR->getStepRecurrence(*SE); |
| 2410 | // For constant IVCount, avoid truncation. |
| 2411 | if (isa<SCEVConstant>(ARStart) && isa<SCEVConstant>(IVCount)) { |
| Sanjoy Das | 0de2fec | 2015-12-17 20:28:46 +0000 | [diff] [blame] | 2412 | const APInt &Start = cast<SCEVConstant>(ARStart)->getAPInt(); |
| 2413 | APInt Count = cast<SCEVConstant>(IVCount)->getAPInt(); |
| Andrew Trick | a1e4118 | 2013-07-12 22:08:48 +0000 | [diff] [blame] | 2414 | // Note that the post-inc value of BackedgeTakenCount may have overflowed |
| 2415 | // above such that IVCount is now zero. |
| 2416 | if (IVCount != BackedgeTakenCount && Count == 0) { |
| 2417 | Count = APInt::getMaxValue(Count.getBitWidth()).zext(CmpIndVarSize); |
| 2418 | ++Count; |
| 2419 | } |
| 2420 | else |
| 2421 | Count = Count.zext(CmpIndVarSize); |
| 2422 | APInt NewLimit; |
| 2423 | if (cast<SCEVConstant>(ARStep)->getValue()->isNegative()) |
| 2424 | NewLimit = Start - Count; |
| 2425 | else |
| 2426 | NewLimit = Start + Count; |
| 2427 | ExitCnt = ConstantInt::get(CmpIndVar->getType(), NewLimit); |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2428 | |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 2429 | LLVM_DEBUG(dbgs() << " Widen RHS:\t" << *ExitCnt << "\n"); |
| Andrew Trick | a1e4118 | 2013-07-12 22:08:48 +0000 | [diff] [blame] | 2430 | } else { |
| Ehsan Amiri | dbcfea9 | 2016-08-11 21:31:40 +0000 | [diff] [blame] | 2431 | // We try to extend trip count first. If that doesn't work we truncate IV. |
| 2432 | // Zext(trunc(IV)) == IV implies equivalence of the following two: |
| 2433 | // Trunc(IV) == ExitCnt and IV == zext(ExitCnt). Similarly for sext. If |
| 2434 | // one of the two holds, extend the trip count, otherwise we truncate IV. |
| 2435 | bool Extended = false; |
| 2436 | const SCEV *IV = SE->getSCEV(CmpIndVar); |
| 2437 | const SCEV *ZExtTrunc = |
| 2438 | SE->getZeroExtendExpr(SE->getTruncateExpr(SE->getSCEV(CmpIndVar), |
| 2439 | ExitCnt->getType()), |
| 2440 | CmpIndVar->getType()); |
| Ehsan Amiri | b9fcc2b | 2016-08-11 13:51:20 +0000 | [diff] [blame] | 2441 | |
| Ehsan Amiri | dbcfea9 | 2016-08-11 21:31:40 +0000 | [diff] [blame] | 2442 | if (ZExtTrunc == IV) { |
| 2443 | Extended = true; |
| 2444 | ExitCnt = Builder.CreateZExt(ExitCnt, IndVar->getType(), |
| 2445 | "wide.trip.count"); |
| 2446 | } else { |
| 2447 | const SCEV *SExtTrunc = |
| 2448 | SE->getSignExtendExpr(SE->getTruncateExpr(SE->getSCEV(CmpIndVar), |
| 2449 | ExitCnt->getType()), |
| 2450 | CmpIndVar->getType()); |
| 2451 | if (SExtTrunc == IV) { |
| 2452 | Extended = true; |
| 2453 | ExitCnt = Builder.CreateSExt(ExitCnt, IndVar->getType(), |
| 2454 | "wide.trip.count"); |
| 2455 | } |
| 2456 | } |
| 2457 | |
| 2458 | if (!Extended) |
| Ehsan Amiri | b9fcc2b | 2016-08-11 13:51:20 +0000 | [diff] [blame] | 2459 | CmpIndVar = Builder.CreateTrunc(CmpIndVar, ExitCnt->getType(), |
| 2460 | "lftr.wideiv"); |
| Andrew Trick | a1e4118 | 2013-07-12 22:08:48 +0000 | [diff] [blame] | 2461 | } |
| 2462 | } |
| Andrew Trick | 7da2417 | 2011-07-18 20:32:31 +0000 | [diff] [blame] | 2463 | Value *Cond = Builder.CreateICmp(P, CmpIndVar, ExitCnt, "exitcond"); |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2464 | Value *OrigCond = BI->getCondition(); |
| 2465 | // It's tempting to use replaceAllUsesWith here to fully replace the old |
| 2466 | // comparison, but that's not immediately safe, since users of the old |
| 2467 | // comparison may not be dominated by the new comparison. Instead, just |
| 2468 | // update the branch to use the new comparison; in the common case this |
| 2469 | // will make old comparison dead. |
| 2470 | BI->setCondition(Cond); |
| 2471 | DeadInsts.push_back(OrigCond); |
| 2472 | |
| 2473 | ++NumLFTR; |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 2474 | return true; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2475 | } |
| 2476 | |
| 2477 | //===----------------------------------------------------------------------===// |
| Sanjoy Das | b873cbe | 2015-10-13 07:17:38 +0000 | [diff] [blame] | 2478 | // sinkUnusedInvariants. A late subpass to cleanup loop preheaders. |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2479 | //===----------------------------------------------------------------------===// |
| 2480 | |
| 2481 | /// If there's a single exit block, sink any loop-invariant values that |
| 2482 | /// were defined in the preheader but not used inside the loop into the |
| 2483 | /// exit block to reduce register pressure in the loop. |
| Max Kazantsev | 4d10ba3 | 2018-09-10 06:32:00 +0000 | [diff] [blame] | 2484 | bool IndVarSimplify::sinkUnusedInvariants(Loop *L) { |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2485 | BasicBlock *ExitBlock = L->getExitBlock(); |
| Max Kazantsev | 4d10ba3 | 2018-09-10 06:32:00 +0000 | [diff] [blame] | 2486 | if (!ExitBlock) return false; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2487 | |
| 2488 | BasicBlock *Preheader = L->getLoopPreheader(); |
| Max Kazantsev | 4d10ba3 | 2018-09-10 06:32:00 +0000 | [diff] [blame] | 2489 | if (!Preheader) return false; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2490 | |
| Max Kazantsev | 4d10ba3 | 2018-09-10 06:32:00 +0000 | [diff] [blame] | 2491 | bool MadeAnyChanges = false; |
| Duncan P. N. Exon Smith | 362d1204 | 2016-08-17 01:54:41 +0000 | [diff] [blame] | 2492 | BasicBlock::iterator InsertPt = ExitBlock->getFirstInsertionPt(); |
| Duncan P. N. Exon Smith | 3a9c9e3 | 2015-10-13 18:26:00 +0000 | [diff] [blame] | 2493 | BasicBlock::iterator I(Preheader->getTerminator()); |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2494 | while (I != Preheader->begin()) { |
| 2495 | --I; |
| 2496 | // New instructions were inserted at the end of the preheader. |
| 2497 | if (isa<PHINode>(I)) |
| 2498 | break; |
| 2499 | |
| 2500 | // Don't move instructions which might have side effects, since the side |
| 2501 | // effects need to complete before instructions inside the loop. Also don't |
| 2502 | // move instructions which might read memory, since the loop may modify |
| 2503 | // memory. Note that it's okay if the instruction might have undefined |
| 2504 | // behavior: LoopSimplify guarantees that the preheader dominates the exit |
| 2505 | // block. |
| 2506 | if (I->mayHaveSideEffects() || I->mayReadFromMemory()) |
| 2507 | continue; |
| 2508 | |
| 2509 | // Skip debug info intrinsics. |
| 2510 | if (isa<DbgInfoIntrinsic>(I)) |
| 2511 | continue; |
| 2512 | |
| David Majnemer | ba275f9 | 2015-08-19 19:54:02 +0000 | [diff] [blame] | 2513 | // Skip eh pad instructions. |
| 2514 | if (I->isEHPad()) |
| Bill Wendling | eed1e89 | 2011-08-26 20:40:15 +0000 | [diff] [blame] | 2515 | continue; |
| 2516 | |
| Eli Friedman | 73beaf7 | 2011-10-27 01:33:51 +0000 | [diff] [blame] | 2517 | // Don't sink alloca: we never want to sink static alloca's out of the |
| 2518 | // entry block, and correctly sinking dynamic alloca's requires |
| 2519 | // checks for stacksave/stackrestore intrinsics. |
| 2520 | // FIXME: Refactor this check somehow? |
| 2521 | if (isa<AllocaInst>(I)) |
| 2522 | continue; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2523 | |
| 2524 | // Determine if there is a use in or before the loop (direct or |
| 2525 | // otherwise). |
| 2526 | bool UsedInLoop = false; |
| Chandler Carruth | cdf4788 | 2014-03-09 03:16:01 +0000 | [diff] [blame] | 2527 | for (Use &U : I->uses()) { |
| 2528 | Instruction *User = cast<Instruction>(U.getUser()); |
| 2529 | BasicBlock *UseBB = User->getParent(); |
| 2530 | if (PHINode *P = dyn_cast<PHINode>(User)) { |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2531 | unsigned i = |
| Chandler Carruth | cdf4788 | 2014-03-09 03:16:01 +0000 | [diff] [blame] | 2532 | PHINode::getIncomingValueNumForOperand(U.getOperandNo()); |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2533 | UseBB = P->getIncomingBlock(i); |
| 2534 | } |
| 2535 | if (UseBB == Preheader || L->contains(UseBB)) { |
| 2536 | UsedInLoop = true; |
| 2537 | break; |
| 2538 | } |
| 2539 | } |
| 2540 | |
| 2541 | // If there is, the def must remain in the preheader. |
| 2542 | if (UsedInLoop) |
| 2543 | continue; |
| 2544 | |
| 2545 | // Otherwise, sink it to the exit block. |
| Duncan P. N. Exon Smith | 3a9c9e3 | 2015-10-13 18:26:00 +0000 | [diff] [blame] | 2546 | Instruction *ToMove = &*I; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2547 | bool Done = false; |
| 2548 | |
| 2549 | if (I != Preheader->begin()) { |
| 2550 | // Skip debug info intrinsics. |
| 2551 | do { |
| 2552 | --I; |
| 2553 | } while (isa<DbgInfoIntrinsic>(I) && I != Preheader->begin()); |
| 2554 | |
| 2555 | if (isa<DbgInfoIntrinsic>(I) && I == Preheader->begin()) |
| 2556 | Done = true; |
| 2557 | } else { |
| 2558 | Done = true; |
| 2559 | } |
| 2560 | |
| Max Kazantsev | 4d10ba3 | 2018-09-10 06:32:00 +0000 | [diff] [blame] | 2561 | MadeAnyChanges = true; |
| Duncan P. N. Exon Smith | 362d1204 | 2016-08-17 01:54:41 +0000 | [diff] [blame] | 2562 | ToMove->moveBefore(*ExitBlock, InsertPt); |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2563 | if (Done) break; |
| Duncan P. N. Exon Smith | 362d1204 | 2016-08-17 01:54:41 +0000 | [diff] [blame] | 2564 | InsertPt = ToMove->getIterator(); |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2565 | } |
| Max Kazantsev | 4d10ba3 | 2018-09-10 06:32:00 +0000 | [diff] [blame] | 2566 | |
| 2567 | return MadeAnyChanges; |
| Andrew Trick | cdc2297 | 2011-07-12 00:08:50 +0000 | [diff] [blame] | 2568 | } |
| 2569 | |
| 2570 | //===----------------------------------------------------------------------===// |
| 2571 | // IndVarSimplify driver. Manage several subpasses of IV simplification. |
| 2572 | //===----------------------------------------------------------------------===// |
| 2573 | |
| Sanjoy Das | 496f274 | 2016-05-29 21:42:00 +0000 | [diff] [blame] | 2574 | bool IndVarSimplify::run(Loop *L) { |
| Sanjoy Das | 3e5ce2b | 2016-05-30 01:37:39 +0000 | [diff] [blame] | 2575 | // We need (and expect!) the incoming loop to be in LCSSA. |
| Igor Laevsky | 04423cf | 2016-10-11 13:37:22 +0000 | [diff] [blame] | 2576 | assert(L->isRecursivelyLCSSAForm(*DT, *LI) && |
| 2577 | "LCSSA required to run indvars!"); |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 2578 | bool Changed = false; |
| Sanjoy Das | 3e5ce2b | 2016-05-30 01:37:39 +0000 | [diff] [blame] | 2579 | |
| Dan Gohman | f3aea7a | 2010-06-18 01:35:11 +0000 | [diff] [blame] | 2580 | // If LoopSimplify form is not available, stay out of trouble. Some notes: |
| 2581 | // - LSR currently only supports LoopSimplify-form loops. Indvars' |
| 2582 | // canonicalization can be a pessimization without LSR to "clean up" |
| 2583 | // afterwards. |
| 2584 | // - We depend on having a preheader; in particular, |
| 2585 | // Loop::getCanonicalInductionVariable only supports loops with preheaders, |
| 2586 | // and we're in trouble if we can't find the induction variable even when |
| 2587 | // we've manually inserted one. |
| Sanjoy Das | 85cd132 | 2017-02-20 23:37:11 +0000 | [diff] [blame] | 2588 | // - LFTR relies on having a single backedge. |
| Dan Gohman | f3aea7a | 2010-06-18 01:35:11 +0000 | [diff] [blame] | 2589 | if (!L->isLoopSimplifyForm()) |
| 2590 | return false; |
| 2591 | |
| Dan Gohman | 0a40ad9 | 2009-04-16 03:18:22 +0000 | [diff] [blame] | 2592 | // If there are any floating-point recurrences, attempt to |
| Dan Gohman | 4330034 | 2009-02-17 20:49:49 +0000 | [diff] [blame] | 2593 | // transform them to use integer recurrences. |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 2594 | Changed |= rewriteNonIntegerIVs(L); |
| Dan Gohman | 4330034 | 2009-02-17 20:49:49 +0000 | [diff] [blame] | 2595 | |
| Dan Gohman | af75234 | 2009-07-07 17:06:11 +0000 | [diff] [blame] | 2596 | const SCEV *BackedgeTakenCount = SE->getBackedgeTakenCount(L); |
| Chris Lattner | 1f7648e | 2007-03-04 01:00:28 +0000 | [diff] [blame] | 2597 | |
| Dan Gohman | daafbe6 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 2598 | // Create a rewriter object which we'll use to transform the code with. |
| Mehdi Amini | a28d91d | 2015-03-10 02:37:25 +0000 | [diff] [blame] | 2599 | SCEVExpander Rewriter(*SE, DL, "indvars"); |
| Andrew Trick | f9201c5 | 2011-10-11 02:28:51 +0000 | [diff] [blame] | 2600 | #ifndef NDEBUG |
| 2601 | Rewriter.setDebugType(DEBUG_TYPE); |
| 2602 | #endif |
| Andrew Trick | 163b4a7 | 2011-06-27 23:17:44 +0000 | [diff] [blame] | 2603 | |
| 2604 | // Eliminate redundant IV users. |
| Andrew Trick | 8a3c39c | 2011-06-28 02:49:20 +0000 | [diff] [blame] | 2605 | // |
| 2606 | // Simplification works best when run before other consumers of SCEV. We |
| 2607 | // attempt to avoid evaluating SCEVs for sign/zero extend operations until |
| 2608 | // other expressions involving loop IVs have been evaluated. This helps SCEV |
| Andrew Trick | 4426f5b | 2011-06-28 16:45:04 +0000 | [diff] [blame] | 2609 | // set no-wrap flags before normalizing sign/zero extension. |
| Andrew Trick | f47d0af | 2012-03-22 17:10:11 +0000 | [diff] [blame] | 2610 | Rewriter.disableCanonicalMode(); |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 2611 | Changed |= simplifyAndExtend(L, Rewriter, LI); |
| Andrew Trick | 1abe296 | 2011-05-04 02:10:13 +0000 | [diff] [blame] | 2612 | |
| Chris Lattner | e61b67d | 2004-04-02 20:24:31 +0000 | [diff] [blame] | 2613 | // Check to see if this loop has a computable loop-invariant execution count. |
| 2614 | // If so, this means that we can compute the final value of any expressions |
| 2615 | // that are recurrent in the loop, and substitute the exit values from the |
| 2616 | // loop into any instructions outside of the loop that use the final values of |
| 2617 | // the current expressions. |
| Chris Lattner | 0b18c1d | 2002-05-10 15:38:35 +0000 | [diff] [blame] | 2618 | // |
| Wei Mi | e2538b5 | 2015-05-28 21:49:07 +0000 | [diff] [blame] | 2619 | if (ReplaceExitValue != NeverRepl && |
| 2620 | !isa<SCEVCouldNotCompute>(BackedgeTakenCount)) |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 2621 | Changed |= rewriteLoopExitValues(L, Rewriter); |
| Chris Lattner | 476e6df | 2001-12-03 17:28:42 +0000 | [diff] [blame] | 2622 | |
| Andrew Trick | 9ea55dc | 2011-07-16 01:06:48 +0000 | [diff] [blame] | 2623 | // Eliminate redundant IV cycles. |
| Andrew Trick | f47d0af | 2012-03-22 17:10:11 +0000 | [diff] [blame] | 2624 | NumElimIV += Rewriter.replaceCongruentIVs(L, DT, DeadInsts); |
| Andrew Trick | 3239055 | 2011-07-06 20:50:43 +0000 | [diff] [blame] | 2625 | |
| Dan Gohman | eb6be65 | 2009-02-12 22:19:27 +0000 | [diff] [blame] | 2626 | // If we have a trip count expression, rewrite the loop's exit condition |
| 2627 | // using it. We can currently only handle loops with a single exit. |
| Serguei Katkov | 38414b5 | 2017-06-09 06:11:59 +0000 | [diff] [blame] | 2628 | if (!DisableLFTR && canExpandBackedgeTakenCount(L, SE, Rewriter) && |
| Philip Reames | 45e7690 | 2019-05-17 02:09:03 +0000 | [diff] [blame^] | 2629 | needsLFTR(L)) { |
| 2630 | PHINode *IndVar = FindLoopCounter(L, BackedgeTakenCount, SE); |
| Andrew Trick | 25553ab | 2012-03-24 00:51:17 +0000 | [diff] [blame] | 2631 | if (IndVar) { |
| 2632 | // Check preconditions for proper SCEVExpander operation. SCEV does not |
| 2633 | // express SCEVExpander's dependencies, such as LoopSimplify. Instead any |
| 2634 | // pass that uses the SCEVExpander must do it. This does not work well for |
| Andrew Trick | b70d978 | 2014-01-07 01:02:52 +0000 | [diff] [blame] | 2635 | // loop passes because SCEVExpander makes assumptions about all loops, |
| 2636 | // while LoopPassManager only forces the current loop to be simplified. |
| Andrew Trick | 25553ab | 2012-03-24 00:51:17 +0000 | [diff] [blame] | 2637 | // |
| 2638 | // FIXME: SCEV expansion has no way to bail out, so the caller must |
| 2639 | // explicitly check any assumptions made by SCEV. Brittle. |
| 2640 | const SCEVAddRecExpr *AR = dyn_cast<SCEVAddRecExpr>(BackedgeTakenCount); |
| 2641 | if (!AR || AR->getLoop()->getLoopPreheader()) |
| Max Kazantsev | e641391 | 2018-09-11 03:57:22 +0000 | [diff] [blame] | 2642 | Changed |= linearFunctionTestReplace(L, BackedgeTakenCount, IndVar, |
| 2643 | Rewriter); |
| Andrew Trick | 25553ab | 2012-03-24 00:51:17 +0000 | [diff] [blame] | 2644 | } |
| Chris Lattner | c1a682d | 2004-04-22 14:59:40 +0000 | [diff] [blame] | 2645 | } |
| Andrew Trick | 87716c9 | 2011-03-17 23:51:11 +0000 | [diff] [blame] | 2646 | // Clear the rewriter cache, because values that are in the rewriter's cache |
| 2647 | // can be deleted in the loop below, causing the AssertingVH in the cache to |
| 2648 | // trigger. |
| 2649 | Rewriter.clear(); |
| 2650 | |
| 2651 | // Now that we're done iterating through lists, clean up any instructions |
| 2652 | // which are now dead. |
| Duncan P. N. Exon Smith | 817ac8f | 2015-06-24 22:23:21 +0000 | [diff] [blame] | 2653 | while (!DeadInsts.empty()) |
| 2654 | if (Instruction *Inst = |
| 2655 | dyn_cast_or_null<Instruction>(DeadInsts.pop_back_val())) |
| Max Kazantsev | 9e6845d | 2018-09-07 07:23:39 +0000 | [diff] [blame] | 2656 | Changed |= RecursivelyDeleteTriviallyDeadInstructions(Inst, TLI); |
| Andrew Trick | 87716c9 | 2011-03-17 23:51:11 +0000 | [diff] [blame] | 2657 | |
| Dan Gohman | daafbe6 | 2009-06-26 22:53:46 +0000 | [diff] [blame] | 2658 | // The Rewriter may not be used from this point on. |
| Torok Edwin | 26895b5 | 2009-05-24 20:08:21 +0000 | [diff] [blame] | 2659 | |
| Dan Gohman | d76d71a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 2660 | // Loop-invariant instructions in the preheader that aren't used in the |
| 2661 | // loop may be sunk below the loop to reduce register pressure. |
| Max Kazantsev | 4d10ba3 | 2018-09-10 06:32:00 +0000 | [diff] [blame] | 2662 | Changed |= sinkUnusedInvariants(L); |
| Dan Gohman | d76d71a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 2663 | |
| Chen Li | 5cde838 | 2016-01-27 07:40:41 +0000 | [diff] [blame] | 2664 | // rewriteFirstIterationLoopExitValues does not rely on the computation of |
| 2665 | // trip count and therefore can further simplify exit values in addition to |
| 2666 | // rewriteLoopExitValues. |
| Max Kazantsev | fde8857 | 2018-09-10 06:50:16 +0000 | [diff] [blame] | 2667 | Changed |= rewriteFirstIterationLoopExitValues(L); |
| Chen Li | 5cde838 | 2016-01-27 07:40:41 +0000 | [diff] [blame] | 2668 | |
| Dan Gohman | d76d71a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 2669 | // Clean up dead instructions. |
| Benjamin Kramer | 8bcc971 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 2670 | Changed |= DeleteDeadPHIs(L->getHeader(), TLI); |
| Sanjoy Das | 683bf07 | 2015-12-08 00:13:21 +0000 | [diff] [blame] | 2671 | |
| Dan Gohman | d76d71a | 2009-05-12 02:17:14 +0000 | [diff] [blame] | 2672 | // Check a post-condition. |
| Igor Laevsky | 04423cf | 2016-10-11 13:37:22 +0000 | [diff] [blame] | 2673 | assert(L->isRecursivelyLCSSAForm(*DT, *LI) && |
| 2674 | "Indvars did not preserve LCSSA!"); |
| Andrew Trick | 494c549 | 2011-07-18 18:44:20 +0000 | [diff] [blame] | 2675 | |
| 2676 | // Verify that LFTR, and any other change have not interfered with SCEV's |
| 2677 | // ability to compute trip count. |
| 2678 | #ifndef NDEBUG |
| Andrew Trick | f47d0af | 2012-03-22 17:10:11 +0000 | [diff] [blame] | 2679 | if (VerifyIndvars && !isa<SCEVCouldNotCompute>(BackedgeTakenCount)) { |
| Andrew Trick | 494c549 | 2011-07-18 18:44:20 +0000 | [diff] [blame] | 2680 | SE->forgetLoop(L); |
| 2681 | const SCEV *NewBECount = SE->getBackedgeTakenCount(L); |
| 2682 | if (SE->getTypeSizeInBits(BackedgeTakenCount->getType()) < |
| 2683 | SE->getTypeSizeInBits(NewBECount->getType())) |
| 2684 | NewBECount = SE->getTruncateOrNoop(NewBECount, |
| 2685 | BackedgeTakenCount->getType()); |
| 2686 | else |
| 2687 | BackedgeTakenCount = SE->getTruncateOrNoop(BackedgeTakenCount, |
| 2688 | NewBECount->getType()); |
| 2689 | assert(BackedgeTakenCount == NewBECount && "indvars must preserve SCEV"); |
| 2690 | } |
| 2691 | #endif |
| 2692 | |
| Devang Patel | 2ac57e1 | 2007-03-07 06:39:01 +0000 | [diff] [blame] | 2693 | return Changed; |
| Chris Lattner | 476e6df | 2001-12-03 17:28:42 +0000 | [diff] [blame] | 2694 | } |
| Sanjoy Das | 496f274 | 2016-05-29 21:42:00 +0000 | [diff] [blame] | 2695 | |
| Chandler Carruth | 410eaeb | 2017-01-11 06:23:21 +0000 | [diff] [blame] | 2696 | PreservedAnalyses IndVarSimplifyPass::run(Loop &L, LoopAnalysisManager &AM, |
| 2697 | LoopStandardAnalysisResults &AR, |
| 2698 | LPMUpdater &) { |
| Sanjoy Das | 4d4339d | 2016-06-05 18:01:19 +0000 | [diff] [blame] | 2699 | Function *F = L.getHeader()->getParent(); |
| 2700 | const DataLayout &DL = F->getParent()->getDataLayout(); |
| 2701 | |
| Chandler Carruth | 410eaeb | 2017-01-11 06:23:21 +0000 | [diff] [blame] | 2702 | IndVarSimplify IVS(&AR.LI, &AR.SE, &AR.DT, DL, &AR.TLI, &AR.TTI); |
| Sanjoy Das | 4d4339d | 2016-06-05 18:01:19 +0000 | [diff] [blame] | 2703 | if (!IVS.run(&L)) |
| 2704 | return PreservedAnalyses::all(); |
| 2705 | |
| Chandler Carruth | ca68a3e | 2017-01-15 06:32:49 +0000 | [diff] [blame] | 2706 | auto PA = getLoopPassPreservedAnalyses(); |
| 2707 | PA.preserveSet<CFGAnalyses>(); |
| 2708 | return PA; |
| Sanjoy Das | 4d4339d | 2016-06-05 18:01:19 +0000 | [diff] [blame] | 2709 | } |
| 2710 | |
| Sanjoy Das | 496f274 | 2016-05-29 21:42:00 +0000 | [diff] [blame] | 2711 | namespace { |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 2712 | |
| Sanjoy Das | 496f274 | 2016-05-29 21:42:00 +0000 | [diff] [blame] | 2713 | struct IndVarSimplifyLegacyPass : public LoopPass { |
| 2714 | static char ID; // Pass identification, replacement for typeid |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 2715 | |
| Sanjoy Das | 496f274 | 2016-05-29 21:42:00 +0000 | [diff] [blame] | 2716 | IndVarSimplifyLegacyPass() : LoopPass(ID) { |
| 2717 | initializeIndVarSimplifyLegacyPassPass(*PassRegistry::getPassRegistry()); |
| 2718 | } |
| 2719 | |
| 2720 | bool runOnLoop(Loop *L, LPPassManager &LPM) override { |
| 2721 | if (skipLoop(L)) |
| 2722 | return false; |
| 2723 | |
| 2724 | auto *LI = &getAnalysis<LoopInfoWrapperPass>().getLoopInfo(); |
| 2725 | auto *SE = &getAnalysis<ScalarEvolutionWrapperPass>().getSE(); |
| 2726 | auto *DT = &getAnalysis<DominatorTreeWrapperPass>().getDomTree(); |
| 2727 | auto *TLIP = getAnalysisIfAvailable<TargetLibraryInfoWrapperPass>(); |
| 2728 | auto *TLI = TLIP ? &TLIP->getTLI() : nullptr; |
| 2729 | auto *TTIP = getAnalysisIfAvailable<TargetTransformInfoWrapperPass>(); |
| 2730 | auto *TTI = TTIP ? &TTIP->getTTI(*L->getHeader()->getParent()) : nullptr; |
| 2731 | const DataLayout &DL = L->getHeader()->getModule()->getDataLayout(); |
| 2732 | |
| 2733 | IndVarSimplify IVS(LI, SE, DT, DL, TLI, TTI); |
| 2734 | return IVS.run(L); |
| 2735 | } |
| 2736 | |
| 2737 | void getAnalysisUsage(AnalysisUsage &AU) const override { |
| 2738 | AU.setPreservesCFG(); |
| 2739 | getLoopAnalysisUsage(AU); |
| 2740 | } |
| 2741 | }; |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 2742 | |
| 2743 | } // end anonymous namespace |
| Sanjoy Das | 496f274 | 2016-05-29 21:42:00 +0000 | [diff] [blame] | 2744 | |
| 2745 | char IndVarSimplifyLegacyPass::ID = 0; |
| Eugene Zelenko | dd40f5e | 2017-10-16 21:34:24 +0000 | [diff] [blame] | 2746 | |
| Sanjoy Das | 496f274 | 2016-05-29 21:42:00 +0000 | [diff] [blame] | 2747 | INITIALIZE_PASS_BEGIN(IndVarSimplifyLegacyPass, "indvars", |
| 2748 | "Induction Variable Simplification", false, false) |
| 2749 | INITIALIZE_PASS_DEPENDENCY(LoopPass) |
| 2750 | INITIALIZE_PASS_END(IndVarSimplifyLegacyPass, "indvars", |
| 2751 | "Induction Variable Simplification", false, false) |
| 2752 | |
| 2753 | Pass *llvm::createIndVarSimplifyPass() { |
| 2754 | return new IndVarSimplifyLegacyPass(); |
| 2755 | } |