[PATCH] D117129: [clang-tidy] Extract Class IncluderClangTidyCheck

Richard via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 12 21:28:50 PST 2022


LegalizeAdulthood added a comment.

In D117129#3239143 <https://reviews.llvm.org/D117129#3239143>, @njames93 wrote:

> My proposed idea was putting the `IncludeInserter` instance
> inside the `ClangTidyContext`, but then only registering it if
> any checks actually want to use it.

Use of the `IncludeInserter` would be signaled by an argument
to the ctor fo r`ClangTidyCheck` in the c'tor for the derived
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