aaron.ballman added a comment.

I *really* like this idea, thank you for working on this! A few things:

(0) I'm uncomfortable with the lack of tests in the patch. I'm not certain of a 
good way to test this, however. @alexfh, do you have ideas?
(1) I think that the aliases and the originals should be listed with 
-list-checks, because these are names under which the checks may be run (and 
sometimes the name may even imply different semantics for the check).
(2) I'm not as certain about -dump-config, since I am not really familiar with 
that option, but I think we want to list the alias and the original under that 
as well because different check names may have different configuration options.


https://reviews.llvm.org/D25659



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

Reply via email to