[llvm] [Delinearization] Modernize loops (NFC) (PR #146151)

Ryotaro Kasuga via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 30 02:52:53 PDT 2025


================
@@ -349,24 +349,22 @@ void llvm::computeAccessFunctions(ScalarEvolution &SE, const SCEV *Expr,
       return;
 
   const SCEV *Res = Expr;
-  int Last = Sizes.size() - 1;
-  for (int i = Last; i >= 0; i--) {
+  for (const auto &[Idx, Sz] : enumerate(reverse(Sizes))) {
     const SCEV *Q, *R;
-    SCEVDivision::divide(SE, Res, Sizes[i], &Q, &R);
+    SCEVDivision::divide(SE, Res, Sz, &Q, &R);
 
     LLVM_DEBUG({
       dbgs() << "Res: " << *Res << "\n";
-      dbgs() << "Sizes[i]: " << *Sizes[i] << "\n";
+      dbgs() << "Sizes[i]: " << *Sz << "\n";
----------------
kasuga-fj wrote:

Can you keep the variable name as it is? This is no longer `Sizes[i]`.

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


More information about the llvm-commits mailing list