[llvm] [Analysis] Avoid repeated hash lookups (NFC) (PR #131066)

via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 12 20:55:42 PDT 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-llvm-analysis

Author: Kazu Hirata (kazutakahirata)

<details>
<summary>Changes</summary>



---
Full diff: https://github.com/llvm/llvm-project/pull/131066.diff


1 Files Affected:

- (modified) llvm/lib/Analysis/MemoryDependenceAnalysis.cpp (+4-2) 


``````````diff
diff --git a/llvm/lib/Analysis/MemoryDependenceAnalysis.cpp b/llvm/lib/Analysis/MemoryDependenceAnalysis.cpp
index dec5ee5361ca3..8550aaacb4fc9 100644
--- a/llvm/lib/Analysis/MemoryDependenceAnalysis.cpp
+++ b/llvm/lib/Analysis/MemoryDependenceAnalysis.cpp
@@ -1620,10 +1620,12 @@ void MemoryDependenceResults::removeInstruction(Instruction *RemInst) {
       assert(P.getPointer() != RemInst &&
              "Already removed NonLocalPointerDeps info for RemInst");
 
-      NonLocalDepInfo &NLPDI = NonLocalPointerDeps[P].NonLocalDeps;
+      auto &NLPD = NonLocalPointerDeps[P];
+
+      NonLocalDepInfo &NLPDI = NLPD.NonLocalDeps;
 
       // The cache is not valid for any specific block anymore.
-      NonLocalPointerDeps[P].Pair = BBSkipFirstBlockPair();
+      NLPD.Pair = BBSkipFirstBlockPair();
 
       // Update any entries for RemInst to use the instruction after it.
       for (auto &Entry : NLPDI) {

``````````

</details>


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


More information about the llvm-commits mailing list