[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