aaron.ballman added inline comments.
================
Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:3841
+/// matches if ObjCMessageExpr's callee declaration matches
+///
----------------
NoQ wrote:
> Nitpick carryover: needs capital letter and `.`
Don't forget to regen the docs when you fix this, too. :-)
================
Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:3853
+/// objcMessageExpr(objcMessageCallee(objcMethodDecl(hasName("foo"))))
+AST_MATCHER_P(ObjCMessageExpr, objcMessageCallee,
+ internal::Matcher<ObjCMethodDecl>, InnerMatcher) {
----------------
Is there a reason why we want a separate matcher here instead of overloading
`callee()`?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D129398/new/
https://reviews.llvm.org/D129398
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
