diff options
Diffstat (limited to 'lib/AST')
-rw-r--r-- | lib/AST/ASTContext.cpp | 2 | ||||
-rw-r--r-- | lib/AST/ASTImporter.cpp | 2 | ||||
-rw-r--r-- | lib/AST/DeclTemplate.cpp | 5 | ||||
-rw-r--r-- | lib/AST/TemplateBase.cpp | 12 |
4 files changed, 16 insertions, 5 deletions
diff --git a/lib/AST/ASTContext.cpp b/lib/AST/ASTContext.cpp index a98e0636d4..e1e6fd08aa 100644 --- a/lib/AST/ASTContext.cpp +++ b/lib/AST/ASTContext.cpp @@ -2768,7 +2768,7 @@ ASTContext::getCanonicalTemplateArgument(const TemplateArgument &Arg) const { case TemplateArgument::TemplateExpansion: return TemplateArgument(getCanonicalTemplateName( Arg.getAsTemplateOrTemplatePattern()), - true); + Arg.getNumTemplateExpansions()); case TemplateArgument::Integral: return TemplateArgument(*Arg.getAsIntegral(), diff --git a/lib/AST/ASTImporter.cpp b/lib/AST/ASTImporter.cpp index 7989b9f278..f9fe18f5df 100644 --- a/lib/AST/ASTImporter.cpp +++ b/lib/AST/ASTImporter.cpp @@ -1823,7 +1823,7 @@ ASTNodeImporter::ImportTemplateArgument(const TemplateArgument &From) { if (ToTemplate.isNull()) return TemplateArgument(); - return TemplateArgument(ToTemplate, true); + return TemplateArgument(ToTemplate, From.getNumTemplateExpansions()); } case TemplateArgument::Expression: diff --git a/lib/AST/DeclTemplate.cpp b/lib/AST/DeclTemplate.cpp index e7902e996a..b6716b34bd 100644 --- a/lib/AST/DeclTemplate.cpp +++ b/lib/AST/DeclTemplate.cpp @@ -337,7 +337,10 @@ ClassTemplateDecl::getInjectedClassNameSpecialization() { Arg = TemplateArgument(E); } else { TemplateTemplateParmDecl *TTP = cast<TemplateTemplateParmDecl>(*Param); - Arg = TemplateArgument(TemplateName(TTP), TTP->isParameterPack()); + if (TTP->isParameterPack()) + Arg = TemplateArgument(TemplateName(TTP), llvm::Optional<unsigned>()); + else + Arg = TemplateArgument(TemplateName(TTP)); } if ((*Param)->isTemplateParameterPack()) diff --git a/lib/AST/TemplateBase.cpp b/lib/AST/TemplateBase.cpp index 26c0c08971..f3def3eff2 100644 --- a/lib/AST/TemplateBase.cpp +++ b/lib/AST/TemplateBase.cpp @@ -135,6 +135,14 @@ bool TemplateArgument::containsUnexpandedParameterPack() const { return false; } +llvm::Optional<unsigned> TemplateArgument::getNumTemplateExpansions() const { + assert(Kind == TemplateExpansion); + if (TemplateArg.NumExpansions) + return TemplateArg.NumExpansions - 1; + + return llvm::Optional<unsigned>(); +} + void TemplateArgument::Profile(llvm::FoldingSetNodeID &ID, const ASTContext &Context) const { ID.AddInteger(Kind); @@ -223,7 +231,7 @@ TemplateArgument TemplateArgument::getPackExpansionPattern() const { return cast<PackExpansionExpr>(getAsExpr())->getPattern(); case TemplateExpansion: - return TemplateArgument(getAsTemplateOrTemplatePattern(), false); + return TemplateArgument(getAsTemplateOrTemplatePattern()); case Declaration: case Integral: @@ -389,8 +397,8 @@ TemplateArgumentLoc::getPackExpansionPattern(SourceLocation &Ellipsis, } case TemplateArgument::TemplateExpansion: - // FIXME: Variadic templates num expansions Ellipsis = getTemplateEllipsisLoc(); + NumExpansions = Argument.getNumTemplateExpansions(); return TemplateArgumentLoc(Argument.getPackExpansionPattern(), getTemplateQualifierRange(), getTemplateNameLoc()); |