[PATCH] D127446: [clang-tidy] Add `-verify-config` command line argument
Nathan James via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jun 10 01:42:58 PDT 2022
njames93 added a comment.
In D127446#3572491 <https://reviews.llvm.org/D127446#3572491>, @carlosgalvezp wrote:
> This sounds like great functionality, surely saving a lot of headaches! Any reason why we wouldn't want this active by default? I'd personally even go one step further and make it hard errors - warnings are easy to miss and ignore - but I can see how it can be problematic for some people.
I choose not to as I'd imagine it could would result in duplicated warnings when batch processing using run_clang_tidy over a large project.
Also if there's any edge cases, maybe with the static analyser, I wouldn't want those diagnostics triggering by default with no way to silence them.
I wouldn't be opposed to the possibility of making on by default after its had some time in the wild.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D127446/new/
https://reviews.llvm.org/D127446
More information about the cfe-commits
mailing list