[PATCH] D117129: [clang-tidy] Extract Class IncluderClangTidyCheck
Yitzhak Mandelbaum via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jan 12 13:43:32 PST 2022
ymandel added a comment.
In D117129#3238441 <https://reviews.llvm.org/D117129#3238441>, @LegalizeAdulthood wrote:
> Another option would be to migrate this to `ClangTidyCheck` itself
> and add a flag/option passed to the ClangTidyCheck c'tor to opt-in
> to the feature?
Right, I was wondering about this. The feature is pretty core to handling C++ files and widely used, so it may warrant that. But, I don't know who the right reviewers are to ask about making changes to the `ClangTidyCheck` class.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D117129/new/
https://reviews.llvm.org/D117129
More information about the cfe-commits
mailing list