erichkeane wrote:

> LGTM with a minor nit on the documentation, but I think this is a great new 
> feature for the diagnostics engine. Thank you for it!
> 
> btw, if you happen to have a list of diagnostics that could benefit from 
> switching to use it, that sure seems like Good First Issue territory.

Hmm... I don't.  I DID start walking down the list of diagnostics with 'large' 
selects at one point, but stopped at the first one I could find.  Perhaps I'll 
create a bug as a 'starter' bug with some guidance how to FIND these, and and 
hope someone comes along and does such an audit


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

Reply via email to