Rather than trying to maintain the horrible duplication between DataRecursiveASTVisitor and RecursiveASTVisitor, can we just delete DataRecursiveASTVisitor? RecursiveASTVisitor is data-recursive too these days (and has a smarter implementation than DataRecursiveASTVisitor's from what I can see), but doesn't yet apply data recursion in so many cases.
On Mon, Nov 16, 2015 at 1:07 PM, Argyrios Kyrtzidis <[email protected]> wrote: > LGTM. > > > On Nov 16, 2015, at 12:32 PM, Ben Craig <[email protected]> > wrote: > > > > bcraig added a comment. > > > > Ping. Note that the test is basically a copy / paste job, and the new > code in DataRecursiveASTVisitor.h is a very direct translation from the > 'regular' RecursiveASTVisitor.h. > > > > > > http://reviews.llvm.org/D14506 > > > > > > > >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
