NagyDonat wrote:

I reverted this PR to an earlier stage and added a more verbose explanation for 
the (current) requirement that classes derived from `CheckerFamily` need to 
override `getTagDescription`.

@steakhal  In addition to the other changes, I also reverted commit 
https://github.com/llvm/llvm-project/pull/139256/commits/7da3862e87c3e6e94b80e04bddbfd04848f6ce04
 where you renamed `getTagDescription` to `getDebugName`, because although I 
completely agree with that change, I think it would be nicer to merge it as a 
separate commit (because e.g. this relatively large commit would be easier to 
understand without the name change). Do you want to create the PR for it (after 
this is merged) or should I create it?

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

Reply via email to