[llvm] [Vectorize] Simplify code with MapVector::operator[] (NFC) (PR #115592)
via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 8 23:33:11 PST 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-transforms
@llvm/pr-subscribers-vectorizers
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/115592.diff
1 Files Affected:
- (modified) llvm/lib/Transforms/Vectorize/LoopVectorize.cpp (+4-4)
``````````diff
diff --git a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
index 82d85b112397f3..1ebc62f9843905 100644
--- a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
+++ b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
@@ -6858,8 +6858,8 @@ void LoopVectorizationCostModel::collectValuesToIgnore() {
if ((SI = dyn_cast<StoreInst>(&I)) &&
Legal->isInvariantAddressOfReduction(SI->getPointerOperand())) {
ValuesToIgnore.insert(&I);
- auto I = DeadInvariantStoreOps.insert({SI->getPointerOperand(), {}});
- I.first->second.push_back(SI->getValueOperand());
+ DeadInvariantStoreOps[SI->getPointerOperand()].push_back(
+ SI->getValueOperand());
}
if (VecValuesToIgnore.contains(&I) || ValuesToIgnore.contains(&I))
@@ -8084,9 +8084,9 @@ void VPRecipeBuilder::createSwitchEdgeMasks(SwitchInst *SI) {
// ignored - they will get there anyhow.
if (Dst == DefaultDst)
continue;
- auto I = Dst2Compares.insert({Dst, {}});
+ auto &Compares = Dst2Compares[Dst];
VPValue *V = getVPValueOrAddLiveIn(C.getCaseValue());
- I.first->second.push_back(Builder.createICmp(CmpInst::ICMP_EQ, Cond, V));
+ Compares.push_back(Builder.createICmp(CmpInst::ICMP_EQ, Cond, V));
}
// We need to handle 2 separate cases below for all entries in Dst2Compares,
``````````
</details>
https://github.com/llvm/llvm-project/pull/115592
More information about the llvm-commits
mailing list