stephanemoore updated this revision to Diff 198956.
stephanemoore added a comment.
Present one potential option for making isDerivedFrom support Objective-C
classes.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60543/new/
https://reviews.llvm.org
aaron.ballman added inline comments.
Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:1479
+/// \endcode
+AST_MATCHER_P(ObjCInterfaceDecl, isSubclassOfInterface,
+ internal::Matcher,
jordan_rose wrote:
> aaron.ballman wrote:
> > jordan_rose
jordan_rose added inline comments.
Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:1479
+/// \endcode
+AST_MATCHER_P(ObjCInterfaceDecl, isSubclassOfInterface,
+ internal::Matcher,
aaron.ballman wrote:
> jordan_rose wrote:
> > aaron.ballman
aaron.ballman added inline comments.
Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:1479
+/// \endcode
+AST_MATCHER_P(ObjCInterfaceDecl, isSubclassOfInterface,
+ internal::Matcher,
jordan_rose wrote:
> aaron.ballman wrote:
> > jordan_rose
jordan_rose added inline comments.
Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:1479
+/// \endcode
+AST_MATCHER_P(ObjCInterfaceDecl, isSubclassOfInterface,
+ internal::Matcher,
aaron.ballman wrote:
> jordan_rose wrote:
> > aaron.ballman
aaron.ballman added inline comments.
Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:1479
+/// \endcode
+AST_MATCHER_P(ObjCInterfaceDecl, isSubclassOfInterface,
+ internal::Matcher,
jordan_rose wrote:
> aaron.ballman wrote:
> > stephanemoor
jordan_rose added inline comments.
Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:1479
+/// \endcode
+AST_MATCHER_P(ObjCInterfaceDecl, isSubclassOfInterface,
+ internal::Matcher,
aaron.ballman wrote:
> stephanemoore wrote:
> > aaron.ballma
aaron.ballman added reviewers: jordan_rose, rjmccall.
aaron.ballman added a comment.
Adding some ObjC experts in case they'd like to weigh in on the name
`isDerivedFrom` in this context.
Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:1479
+/// \endcode
+AST_MATCHER_
stephanemoore planned changes to this revision.
stephanemoore added inline comments.
Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:1479
+/// \endcode
+AST_MATCHER_P(ObjCInterfaceDecl, isSubclassOfInterface,
+ internal::Matcher,
aaron.ball
aaron.ballman added inline comments.
Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:1479
+/// \endcode
+AST_MATCHER_P(ObjCInterfaceDecl, isSubclassOfInterface,
+ internal::Matcher,
stephanemoore wrote:
> I am still uncertain about the nami
stephanemoore marked an inline comment as done.
stephanemoore added inline comments.
Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:1479
+/// \endcode
+AST_MATCHER_P(ObjCInterfaceDecl, isSubclassOfInterface,
+ internal::Matcher,
I am still
stephanemoore created this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
stephanemoore added a reviewer: aaron.ballman.
This change adds a new AST matcher to detect Objective-C classes that
are subclasses of matching Objective-C interfaces.
Test Notes:
Ran clang
12 matches
Mail list logo