[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