[cfe-commits] PATCH: add an option to RecursiveASTVisitor to visit implicit declarations

Richard Smith richard at metafoo.co.uk
Tue Jun 5 09:20:08 PDT 2012


Hi James,

On Mon, Jun 4, 2012 at 4:37 PM, James Dennett <jdennett at googlers.com> wrote:

> Sometimes an AST visitor can be most straightforward if it can assume that
> all decls that are referred to have been visited.  The current code doesn't
> visit such nodes.  With the attached patch they are visited if an AST
> visitor asks for it; this should have no effect for existing visitors
> (unless they coincidentally happen to have a member
> called shouldVisitImplicitDeclarations).


LGTM, r158002.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120605/f6607636/attachment.html>


More information about the cfe-commits mailing list