================ @@ -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:
> At least I can’t find a mention of ‘N3337’ anywhere else in the codebase Interestingly, I found people tend not to spell out the standard file number throughout the codebase - seemingly, we usually speak `c++(version) [paragraph]/X.Y`. (I will respect such a convention next time; thanks for reminding me!) https://github.com/llvm/llvm-project/pull/86914 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits