github-actions[bot] wrote:
<!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
git-clang-format --diff origin/main HEAD --extensions cpp,h --
clang-tools-extra/clang-tidy/misc/DefinitionsInHeadersCheck.cpp
clang-tools-extra/clangd/SemanticHighlighting.cpp
clang-tools-extra/clangd/refactor/tweaks/DefineInline.cpp
clang/include/clang/AST/Decl.h clang/include/clang/AST/DeclTemplate.h
clang/include/clang/AST/JSONNodeDumper.h
clang/include/clang/AST/RecursiveASTVisitor.h
clang/include/clang/ASTMatchers/ASTMatchers.h
clang/include/clang/ASTMatchers/ASTMatchersInternal.h
clang/include/clang/Basic/Specifiers.h clang/include/clang/Sema/Sema.h
clang/lib/AST/ASTContext.cpp clang/lib/AST/ASTDumper.cpp
clang/lib/AST/ASTImporter.cpp clang/lib/AST/Comment.cpp clang/lib/AST/Decl.cpp
clang/lib/AST/DeclPrinter.cpp clang/lib/AST/DeclTemplate.cpp
clang/lib/AST/JSONNodeDumper.cpp clang/lib/AST/TextNodeDumper.cpp
clang/lib/ASTMatchers/Dynamic/Registry.cpp
clang/lib/Analysis/ExprMutationAnalyzer.cpp
clang/lib/CIR/CodeGen/CIRGenVTables.cpp clang/lib/CodeGen/CGVTables.cpp
clang/lib/Index/IndexingContext.cpp clang/lib/InstallAPI/Visitor.cpp
clang/lib/Parse/ParseDeclCXX.cpp clang/lib/Sema/HLSLExternalSemaSource.cpp
clang/lib/Sema/SemaConcept.cpp clang/lib/Sema/SemaDecl.cpp
clang/lib/Sema/SemaDeclCXX.cpp clang/lib/Sema/SemaExprMember.cpp
clang/lib/Sema/SemaOverload.cpp clang/lib/Sema/SemaTemplate.cpp
clang/lib/Sema/SemaTemplateDeduction.cpp
clang/lib/Sema/SemaTemplateDeductionGuide.cpp
clang/lib/Sema/SemaTemplateInstantiate.cpp
clang/lib/Sema/SemaTemplateInstantiateDecl.cpp
clang/lib/Serialization/ASTReaderDecl.cpp
clang/lib/Serialization/ASTWriterDecl.cpp
clang/lib/StaticAnalyzer/Core/BugSuppression.cpp
clang/lib/Tooling/Syntax/BuildTree.cpp
clang/test/AST/ast-dump-templates-pattern.cpp
clang/test/CXX/basic/basic.link/p11.cpp
clang/test/CXX/temp/temp.arg/temp.arg.template/p3-2a.cpp
clang/test/CXX/temp/temp.constr/temp.constr.decl/p4.cpp
clang/test/CXX/temp/temp.decls/temp.spec.partial/temp.spec.partial.member/p2.cpp
clang/test/CXX/temp/temp.spec/temp.expl.spec/p7.cpp
clang/test/CodeGenCXX/default-arguments.cpp
clang/test/CodeGenCXX/explicit-instantiation.cpp
clang/test/SemaTemplate/concepts-out-of-line-def.cpp
clang/test/SemaTemplate/friend-template.cpp
clang/test/SemaTemplate/instantiate-scope.cpp
clang/test/Templight/templight-default-func-arg.cpp
clang/test/Templight/templight-empty-entries-fix.cpp
clang/tools/libclang/CIndex.cpp clang/unittests/AST/ASTImporterTest.cpp
clang/unittests/ASTMatchers/ASTMatchersTraversalTest.cpp
lldb/source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp
lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp
--diff_from_common_commit
``````````
:warning:
The reproduction instructions above might return results for more than one PR
in a stack if you are using a stacked PR workflow. You can limit the results by
changing `origin/main` to the base branch/commit you want to compare against.
:warning:
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/clang/include/clang/AST/DeclTemplate.h
b/clang/include/clang/AST/DeclTemplate.h
index 4f5a4e1b7..f4d608e47 100644
--- a/clang/include/clang/AST/DeclTemplate.h
+++ b/clang/include/clang/AST/DeclTemplate.h
@@ -2105,7 +2105,7 @@ public:
};
class ClassTemplatePartialSpecializationDecl
- : public ClassTemplateSpecializationDecl {
+ : public ClassTemplateSpecializationDecl {
/// The list of template parameters
TemplateParameterList *TemplateParams = nullptr;
diff --git a/clang/lib/CodeGen/CGVTables.cpp b/clang/lib/CodeGen/CGVTables.cpp
index 64fdc0150..53f0e4d74 100644
--- a/clang/lib/CodeGen/CGVTables.cpp
+++ b/clang/lib/CodeGen/CGVTables.cpp
@@ -1143,21 +1143,21 @@ CodeGenModule::getVTableLinkage(const CXXRecordDecl
*RD) {
return llvm::GlobalVariable::ExternalLinkage;
- case TSK_ImplicitInstantiation:
- return !Context.getLangOpts().AppleKext ?
- llvm::GlobalVariable::LinkOnceODRLinkage :
- llvm::Function::InternalLinkage;
-
- case TSK_ExplicitInstantiationDefinition:
- return !Context.getLangOpts().AppleKext ?
- llvm::GlobalVariable::WeakODRLinkage :
- llvm::Function::InternalLinkage;
-
- case TSK_ExplicitInstantiationDeclaration:
- return IsExternalDefinition
- ? llvm::GlobalVariable::AvailableExternallyLinkage
- : llvm::GlobalVariable::ExternalLinkage;
- }
+ case TSK_ImplicitInstantiation:
+ return !Context.getLangOpts().AppleKext
+ ? llvm::GlobalVariable::LinkOnceODRLinkage
+ : llvm::Function::InternalLinkage;
+
+ case TSK_ExplicitInstantiationDefinition:
+ return !Context.getLangOpts().AppleKext
+ ? llvm::GlobalVariable::WeakODRLinkage
+ : llvm::Function::InternalLinkage;
+
+ case TSK_ExplicitInstantiationDeclaration:
+ return IsExternalDefinition
+ ? llvm::GlobalVariable::AvailableExternallyLinkage
+ : llvm::GlobalVariable::ExternalLinkage;
+ }
}
// -fapple-kext mode does not support weak linkage, so we must use
diff --git a/clang/lib/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp
index cddcf3a01..72b69040b 100644
--- a/clang/lib/Sema/SemaDecl.cpp
+++ b/clang/lib/Sema/SemaDecl.cpp
@@ -10245,9 +10245,9 @@ Sema::ActOnFunctionDeclarator(Scope *S, Declarator &D,
DeclContext *DC,
// For source fidelity, store the other template param lists.
if (TemplateParamLists.size() > 1) {
- NewFD->setTemplateParameterListsInfo(Context,
- ArrayRef<TemplateParameterList *>(TemplateParamLists)
- .drop_back(1));
+ NewFD->setTemplateParameterListsInfo(
+ Context, ArrayRef<TemplateParameterList *>(TemplateParamLists)
+ .drop_back(1));
}
} else {
// This is a function template specialization.
diff --git a/clang/lib/Sema/SemaDeclCXX.cpp b/clang/lib/Sema/SemaDeclCXX.cpp
index 418ff01f3..b21b53f30 100644
--- a/clang/lib/Sema/SemaDeclCXX.cpp
+++ b/clang/lib/Sema/SemaDeclCXX.cpp
@@ -13938,10 +13938,9 @@ Decl *Sema::ActOnAliasDeclaration(Scope *S,
AccessSpecifier AS,
}
if (!Invalid && OldDecl && !OldDecl->isInvalidDecl()) {
- if (TemplateParameterListsAreEqual(TemplateParams,
- OldDecl->getTemplateParameters(),
- /*Complain=*/true,
- TPL_TemplateMatch))
+ if (TemplateParameterListsAreEqual(
+ TemplateParams, OldDecl->getTemplateParameters(),
+ /*Complain=*/true, TPL_TemplateMatch))
OldTemplateParams =
OldDecl->getMostRecentDecl()->getTemplateParameters();
else
diff --git a/clang/lib/Sema/SemaOverload.cpp b/clang/lib/Sema/SemaOverload.cpp
index ecda430c8..fdbfec627 100644
--- a/clang/lib/Sema/SemaOverload.cpp
+++ b/clang/lib/Sema/SemaOverload.cpp
@@ -1326,7 +1326,7 @@ OverloadKind Sema::CheckOverload(Scope *S, FunctionDecl
*New,
LookupResult TemplateSpecResult(LookupResult::Temporary, Old);
TemplateSpecResult.addAllDecls(Old);
if (CheckFunctionTemplateSpecialization(New, nullptr, TemplateSpecResult,
- /*QualifiedFriend*/true)) {
+ /*QualifiedFriend*/ true)) {
New->setInvalidDecl();
return OverloadKind::Overload;
}
diff --git a/clang/lib/Sema/SemaTemplate.cpp b/clang/lib/Sema/SemaTemplate.cpp
index 8c94a1ad3..dbdeedd74 100644
--- a/clang/lib/Sema/SemaTemplate.cpp
+++ b/clang/lib/Sema/SemaTemplate.cpp
@@ -2233,10 +2233,9 @@ DeclResult Sema::CheckClassTemplate(
AddMsStructLayoutForRecord(NewClass);
}
- ClassTemplateDecl *NewTemplate
- = ClassTemplateDecl::Create(Context, SemanticContext, NameLoc,
- DeclarationName(Name), TemplateParams,
- NewClass);
+ ClassTemplateDecl *NewTemplate = ClassTemplateDecl::Create(
+ Context, SemanticContext, NameLoc, DeclarationName(Name), TemplateParams,
+ NewClass);
if (ShouldAddRedecl)
NewTemplate->setPreviousDecl(PrevClassTemplate);
@@ -3100,9 +3099,9 @@ TemplateParameterList
*Sema::MatchTemplateParametersToScopeSpecifier(
if (ParamIdx < ParamLists.size()) {
// Check the template parameter list, if we can.
if (ExpectedTemplateParams &&
- !TemplateParameterListsAreEqual(ParamLists[ParamIdx],
- ExpectedTemplateParams,
- !SuppressDiagnostic,
TPL_TemplateMatch))
+ !TemplateParameterListsAreEqual(
+ ParamLists[ParamIdx], ExpectedTemplateParams,
+ !SuppressDiagnostic, TPL_TemplateMatch))
Invalid = true;
if (!Invalid &&
@@ -9975,12 +9974,12 @@ Sema::CheckMemberSpecialization(NamedDecl *Member,
LookupResult &Previous) {
// Preserve instantiation information.
if (InstantiatedFrom && isa<CXXMethodDecl>(Member)) {
cast<CXXMethodDecl>(Member)->setInstantiationOfMemberFunction(
- cast<CXXMethodDecl>(InstantiatedFrom),
- cast<CXXMethodDecl>(Instantiation)->getTemplateSpecializationKind());
+ cast<CXXMethodDecl>(InstantiatedFrom),
+ cast<CXXMethodDecl>(Instantiation)->getTemplateSpecializationKind());
} else if (InstantiatedFrom && isa<CXXRecordDecl>(Member)) {
cast<CXXRecordDecl>(Member)->setInstantiationOfMemberClass(
- cast<CXXRecordDecl>(InstantiatedFrom),
- cast<CXXRecordDecl>(Instantiation)->getTemplateSpecializationKind());
+ cast<CXXRecordDecl>(InstantiatedFrom),
+ cast<CXXRecordDecl>(Instantiation)->getTemplateSpecializationKind());
}
Previous.clear();
diff --git a/clang/lib/Sema/SemaTemplateInstantiateDecl.cpp
b/clang/lib/Sema/SemaTemplateInstantiateDecl.cpp
index 324d6bf38..b61a2c53b 100644
--- a/clang/lib/Sema/SemaTemplateInstantiateDecl.cpp
+++ b/clang/lib/Sema/SemaTemplateInstantiateDecl.cpp
@@ -49,7 +49,7 @@ static bool isDeclWithinFunction(const Decl *D) {
return false;
}
-template<typename DeclT>
+template <typename DeclT>
static bool SubstQualifier(Sema &SemaRef, const DeclT *OldDecl, DeclT *NewDecl,
const MultiLevelTemplateArgumentList &TemplateArgs)
{
if (!OldDecl->getQualifierLoc())
@@ -64,9 +64,8 @@ static bool SubstQualifier(Sema &SemaRef, const DeclT
*OldDecl, DeclT *NewDecl,
? NewDecl->getLexicalDeclContext()
: OldDecl->getLexicalDeclContext()));
- NestedNameSpecifierLoc NewQualifierLoc
- = SemaRef.SubstNestedNameSpecifierLoc(OldDecl->getQualifierLoc(),
- TemplateArgs);
+ NestedNameSpecifierLoc NewQualifierLoc = SemaRef.SubstNestedNameSpecifierLoc(
+ OldDecl->getQualifierLoc(), TemplateArgs);
if (!NewQualifierLoc)
return true;
@@ -1752,9 +1751,10 @@ Decl *TemplateDeclInstantiator::VisitVarDecl(VarDecl *D)
{
return VisitVarDecl(D, /*InstantiatingVarTemplate=*/false);
}
-Decl *TemplateDeclInstantiator::VisitVarDecl(VarDecl *D,
- bool InstantiatingVarTemplate,
- ArrayRef<BindingDecl*> *Bindings)
{
+Decl *
+TemplateDeclInstantiator::VisitVarDecl(VarDecl *D,
+ bool InstantiatingVarTemplate,
+ ArrayRef<BindingDecl *> *Bindings) {
// Do substitution on the type of the declaration
TypeSourceInfo *TSI = SemaRef.SubstType(
@@ -2752,8 +2752,8 @@ Decl *TemplateDeclInstantiator::VisitFunctionDecl(
ArrayRef<TemplateArgument> Innermost = TemplateArgs.getInnermost();
void *InsertPos = nullptr;
- FunctionDecl *SpecFunc
- = FunctionTemplate->findSpecialization(Innermost, InsertPos);
+ FunctionDecl *SpecFunc =
+ FunctionTemplate->findSpecialization(Innermost, InsertPos);
// If we already have a function template specialization, return it.
if (SpecFunc)
@@ -2915,10 +2915,10 @@ Decl *TemplateDeclInstantiator::VisitFunctionDecl(
Sema::CodeSynthesisContext::BuildingDeductionGuides) {
// Record this function template specialization.
ArrayRef<TemplateArgument> Innermost = TemplateArgs.getInnermost();
- Function->setFunctionTemplateSpecialization(FunctionTemplate,
- TemplateArgumentList::CreateCopy(SemaRef.Context,
- Innermost),
- /*InsertPos=*/nullptr);
+ Function->setFunctionTemplateSpecialization(
+ FunctionTemplate,
+ TemplateArgumentList::CreateCopy(SemaRef.Context, Innermost),
+ /*InsertPos=*/nullptr);
} else if (FunctionRewriteKind == RewriteKind::None) {
if (isFriend && D->isThisDeclarationADefinition()) {
// Do not connect the friend to the template unless it's actually a
@@ -2994,8 +2994,7 @@ Decl *TemplateDeclInstantiator::VisitFunctionDecl(
ExplicitArgs))
return nullptr;
- if (SemaRef.CheckFunctionTemplateSpecialization(Function,
- &ExplicitArgs,
+ if (SemaRef.CheckFunctionTemplateSpecialization(Function, &ExplicitArgs,
Previous))
Function->setInvalidDecl();
@@ -3120,8 +3119,8 @@ Decl *TemplateDeclInstantiator::VisitCXXMethodDecl(
ArrayRef<TemplateArgument> Innermost = TemplateArgs.getInnermost();
void *InsertPos = nullptr;
- FunctionDecl *SpecFunc
- = FunctionTemplate->findSpecialization(Innermost, InsertPos);
+ FunctionDecl *SpecFunc =
+ FunctionTemplate->findSpecialization(Innermost, InsertPos);
// If we already have a function template specialization, return it.
if (SpecFunc)
@@ -3314,10 +3313,10 @@ Decl *TemplateDeclInstantiator::VisitCXXMethodDecl(
} else if (FunctionTemplate) {
// Record this function template specialization.
ArrayRef<TemplateArgument> Innermost = TemplateArgs.getInnermost();
- Method->setFunctionTemplateSpecialization(FunctionTemplate,
- TemplateArgumentList::CreateCopy(SemaRef.Context,
- Innermost),
- /*InsertPos=*/nullptr);
+ Method->setFunctionTemplateSpecialization(
+ FunctionTemplate,
+ TemplateArgumentList::CreateCopy(SemaRef.Context, Innermost),
+ /*InsertPos=*/nullptr);
} else if (!isFriend && FunctionRewriteKind == RewriteKind::None) {
// Record that this is an instantiation of a member function.
Method->setInstantiationOfMemberFunction(D, TSK_ImplicitInstantiation);
@@ -3388,8 +3387,7 @@ Decl *TemplateDeclInstantiator::VisitCXXMethodDecl(
ExplicitArgs))
return nullptr;
- if (SemaRef.CheckFunctionTemplateSpecialization(Method,
- &ExplicitArgs,
+ if (SemaRef.CheckFunctionTemplateSpecialization(Method, &ExplicitArgs,
Previous))
Method->setInvalidDecl();
@@ -6296,7 +6294,8 @@ void Sema::InstantiateVariableInitializer(
void Sema::InstantiateVariableDefinition(SourceLocation PointOfInstantiation,
VarDecl *Var, bool Recursive,
- bool DefinitionRequired, bool AtEndOfTU)
{
+ bool DefinitionRequired,
+ bool AtEndOfTU) {
if (Var->isInvalidDecl())
return;
diff --git a/clang/lib/Serialization/ASTReaderDecl.cpp
b/clang/lib/Serialization/ASTReaderDecl.cpp
index fb291a4b0..a3c2a301c 100644
--- a/clang/lib/Serialization/ASTReaderDecl.cpp
+++ b/clang/lib/Serialization/ASTReaderDecl.cpp
@@ -2593,7 +2593,7 @@ RedeclarableResult
ASTDeclReader::VisitClassTemplateSpecializationDeclImpl(
}
void ASTDeclReader::VisitClassTemplatePartialSpecializationDecl(
- ClassTemplatePartialSpecializationDecl *D)
{
+ ClassTemplatePartialSpecializationDecl *D) {
// We need to read the template params first because redeclarable is going to
// need them for profiling
TemplateParameterList *Params = Record.readTemplateParameterList();
diff --git a/clang/lib/Serialization/ASTWriterDecl.cpp
b/clang/lib/Serialization/ASTWriterDecl.cpp
index 7f5005aa6..7d003b44f 100644
--- a/clang/lib/Serialization/ASTWriterDecl.cpp
+++ b/clang/lib/Serialization/ASTWriterDecl.cpp
@@ -1979,7 +1979,7 @@ void ASTDeclWriter::VisitClassTemplateSpecializationDecl(
}
void ASTDeclWriter::VisitClassTemplatePartialSpecializationDecl(
- ClassTemplatePartialSpecializationDecl *D)
{
+ ClassTemplatePartialSpecializationDecl *D) {
Record.AddTemplateParameterList(D->getTemplateParameters());
VisitClassTemplateSpecializationDecl(D);
``````````
</details>
https://github.com/llvm/llvm-project/pull/201864
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
