This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rGb765eaf9a617: [ASTMatchers] Add support for
CXXRewrittenBinaryOperator (authored by stephenkelly).
Changed prior to commit:
https://reviews.llvm.o
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
LGTM aside from the `auto` usage.
Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:5346
+AST_POLYMORPHIC_SUPPORTED_TYPES(BinaryOperator, CXXOperato
steveire marked an inline comment as done.
steveire added inline comments.
Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:5346
+AST_POLYMORPHIC_SUPPORTED_TYPES(BinaryOperator, CXXOperatorCallExpr,
+CXXRewrittenBinaryOperator)) {
steveire updated this revision to Diff 316751.
steveire added a comment.
Update
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D94130/new/
https://reviews.llvm.org/D94130
Files:
clang/docs/LibASTMatchersReference.html
clang/include/clang/AST/AST
aaron.ballman added inline comments.
Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:2000
+ CXXRewrittenBinaryOperator>
+cxxRewrittenBinaryOperator;
+
Missing changes to Registry.cpp to expose this t
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.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D94130
Files:
clang/docs/LibASTMatchersRef
