[PATCH] D39948: Make isDefinition matcher support ObjCMethodDecl

2017-11-14 Thread Dave Lee via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL318152: Make isDefinition matcher support ObjCMethodDecl (authored by kastiglione). Repository: rL LLVM https://reviews.llvm.org/D39948 Files: cfe/trunk/docs/LibASTMatchersReference.html cfe/trunk

[PATCH] D39948: Make isDefinition matcher support ObjCMethodDecl

2017-11-14 Thread Dave Lee via Phabricator via cfe-commits
kastiglione updated this revision to Diff 122831. kastiglione added a comment. Regenerate AST Matcher docs https://reviews.llvm.org/D39948 Files: docs/LibASTMatchersReference.html include/clang/ASTMatchers/ASTMatchers.h unittests/ASTMatchers/ASTMatchersNarrowingTest.cpp Index: unittests/

[PATCH] D39948: Make isDefinition matcher support ObjCMethodDecl

2017-11-14 Thread Dave Lee via Phabricator via cfe-commits
kastiglione updated this revision to Diff 122830. kastiglione added a comment. Update isDefinition comment docs to reflect addition of ObjCMethodDecl https://reviews.llvm.org/D39948 Files: docs/LibASTMatchersReference.html include/clang/ASTMatchers/ASTMatchers.h unittests/ASTMatchers/ASTM

[PATCH] D39948: Make isDefinition matcher support ObjCMethodDecl

2017-11-13 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM! https://reviews.llvm.org/D39948 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman

[PATCH] D39948: Make isDefinition matcher support ObjCMethodDecl

2017-11-12 Thread Dave Lee via Phabricator via cfe-commits
kastiglione created this revision. Herald added a subscriber: klimek. Allow the `isDefinition()` matcher to apply to `ObjCMethodDecl` nodes, in addition to those it already supports. For whatever reason, `ObjCMethodDecl` does not inherit from `FunctionDecl` and so this is specialization is necessa