<div>-  TRY_TO(TraverseTypeLoc(D->getTypeSourceInfo()->getTypeLoc()));</div><div>+  if (clang::TypeSourceInfo *TSI = D->getTypeSourceInfo()) {</div><div>+    TRY_TO(TraverseTypeLoc(TSI->getTypeLoc()));</div><div>
+  }</div><div><br></div><div>I assume the above is triggered by implicit nodes missing the type source info in some cases. Does the test case cover this?</div><div><br></div><div>Cheers,</div><div>/Manuel</div><br><div class="gmail_quote">
On Tue, Jun 5, 2012 at 1:37 AM, 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).<div>

<br></div><div>Any/all comments are welcome.</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>-- James</div><div><br><div><div><br></div><div><br><div><br></div><div><br></div></div></div></div>
</font></span><br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br>