[llvm] [LoopPeel] Implement initial peeling off the last loop iteration. (PR #139551)

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Wed May 14 07:36:11 PDT 2025


================
@@ -325,19 +326,71 @@ static unsigned peelToTurnInvariantLoadsDerefencebale(Loop &L,
   return 0;
 }
 
-// Return the number of iterations to peel off that make conditions in the
-// body true/false. For example, if we peel 2 iterations off the loop below,
-// the condition i < 2 can be evaluated at compile time.
+bool llvm::canPeelLastIteration(const Loop &L, ScalarEvolution &SE) {
+  const SCEV *BTC = SE.getBackedgeTakenCount(&L);
+  Value *Inc;
+  CmpPredicate Pred;
+  BasicBlock *Succ1;
+  BasicBlock *Succ2;
+  // The loop must execute at least 2 iterations to guarantee that peeled
+  // iteration executes.
+  // TODO: Add checks during codegen.
+  if (isa<SCEVCouldNotCompute>(BTC) ||
+      !SE.isKnownPredicate(CmpInst::ICMP_UGT, BTC, SE.getOne(BTC->getType())))
+    return false;
+
+  // Check if the exit condition of the loop can be adjusted by the peeling
+  // codegen. For now, it must
+  // * exit via the latch,
+  // * the exit condition must be a NE/EQ compare of an induction with step
+  // of 1.
+  BasicBlock *Latch = L.getLoopLatch();
+  return Latch == L.getExitingBlock() &&
----------------
preames wrote:

Oh, you're missing a null check on Latch.  If the loop has multiple exits and multiple latches, this will crash on the Latch->getTerminator() check.  (Note that we aggressively canonicalize away from multiple backedge loops, so this is probably hard to exercise in practice.)

https://github.com/llvm/llvm-project/pull/139551


More information about the llvm-commits mailing list