[PATCH] D37200: [AST] Traverse CXXOperatorCallExpr in LexicallyOrderedRecursiveASTVisitor

Johannes Altmanninger via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 28 06:24:37 PDT 2017


johannes added a comment.

The previous version didn't call TraverseDecl of the derived class, this is fixed now.
The public getDerived.TraverseStmt() does not accept a DataRecursionQueue, so this also could not be used (I think)
 I used the wrapper TraverseStmtBase, which should behave exactly as the method that originally traverses CXXOperatorCallExpr



================
Comment at: include/clang/AST/LexicallyOrderedRecursiveASTVisitor.h:63
   using BaseType = RecursiveASTVisitor<Derived>;
+  using typename BaseType::DataRecursionQueue;
 
----------------
arphaman wrote:
> Do you still need the using here?
removed it now as it's only used once


https://reviews.llvm.org/D37200





More information about the cfe-commits mailing list