aaron.ballman added a comment.

Do you expect to use this matcher in a new check in the immediate future? We 
usually don't push specialized matchers into ASTMatchers.h but instead try to 
keep them next to the only check using them. This reduces compile time overhead 
for everyone building Clang. (It's not that there's anything wrong with this 
one, it's more just resistance to adding matches unless they're sufficiently 
necessary.)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125383/new/

https://reviews.llvm.org/D125383

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to