<div dir="ltr">+Ben Kramer, since he wrote <span style="font-family:arial,sans-serif;font-size:13px">r154743.</span><div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div><font face="arial, sans-serif">It seems reasonable for Sema to ask Parser to clean things up at the end of ActOnEndOfTranslationUnit. In this particular edge case we don't have a Parser call bracketing the Sema methods, so we have to resort to something more explicit.</font></div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Sep 2, 2014 at 12:04 PM, Brad King <span dir="ltr"><<a href="mailto:brad.king@kitware.com" target="_blank">brad.king@kitware.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 08/19/2014 03:43 PM, Brad King wrote:<br>
> With -fdelayed-template-parsing enabled (for MSVC compat) then late<br>
> template parsing allocates TemplateIdAnnotation instances that are<br>
> never freed.<br>
</div>[snip]<br>
<div class="">> Here is a patch series to add a test for this case and propose a<br>
> possible fix.<br>
<br>
</div>Ping.  My application cannot run in Assert builds without this, and<br>
I think avoiding memory leaks is good for Clang upstream anyway.<br>
<br>
Thanks,<br>
-Brad<br>
<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>
</blockquote></div><br></div>