diff options
author | Saar Raz <saar@raz.email> | 2020-01-27 00:57:31 +0200 |
---|---|---|
committer | Saar Raz <saar@raz.email> | 2020-01-27 01:00:12 +0200 |
commit | b39efdbcfcb7c7a5e867a99c2d3e756c538fbb9f (patch) | |
tree | 0ac8ee67b7cc32ccd770e756e3754337f3e29ac7 | |
parent | b07b82777b9ad19c44c6eb347507f1fdda3ddf34 (diff) | |
download | bcm5719-llvm-b39efdbcfcb7c7a5e867a99c2d3e756c538fbb9f.tar.gz bcm5719-llvm-b39efdbcfcb7c7a5e867a99c2d3e756c538fbb9f.zip |
[Concepts] Fix incorrect TemplateArgs for introduction of local parameters
The wrong set of TemplateArgs was being provided to addInstantiatedParametersToScope.
Caused bug #44658.
(cherry picked from commit 9c24fca2a33fc0fd059e278bb95c84803dfff9ae)
-rwxr-xr-x | clang/lib/Sema/SemaTemplateInstantiateDecl.cpp | 7 | ||||
-rw-r--r-- | clang/test/SemaTemplate/instantiate-requires-clause.cpp | 7 |
2 files changed, 10 insertions, 4 deletions
diff --git a/clang/lib/Sema/SemaTemplateInstantiateDecl.cpp b/clang/lib/Sema/SemaTemplateInstantiateDecl.cpp index fbbab8f0070..2e437cbe44d 100755 --- a/clang/lib/Sema/SemaTemplateInstantiateDecl.cpp +++ b/clang/lib/Sema/SemaTemplateInstantiateDecl.cpp @@ -4246,18 +4246,17 @@ bool Sema::CheckInstantiatedFunctionTemplateConstraints( Sema::ContextRAII savedContext(*this, Decl); LocalInstantiationScope Scope(*this); - MultiLevelTemplateArgumentList MLTAL = - getTemplateInstantiationArgs(Decl, nullptr, /*RelativeToPrimary*/true); - // If this is not an explicit specialization - we need to get the instantiated // version of the template arguments and add them to scope for the // substitution. if (Decl->isTemplateInstantiation()) { InstantiatingTemplate Inst(*this, Decl->getPointOfInstantiation(), InstantiatingTemplate::ConstraintsCheck{}, Decl->getPrimaryTemplate(), - MLTAL.getInnermost(), SourceRange()); + TemplateArgs, SourceRange()); if (Inst.isInvalid()) return true; + MultiLevelTemplateArgumentList MLTAL( + *Decl->getTemplateSpecializationArgs()); if (addInstantiatedParametersToScope( *this, Decl, Decl->getPrimaryTemplate()->getTemplatedDecl(), Scope, MLTAL)) diff --git a/clang/test/SemaTemplate/instantiate-requires-clause.cpp b/clang/test/SemaTemplate/instantiate-requires-clause.cpp index 31cf484d564..8e9d5bffa90 100644 --- a/clang/test/SemaTemplate/instantiate-requires-clause.cpp +++ b/clang/test/SemaTemplate/instantiate-requires-clause.cpp @@ -51,3 +51,10 @@ struct S2 { static_assert((S2<int>::f(), true)); +template<typename T> +struct S3 { + template<typename... Args> requires true + static constexpr void f(Args...) { } +}; + +static_assert((S3<int>::f(), true));
\ No newline at end of file |