[PATCH] D82875: Added tests for RecursiveASTVisitor for AST nodes that are special cased
Dmitri Gribenko via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jun 30 07:01:01 PDT 2020
gribozavr2 marked an inline comment as done.
gribozavr2 added inline comments.
================
Comment at: clang/unittests/Tooling/RecursiveASTVisitorTests/Callbacks.cpp:398
-TEST(RecursiveASTVisitor, StmtCallbacks_TraverseBinaryOperator) {
+TEST(RecursiveASTVisitor, StmtCallbacks_TraverseUnaryOperator) {
class RecordingVisitor : public RecordingVisitorBase<RecordingVisitor> {
----------------
Phabricator shows the diff as a modification, but it is actually an addition -- new tests are just very similar to existing tests for binary operators, so the diff algorithm is confused. You can find old tests for binary operators below, shown as code being added.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82875/new/
https://reviews.llvm.org/D82875
More information about the cfe-commits
mailing list