[llvm] LAA: use make_early_inc_range to simplify code (NFC) (PR #125488)
via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 3 04:20:12 PST 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-analysis
Author: Ramkumar Ramachandra (artagnon)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/125488.diff
1 Files Affected:
- (modified) llvm/lib/Analysis/LoopAccessAnalysis.cpp (+2-6)
``````````diff
diff --git a/llvm/lib/Analysis/LoopAccessAnalysis.cpp b/llvm/lib/Analysis/LoopAccessAnalysis.cpp
index ac8a35fbc54fb1d..7e4358dc62b1e08 100644
--- a/llvm/lib/Analysis/LoopAccessAnalysis.cpp
+++ b/llvm/lib/Analysis/LoopAccessAnalysis.cpp
@@ -3078,20 +3078,16 @@ const LoopAccessInfo &LoopAccessInfoManager::getInfo(Loop &L) {
return *It->second;
}
void LoopAccessInfoManager::clear() {
- SmallVector<Loop *> ToRemove;
// Collect LoopAccessInfo entries that may keep references to IR outside the
// analyzed loop or SCEVs that may have been modified or invalidated. At the
// moment, that is loops requiring memory or SCEV runtime checks, as those cache
// SCEVs, e.g. for pointer expressions.
- for (const auto &[L, LAI] : LoopAccessInfoMap) {
+ for (const auto &[L, LAI] : make_early_inc_range(LoopAccessInfoMap)) {
if (LAI->getRuntimePointerChecking()->getChecks().empty() &&
LAI->getPSE().getPredicate().isAlwaysTrue())
continue;
- ToRemove.push_back(L);
- }
-
- for (Loop *L : ToRemove)
LoopAccessInfoMap.erase(L);
+ }
}
bool LoopAccessInfoManager::invalidate(
``````````
</details>
https://github.com/llvm/llvm-project/pull/125488
More information about the llvm-commits
mailing list