shylie/llvm-project
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[clang-tools-extra] Use llvm::find_if (NFC) (#140411)

Browse Source
This commit is contained in:
Kazu Hirata 2025-05-17 17:01:37 -07:00 committed by GitHub
parent bc6107a8d2
commit 1a40edf2c0
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 15 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
clang-tools-extra/clang-tidy/misc/NewDeleteOverloadsCheck.cpp
View File

@ -168,22 +168,21 @@ void NewDeleteOverloadsCheck::onEndOfTranslationUnit() {
// complexity when searching for corresponding free store functions.
for (const auto *Overload : RP.second) {
const auto *Match =
std::find_if(RP.second.begin(), RP.second.end(),
[&Overload](const FunctionDecl *FD) {
if (FD == Overload)
return false;
// If the declaration contexts don't match, we don't
// need to check any further.
if (FD->getDeclContext() != Overload->getDeclContext())
return false;
llvm::find_if(RP.second, [&Overload](const FunctionDecl *FD) {
if (FD == Overload)
return false;
// If the declaration contexts don't match, we don't
// need to check any further.
if (FD->getDeclContext() != Overload->getDeclContext())
return false;
// Since the declaration contexts match, see whether
// the current element is the corresponding operator.
if (!areCorrespondingOverloads(Overload, FD))
return false;
// Since the declaration contexts match, see whether
// the current element is the corresponding operator.
if (!areCorrespondingOverloads(Overload, FD))
return false;
return true;
});
return true;
});
if (Match == RP.second.end()) {
// Check to see if there is a corresponding overload in a base class

5
clang-tools-extra/clangd/refactor/Rename.cpp
View File

@ -185,9 +185,8 @@ void filterRenameTargets(llvm::DenseSet<const NamedDecl *> &Decls) {
// For renaming, we're only interested in foo's declaration, so drop the other
// one. There should never be more than one UsingDecl here, otherwise the
// rename would be ambiguos anyway.
auto UD = std::find_if(Decls.begin(), Decls.end(), [](const NamedDecl *D) {
return llvm::isa<UsingDecl>(D);
});
auto UD = llvm::find_if(
Decls, [](const NamedDecl *D) { return llvm::isa<UsingDecl>(D); });
if (UD != Decls.end()) {
Decls.erase(UD);
}