<div>Yiannis, Pedro,</div><div><br></div>I'm not an expert here. Taking closer look I see that it makes sense to do this cleanup job in doFinalization() of the pass, which requested GC info. So the current fix seems to be ok and all external clients will need to do cleanup directly doing the same jobs as Printer::doFinalization() does now.<div>
<br></div><div>By the way, I've just noticed that comments for GCModuleInfo::clear() also needs to be updated, as it mentions deleter pass.<br><br>-Dmitry.<br><br><div class="gmail_quote">On Mon, Feb 25, 2013 at 9:55 PM, Yiannis Tsiouris <span dir="ltr"><<a href="mailto:gtsiour@softlab.ntua.gr" target="_blank">gtsiour@softlab.ntua.gr</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Dmitry,<div class="im"><br>
<br>
On 02/23/13 19:10, Dmitry Babokin wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
What's about the usage case, when LLVM is used as a library and the user implements its custom pass, which dump the code (implemented as a FunctionPass, but not as Printer)?<br>
</blockquote></div>
Hm, I haven't thought of that... I'll take a better look and try to find a more suitable solution. Any suggestion is more than welcome! :-)<div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
You also missed in your changes the declaration of llvm::createGCInfoDeleter() in include/llvm/CodeGen/Passes.h<br>
</blockquote></div>
Oops, sorry! I suppose I missed that! Pedro, will you take care of that (by just removing it) or should I sent a trivial fix in llvm-commits (where I don't have direct commit access)?<span class="HOEnZb"><font color="#888888"><br>

<br>
yiannis</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
-- <br>
Yiannis Tsiouris<br>
Ph.D. student,<br>
Software Engineering Laboratory,<br>
National Technical University of Athens<br>
WWW: <a href="http://www.softlab.ntua.gr/~gtsiour" target="_blank">http://www.softlab.ntua.gr/~<u></u>gtsiour</a><br>
<br>
</div></div></blockquote></div><br></div>