================
@@ -2154,6 +2144,15 @@ bool 
RecursiveASTVisitor<Derived>::TraverseFunctionHelper(FunctionDecl *D) {
                                                   TALI->NumTemplateArgs));
       }
     }
+    // FIXME: Do we want to traverse the explicit template arguments for
----------------
sdkrystian wrote:

I originally added this comment because it changes behavior (i.e. explicit 
template arguments for dependent friends were not previously traversed, but are 
traversed with this change). Upon a second examination, I don't think the 
comment is necessary, and the correct behavior is to traverse the explicit 
arguments (as we would traverse them for a non-dependent specialization).

https://github.com/llvm/llvm-project/pull/66636
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to