<div class="gmail_extra"><div class="gmail_quote">On Tue, Apr 24, 2012 at 1:32 PM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>RecursiveASTVisitor::TraverseClassInstantiations completely ignores templates for which we have instantiated the declaration but not the definition, so your patch does not introduce the problem I was concerned about. So: LGTM.</div>

<div><br></div><div>Separately from that, we have the question of whether we should be visiting such declarations. I think we probably should.</div></blockquote></div><br></div><div class="gmail_extra">Yes, we absolutely should.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">I don't understand why, if this is fixed, the original patch would still be required? I may be missing something though, i've not studied this in depth.</div>