Do the existing tests use a wildcard rule to gather all test sources? We would need to insure that the wildcard rule for the unit tests and the large tests are mutually exclusive.<div><br></div><div>Also, will the unit tests be one executable or many? They will probably compile/run faster if there is a smaller number of executables.<br>
<br><div class="gmail_quote">On Mon, Dec 29, 2008 at 1:43 PM, Misha Brukman <span dir="ltr"><<a href="mailto:brukman@gmail.com">brukman@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
2008/12/29 Talin <span dir="ltr"><<a href="mailto:viridia@gmail.com" target="_blank">viridia@gmail.com</a>></span><br><div class="gmail_quote"><div class="Ih2E3d"><blockquote class="gmail_quote" style="border-left:1px solid rgb(204, 204, 204);margin:0pt 0pt 0pt 0.8ex;padding-left:1ex">

I'm working on an update to the patch. The only thing holding me up is trying to come to a final decision as to where all the various pieces should live. Specifically, the Google Test library, and the actual unit tests themselves.</blockquote>

</div><div><br>I would recommend putting Google Test in llvm/test/googletest .  The unittests should probably go in llvm/test with other tests, in the same directories, e.g., tests of the various analysis algorithms would live in llvm/test/Analysis, 1 file per analysis, unless folks think they should go further into the tree: llvm/test/Analysis/Andersens/, etc., but I think it would be cleaner and easier if the files for a single library were in the same directory, as we would be able to create a single unittest from all of them, to speed up the build/link/test cycle.<br>

</div></div><br>Thoughts?<br>
<br>_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
<br></blockquote></div><br><br clear="all"><br>-- <br>-- Talin<br>
</div>