[clang] [clang] Use *(Set|Map)::contains (NFC) (PR #138464)
Kazu Hirata via cfe-commits
cfe-commits at lists.llvm.org
Sun May 4 11:02:35 PDT 2025
https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/138464
None
>From 988dae3f61b85a357f271336695619060103aeca Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Sun, 4 May 2025 10:41:49 -0700
Subject: [PATCH] [clang] Use *(Set|Map)::contains (NFC)
---
clang/lib/AST/DeclBase.cpp | 3 +--
clang/lib/Serialization/ASTReader.cpp | 5 ++---
clang/tools/libclang/CXCursor.cpp | 2 +-
clang/utils/TableGen/ClangBuiltinTemplatesEmitter.cpp | 4 +---
4 files changed, 5 insertions(+), 9 deletions(-)
diff --git a/clang/lib/AST/DeclBase.cpp b/clang/lib/AST/DeclBase.cpp
index fead99c5f28a9..aea19c51401aa 100644
--- a/clang/lib/AST/DeclBase.cpp
+++ b/clang/lib/AST/DeclBase.cpp
@@ -2149,8 +2149,7 @@ void DeclContext::makeDeclVisibleInContextImpl(NamedDecl *D, bool Internal) {
// have already checked the external source.
if (!Internal)
if (ExternalASTSource *Source = getParentASTContext().getExternalSource())
- if (hasExternalVisibleStorage() &&
- Map->find(D->getDeclName()) == Map->end())
+ if (hasExternalVisibleStorage() && !Map->contains(D->getDeclName()))
Source->FindExternalVisibleDeclsByName(this, D->getDeclName(),
D->getDeclContext());
diff --git a/clang/lib/Serialization/ASTReader.cpp b/clang/lib/Serialization/ASTReader.cpp
index 106a652049b07..e47bac0261356 100644
--- a/clang/lib/Serialization/ASTReader.cpp
+++ b/clang/lib/Serialization/ASTReader.cpp
@@ -8646,9 +8646,8 @@ ASTReader::getLoadedSpecializationsLookupTables(const Decl *D, bool IsPartial) {
bool ASTReader::haveUnloadedSpecializations(const Decl *D) const {
assert(D->isCanonicalDecl());
- return (PartialSpecializationsLookups.find(D) !=
- PartialSpecializationsLookups.end()) ||
- (SpecializationsLookups.find(D) != SpecializationsLookups.end());
+ return PartialSpecializationsLookups.contains(D) ||
+ SpecializationsLookups.contains(D);
}
/// Under non-PCH compilation the consumer receives the objc methods
diff --git a/clang/tools/libclang/CXCursor.cpp b/clang/tools/libclang/CXCursor.cpp
index 1d15120106017..635d03a88d105 100644
--- a/clang/tools/libclang/CXCursor.cpp
+++ b/clang/tools/libclang/CXCursor.cpp
@@ -1638,7 +1638,7 @@ unsigned clang_CXCursorSet_contains(CXCursorSet set, CXCursor cursor) {
CXCursorSet_Impl *setImpl = unpackCXCursorSet(set);
if (!setImpl)
return 0;
- return setImpl->find(cursor) != setImpl->end();
+ return setImpl->contains(cursor);
}
unsigned clang_CXCursorSet_insert(CXCursorSet set, CXCursor cursor) {
diff --git a/clang/utils/TableGen/ClangBuiltinTemplatesEmitter.cpp b/clang/utils/TableGen/ClangBuiltinTemplatesEmitter.cpp
index de79743a06e1f..e2eb65091bc5a 100644
--- a/clang/utils/TableGen/ClangBuiltinTemplatesEmitter.cpp
+++ b/clang/utils/TableGen/ClangBuiltinTemplatesEmitter.cpp
@@ -60,10 +60,8 @@ ParseTemplateParameterList(ParserState &PS,
} else if (Arg->isSubClassOf("NTTP")) {
auto Type = Arg->getValueAsString("TypeName");
- if (TemplateNameToParmName.find(Type.str()) ==
- TemplateNameToParmName.end()) {
+ if (!TemplateNameToParmName.contains(Type.str()))
PrintFatalError("Unkown Type Name");
- }
auto TSIName = "TSI" + std::to_string(PS.UniqueCounter++);
Code << " auto *" << TSIName << " = C.getTrivialTypeSourceInfo(QualType("
More information about the cfe-commits
mailing list