[llvm] [ARM] Avoid repeated hash lookups (NFC) (PR #111935)
via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 10 20:38:40 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-backend-arm
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/111935.diff
1 Files Affected:
- (modified) llvm/lib/Target/ARM/ARMLoadStoreOptimizer.cpp (+3-4)
``````````diff
diff --git a/llvm/lib/Target/ARM/ARMLoadStoreOptimizer.cpp b/llvm/lib/Target/ARM/ARMLoadStoreOptimizer.cpp
index b151a0116a9c41..c7b6b3579120dc 100644
--- a/llvm/lib/Target/ARM/ARMLoadStoreOptimizer.cpp
+++ b/llvm/lib/Target/ARM/ARMLoadStoreOptimizer.cpp
@@ -2532,7 +2532,6 @@ ARMPreAllocLoadStoreOpt::RescheduleLoadStoreInstrs(MachineBasicBlock *MBB) {
bool RetVal = false;
DenseMap<MachineInstr *, unsigned> MI2LocMap;
- using MapIt = DenseMap<unsigned, SmallVector<MachineInstr *, 4>>::iterator;
using Base2InstMap = DenseMap<unsigned, SmallVector<MachineInstr *, 4>>;
using BaseVec = SmallVector<unsigned, 4>;
Base2InstMap Base2LdsMap;
@@ -2571,9 +2570,9 @@ ARMPreAllocLoadStoreOpt::RescheduleLoadStoreInstrs(MachineBasicBlock *MBB) {
int Offset = getMemoryOpOffset(MI);
bool StopHere = false;
auto FindBases = [&](Base2InstMap &Base2Ops, BaseVec &Bases) {
- MapIt BI = Base2Ops.find(Base);
- if (BI == Base2Ops.end()) {
- Base2Ops[Base].push_back(&MI);
+ auto [BI, Inserted] = Base2Ops.try_emplace(Base);
+ if (Inserted) {
+ BI->second.push_back(&MI);
Bases.push_back(Base);
return;
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/111935
More information about the llvm-commits
mailing list