[llvm] [ARM] Avoid repeated map lookups (NFC) (PR #131420)
via llvm-commits
llvm-commits at lists.llvm.org
Fri Mar 14 19:35:30 PDT 2025
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/131420.diff
1 Files Affected:
- (modified) llvm/lib/Target/ARM/ARMParallelDSP.cpp (+8-4)
``````````diff
diff --git a/llvm/lib/Target/ARM/ARMParallelDSP.cpp b/llvm/lib/Target/ARM/ARMParallelDSP.cpp
index 4e92720de4755..e8bbd15c1ca03 100644
--- a/llvm/lib/Target/ARM/ARMParallelDSP.cpp
+++ b/llvm/lib/Target/ARM/ARMParallelDSP.cpp
@@ -715,10 +715,14 @@ void ARMParallelDSP::InsertParallelMACs(Reduction &R) {
MulCandidate *RHSMul = Pair.second;
LoadInst *BaseLHS = LHSMul->getBaseLoad();
LoadInst *BaseRHS = RHSMul->getBaseLoad();
- LoadInst *WideLHS = WideLoads.count(BaseLHS) ?
- WideLoads[BaseLHS]->getLoad() : CreateWideLoad(LHSMul->VecLd, Ty);
- LoadInst *WideRHS = WideLoads.count(BaseRHS) ?
- WideLoads[BaseRHS]->getLoad() : CreateWideLoad(RHSMul->VecLd, Ty);
+ auto LIt = WideLoads.find(BaseLHS);
+ LoadInst *WideLHS = LIt != WideLoads.end()
+ ? LIt->second->getLoad()
+ : CreateWideLoad(LHSMul->VecLd, Ty);
+ auto RIt = WideLoads.find(BaseRHS);
+ LoadInst *WideRHS = RIt != WideLoads.end()
+ ? RIt->second->getLoad()
+ : CreateWideLoad(RHSMul->VecLd, Ty);
Instruction *InsertAfter = GetInsertPoint(WideLHS, WideRHS);
InsertAfter = GetInsertPoint(InsertAfter, Acc);
``````````
</details>
https://github.com/llvm/llvm-project/pull/131420
More information about the llvm-commits
mailing list