<div>Hi James,</div><br><div class="gmail_quote">On Mon, Jun 4, 2012 at 4:37 PM, James Dennett <span dir="ltr"><<a href="mailto:jdennett@googlers.com" target="_blank">jdennett@googlers.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

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).</blockquote>

<div><br></div><div>LGTM, r158002.</div></div>