njames93 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? I feel that this direction is much better, Its something I wanted to introduce a while back, but my work was getting in the way. My proposed idea was putting the `IncludeInserter` instance inside the `ClangTidyContext`, but then only registering it if any checks actually want to use it. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117129/new/ https://reviews.llvm.org/D117129 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
