[clang] [clang][Sema] Fix a CTAD regression after 42239d2e9 (PR #86914)
Younan Zhang via cfe-commits
cfe-commits at lists.llvm.org
Fri Mar 29 08:33:40 PDT 2024
================
@@ -1836,7 +1836,19 @@ static TemplateParameterList *GetTemplateParameterList(TemplateDecl *TD) {
// Make sure we get the template parameter list from the most
// recent declaration, since that is the only one that is guaranteed to
// have all the default template argument information.
- return cast<TemplateDecl>(TD->getMostRecentDecl())->getTemplateParameters();
+ Decl *ND = TD->getMostRecentDecl();
+ // Skip past friend Decls because they are not supposed to contain default
+ // template arguments. Moreover, these declarations may introduce template
+ // parameters living in different template depths than the corresponding
+ // template parameters in TD, causing unmatched constraint substitution.
+ //
+ // C++23 N4950 [temp.param]p12
----------------
zyn0217 wrote:
> Isn’t ‘N3337’ just the last draft of C++11? Just ‘C++11’ should be enough then?
https://github.com/timsong-cpp/cppwp says N3337 is the one with editorial fixes. So, I think it is.
https://github.com/llvm/llvm-project/pull/86914
More information about the cfe-commits
mailing list