[PATCH] D38171: Implement clang-tidy check aliases.

Alexander Kornienko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 22 07:18:59 PDT 2017


alexfh added a comment.

AndrĂ¡s, that's definitely an interesting idea. However, it might be interesting to explore a more principled approach:

1. Make `clang-diagnostic-*` checks first-class citizens and take full control of all diagnostics, i.e. disable all Clang diagnostics by default, and enable the ones that correspond to the enabled clang-diagnostic checks.
2. Make aliases first-class citizens (there was a proposal as well, but we didn't arrive to a consensus at that time). That would include the ability to configure an alias name for any check including clang-diagnostic- and clang-analyzer- checks.
3. Use aliases to map clang-diagnostic- checks to check names under cert-, hicpp-, etc.

I didn't carefully consider all possible implications and there may be issues with any or all of the three parts of this, but I think it's worth exploring.


https://reviews.llvm.org/D38171





More information about the cfe-commits mailing list