[PATCH] D102213: [ASTMatchers] Add forCallable(), a generalization of forFunction().

Valeriy Savchenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 12 01:42:17 PDT 2021


vsavchenko added a comment.

I honestly don't see a reason why it's not a part of `forFunction`.  `forFunction` matches C++ methods and lambdas, Obj-C methods and blocks don't seem that much more special to have an extended matcher just for those.
I really don't think that it will break someone's workflow and that someone really relied on the fact that `forFunction` doesn't match those.
Other than that it's a great patch IMO!


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

https://reviews.llvm.org/D102213



More information about the cfe-commits mailing list