<div class="gmail_extra"><div class="gmail_quote">On Wed, May 9, 2012 at 3:29 PM, Daniel Jasper <span dir="ltr"><<a href="mailto:djasper@google.com" target="_blank">djasper@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Explicit class instantiations and explicit function instantiations are currently handled differently on the AST level in that the former create a dedicated AST entry in the TU decl context whereas the latter do not. This inconsistency currently leads to the RecursiveASTVisitor not visiting explicit function instantiations. The right solution is probably to create a dedicated node (e.g. ExplicitInstantiationDecl) and add a node of this type for any case of explicit instantiations (thoughts?).</blockquote>
<div><br></div><div>This seems like a good direction to me.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">However, as this is an elaborate change, I propose this short-term fix making the RecursiveASTVisitor visit the corresponding explicit function instantiations.
</blockquote></div><br></div><div class="gmail_extra">I've committed the short-term fix as r156522, thanks.</div>