This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rGe810e95e4bb9: [ASTMatchers] Add binaryOperation matcher
(authored by stephenkelly).
Changed prior to commit:
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
LGTM aside from some minor NFC improvements.
Comment at: clang/lib/ASTMatchers/Dynamic/Marshallers.h:957
+
+ void getArgKinds(ASTNodeKind ThisKind, unsigned
steveire updated this revision to Diff 316749.
steveire added a comment.
Update
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D94129/new/
https://reviews.llvm.org/D94129
Files:
clang/docs/LibASTMatchersReference.html
aaron.ballman added inline comments.
Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:2739
+/// Matches nodes which can be used with binary operators
+///
Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:2785
steveire created this revision.
steveire added a reviewer: aaron.ballman.
steveire requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
This is a simple utility which allows matching on binaryOperator and
cxxOperatorCallExpr. It can also be