[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