https://github.com/erichkeane commented:

This is going to need tests on redeclarations, implicit instantiations, and 
specializations.  Also, I wonder why this is applying to CXXRecordDecl rather 
than the primary template.

I don't see any code to do instantiation, which is a pretty manual process, so 
that probably needs attending to.

As far as the value here, I'm on the fence here.  I'd like to see much more 
justification here for this.

https://github.com/llvm/llvm-project/pull/211231
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to