https://github.com/mizvekov commented:

FWIW we don't need to make the `ImplicitConceptSpecializationDecl` a NamedDecl, 
if that helps reducing the overhead here.

The other template specializations just happen to be NamedDecls because they 
also represent the pattern of the template (ie a 
ClassTemplateSpecializationDecl is a CXXRecordDecl, but there is no such 
corresponding thing for a Concept).

Also, we should  drop the `Implicit` for `ImplicitConceptSpecializationDecl`, 
maybe not in this PR though.

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

Reply via email to