[llvm] [llvm] Use llvm::is_contained (NFC) (PR #101855)
via llvm-commits
llvm-commits at lists.llvm.org
Sat Aug 3 17:17:18 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-ir
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/101855.diff
4 Files Affected:
- (modified) llvm/lib/IR/Verifier.cpp (+1-4)
- (modified) llvm/lib/Transforms/Utils/LoopSimplify.cpp (+2-4)
- (modified) llvm/lib/Transforms/Utils/ValueMapper.cpp (+1-2)
- (modified) llvm/lib/Transforms/Vectorize/VectorCombine.cpp (+1-2)
``````````diff
diff --git a/llvm/lib/IR/Verifier.cpp b/llvm/lib/IR/Verifier.cpp
index c5c407637cbf3..cb4eaf80d91ee 100644
--- a/llvm/lib/IR/Verifier.cpp
+++ b/llvm/lib/IR/Verifier.cpp
@@ -6229,10 +6229,7 @@ void Verifier::visitIntrinsicCall(Intrinsic::ID ID, CallBase &Call) {
&Call);
break;
}
- Check(llvm::any_of(CBR->getIndirectDests(),
- [LandingPadBB](const BasicBlock *IndDest) {
- return IndDest == LandingPadBB;
- }),
+ Check(llvm::is_contained(CBR->getIndirectDests(), LandingPadBB),
"Intrinsic's corresponding callbr must have intrinsic's parent basic "
"block in indirect destination list",
&Call);
diff --git a/llvm/lib/Transforms/Utils/LoopSimplify.cpp b/llvm/lib/Transforms/Utils/LoopSimplify.cpp
index 2b2f82fb47cf7..5e69923fd989d 100644
--- a/llvm/lib/Transforms/Utils/LoopSimplify.cpp
+++ b/llvm/lib/Transforms/Utils/LoopSimplify.cpp
@@ -83,10 +83,8 @@ static void placeSplitBlockCarefully(BasicBlock *NewBB,
Loop *L) {
// Check to see if NewBB is already well placed.
Function::iterator BBI = --NewBB->getIterator();
- for (BasicBlock *Pred : SplitPreds) {
- if (&*BBI == Pred)
- return;
- }
+ if (llvm::is_contained(SplitPreds, &*BBI))
+ return;
// If it isn't already after an outside block, move it after one. This is
// always good as it makes the uncond branch from the outside block into a
diff --git a/llvm/lib/Transforms/Utils/ValueMapper.cpp b/llvm/lib/Transforms/Utils/ValueMapper.cpp
index 5af56b01cb69c..56e0eca7cea56 100644
--- a/llvm/lib/Transforms/Utils/ValueMapper.cpp
+++ b/llvm/lib/Transforms/Utils/ValueMapper.cpp
@@ -575,8 +575,7 @@ void Mapper::remapDbgRecord(DbgRecord &DR) {
return;
// Otherwise, do some replacement.
- if (!IgnoreMissingLocals &&
- llvm::any_of(NewVals, [&](Value *V) { return V == nullptr; })) {
+ if (!IgnoreMissingLocals && llvm::is_contained(NewVals, nullptr)) {
V.setKillLocation();
} else {
// Either we have all non-empty NewVals, or we're permitted to ignore
diff --git a/llvm/lib/Transforms/Vectorize/VectorCombine.cpp b/llvm/lib/Transforms/Vectorize/VectorCombine.cpp
index d81c747684f07..99bd383ab0dea 100644
--- a/llvm/lib/Transforms/Vectorize/VectorCombine.cpp
+++ b/llvm/lib/Transforms/Vectorize/VectorCombine.cpp
@@ -1406,8 +1406,7 @@ bool VectorCombine::foldShuffleOfBinops(Instruction &I) {
return false;
// Don't introduce poison into div/rem.
- if (any_of(OldMask, [](int M) { return M == PoisonMaskElem; }) &&
- B0->isIntDivRem())
+ if (llvm::is_contained(OldMask, PoisonMaskElem) && B0->isIntDivRem())
return false;
// TODO: Add support for addlike etc.
``````````
</details>
https://github.com/llvm/llvm-project/pull/101855
More information about the llvm-commits
mailing list