<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Mar 24, 2014 at 2:32 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="">> Especially troubling is that if you remove the explicit instantiation<br>

> declaration, we no longer emit the vtable, so if this *is* necessary for<br>
> some reason, we're not doing it right.<br>
<br>
</div>Yes, that has to be a bug.<br>
<div class=""><br>
> So... I think this was originally incorrect, and still works only due to a<br>
> bug. The attached patch fixes the glitch, but I wanted to make sure I'm not<br>
> missing some subtlety here before going ahead with this.<br>
<br>
</div>Looks like the test was first added in r99226.  The relevant email<br>
thread seems to be<br>
<br>
<a href="http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20100322/028636.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20100322/028636.html</a><br>
<br>
Reading it again it seems that the real bug was the vtable not being<br>
produced on the use,  and me being confused about specializations<br>
having key functions.<br>
<br>
LGTM. Thanks for catching this. <br></blockquote></div><br></div><div class="gmail_extra">Thanks for digging into the history here! Committed as r204686.</div></div>