<div dir="ltr"><div class="gmail_default" style>On Fri, Jan 4, 2013 at 10:04 AM, Pedro Artigas <span dir="ltr"><<a href="mailto:partigas@apple.com" target="_blank" class="cremed">partigas@apple.com</a>></span> wrote:<br>
</div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: partigas<br>
Date: Fri Jan  4 12:04:42 2013<br>
New Revision: 171475<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=171475&view=rev" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project?rev=171475&view=rev</a><br>
Log:<br>
small fixes to enable the reuse of the pass manager across multiple modules<br></blockquote><div><br></div><div style>Cool!</div><div style><br></div><div style>As you keep finding issues here, I think it would be really good to add some regression tests. I see two ways of doing it:</div>
<div style><br></div><div style>1) Write unittests in llvm/unittests/... that spin up an LLVMContext and pass manager and run N boring modules through it.</div><div style><br></div><div style>2) Add support to 'opt' to run N modules (maybe N files) through a single LLVMContext and pass manager.</div>
<div style><br></div><div style><br></div><div style>While #1 seems slightly cleaner to me (it would let you more accurately reproduce bugs found with other uses of the LLVM pass manager libraries in general, it doesn't matter much, and both seem fine.</div>
<div style><br></div><div style>Hopefully that way these bugs will stay fixed. =]</div></div></div></div>