vbvictor wrote: I'd suggest marking it `[[deprecated]]` for now and removing it completely after 2 releases of LLVM (AFAIK that's usual procedure).
I've [searched](https://sourcegraph.com/search?q=context:global+::clang::ast_matchers::internal::makeMatcher+-file:.*Matchers%5C.h%24+-file:.*ASTMatchersInternal%5C.h%24+-file:.*ASTMatchersMacros%5C.h%24+-file:.*RewriteRule%5C.cpp%24&patternType=keyword&sm=0) for usages too, and it doesn't seem to be used anywhere, but I think we still need to take into account private forks. If it is used in `clang/lib/Tooling/Transformer/RewriteRule.cpp`, some custom checks/tooling could use it too. https://github.com/llvm/llvm-project/pull/147197 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
