erichkeane added inline comments.

================
Comment at: lib/Sema/SemaTemplate.cpp:4811
+  TemplateParameterList *Params =
+      cast<TemplateDecl>(Template->getMostRecentDecl())
+          ->getTemplateParameters();
----------------
mstorsjo wrote:
> How does this work if there's another forward declaration missing the 
> parameter later? Is the logic with "most recent" ok, or should it be "most 
> qualified/complete" instead?
I'm not sure the case you mean.  Any future declarations with fewer parameters 
would be aspecialization and a different decl, right?


https://reviews.llvm.org/D39127



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to