[llvm] [InstCombine] Avoid repeated hash lookups (NFC) (PR #123559)
via llvm-commits
llvm-commits at lists.llvm.org
Sun Jan 19 23:55:13 PST 2025
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/123559.diff
1 Files Affected:
- (modified) llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp (+4-3)
``````````diff
diff --git a/llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp b/llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp
index 842881156dc67f..a2730396a0ea26 100644
--- a/llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp
+++ b/llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp
@@ -4179,13 +4179,14 @@ Instruction *InstCombinerImpl::visitCallBase(CallBase &Call) {
DenseMap<Value *, unsigned> Val2Idx;
std::vector<Value *> NewLiveGc;
for (Value *V : Bundle->Inputs) {
- if (Val2Idx.count(V))
+ auto [It, Inserted] = Val2Idx.try_emplace(V);
+ if (!Inserted)
continue;
if (LiveGcValues.count(V)) {
- Val2Idx[V] = NewLiveGc.size();
+ It->second = NewLiveGc.size();
NewLiveGc.push_back(V);
} else
- Val2Idx[V] = NumOfGCLives;
+ It->second = NumOfGCLives;
}
// Update all gc.relocates
for (const GCRelocateInst *Reloc : GCSP.getGCRelocates()) {
``````````
</details>
https://github.com/llvm/llvm-project/pull/123559
More information about the llvm-commits
mailing list