[llvm] [Transforms] Avoid repeated hash lookups (NFC) (PR #112654)
via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 16 20:34:30 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-transforms
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/112654.diff
1 Files Affected:
- (modified) llvm/lib/Transforms/Utils/LoopPeel.cpp (+4-6)
``````````diff
diff --git a/llvm/lib/Transforms/Utils/LoopPeel.cpp b/llvm/lib/Transforms/Utils/LoopPeel.cpp
index 760f1619e030c3..3cbde39b30b4e4 100644
--- a/llvm/lib/Transforms/Utils/LoopPeel.cpp
+++ b/llvm/lib/Transforms/Utils/LoopPeel.cpp
@@ -206,13 +206,11 @@ PhiAnalyzer::PhiAnalyzer(const Loop &L, unsigned MaxIterations)
// G(%y) = Unknown otherwise (including phi not in header block)
PhiAnalyzer::PeelCounter PhiAnalyzer::calculate(const Value &V) {
// If we already know the answer, take it from the map.
- auto I = IterationsToInvariance.find(&V);
- if (I != IterationsToInvariance.end())
- return I->second;
-
- // Place Unknown to map to avoid infinite recursion. Such
+ // Otherwise, place Unknown to map to avoid infinite recursion. Such
// cycles can never stop on an invariant.
- IterationsToInvariance[&V] = Unknown;
+ auto [I, Inserted] = IterationsToInvariance.try_emplace(&V, Unknown);
+ if (!Inserted)
+ return I->second;
if (L.isLoopInvariant(&V))
// Loop invariant so known at start.
``````````
</details>
https://github.com/llvm/llvm-project/pull/112654
More information about the llvm-commits
mailing list