[PATCH] D14506: Porting shouldVisitImplicitCode to DataRecursiveASTVisitor.

Craig, Ben via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 16 14:28:32 PST 2015


I'm fine with this approach.  How about I leave the file in place, but 
replace the contents with a "using DataRecursiveASTVisitor = 
RecursiveASTVisitor;" and see what breaks?  That way I won't need to go 
through a large retrofit.

On 11/16/2015 3:28 PM, Richard Smith wrote:
> 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 <akyrtzi at gmail.com 
> <mailto:akyrtzi at gmail.com>> wrote:
>
>     LGTM.
>
>     > On Nov 16, 2015, at 12:32 PM, Ben Craig
>     <ben.craig at codeaurora.org <mailto:ben.craig at codeaurora.org>> 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
>     >
>     >
>     >
>
>

-- 
Employee of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20151116/f02087a1/attachment.html>


More information about the cfe-commits mailing list