[llvm] [Transforms] Use a range-based for loop (NFC) (PR #105769)

Kazu Hirata via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 22 19:09:52 PDT 2024


https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/105769

None

>From 38a0a7ac012e6da87cd251e28dd39da50b72bc0b Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Fri, 16 Aug 2024 08:48:26 -0700
Subject: [PATCH] [Transforms] Use a range-based for loop (NFC)

---
 llvm/lib/Transforms/Scalar/PlaceSafepoints.cpp | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/llvm/lib/Transforms/Scalar/PlaceSafepoints.cpp b/llvm/lib/Transforms/Scalar/PlaceSafepoints.cpp
index 77d67a2ce0f380..1f9c3837cc06dc 100644
--- a/llvm/lib/Transforms/Scalar/PlaceSafepoints.cpp
+++ b/llvm/lib/Transforms/Scalar/PlaceSafepoints.cpp
@@ -331,12 +331,9 @@ bool PlaceSafepointsPass::runImpl(Function &F, const TargetLibraryInfo &TLI) {
         // and b) edges to distinct loop headers.  We need to insert pools on
         // each.
         SetVector<BasicBlock *> Headers;
-        for (unsigned i = 0; i < Term->getNumSuccessors(); i++) {
-          BasicBlock *Succ = Term->getSuccessor(i);
-          if (DT.dominates(Succ, Term->getParent())) {
+        for (BasicBlock *Succ : successors(Term->getParent()))
+          if (DT.dominates(Succ, Term->getParent()))
             Headers.insert(Succ);
-          }
-        }
         assert(!Headers.empty() && "poll location is not a loop latch?");
 
         // The split loop structure here is so that we only need to recalculate



More information about the llvm-commits mailing list