[PATCH] D25659: [clang-tidy] Avoid running aliased checks twice

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 1 11:22:53 PDT 2016


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





More information about the cfe-commits mailing list