[clang-tools-extra] Add option to exclude headers from clang-tidy analysis (PR #91400)
Justin Cady via cfe-commits
cfe-commits at lists.llvm.org
Fri May 10 12:46:21 PDT 2024
================
@@ -311,7 +311,12 @@ ClangTidyDiagnosticConsumer::ClangTidyDiagnosticConsumer(
: Context(Ctx), ExternalDiagEngine(ExternalDiagEngine),
RemoveIncompatibleErrors(RemoveIncompatibleErrors),
GetFixesFromNotes(GetFixesFromNotes),
- EnableNolintBlocks(EnableNolintBlocks) {}
+ EnableNolintBlocks(EnableNolintBlocks) {
+
+ if (Context.getOptions().ExcludeHeaderFilterRegex)
+ ExcludeHeaderFilter = std::make_unique<llvm::Regex>(
+ *Context.getOptions().ExcludeHeaderFilterRegex);
+}
----------------
justincady wrote:
The change to make `HeaderFilter` mirror `ExcludeHeaderFilter` was not too bad, so I went ahead with it. I thought it might actually be better to do now while everything is fresh in my mind. 🙂
https://github.com/llvm/llvm-project/pull/91400
More information about the cfe-commits
mailing list