2008/12/30 Talin <span dir="ltr"><<a href="mailto:viridia@gmail.com">viridia@gmail.com</a>></span><br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Here's the version of the unit test patch, incorporating the feedback I have received so far.<div></div></blockquote><div><br>Looks good, a few comments below.<br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div>Some notes on the patch:</div><div><ul><li>This patch doesn't include googletest itself, that will need to be checked in separately. The source distribution will live in llvm/utils/unittest/googletest. (The reason for the extra directory level is so that the LLVM-specific makefiles can live in llvm/utils/unittest, while the googletest directory itself can be the pristine, unmodified distribution.)</li>
</ul></div></blockquote><div>Are you going to send a separate patch for this, or should I do this?<br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div><ul><li></li>
<li>The individual tests are in llvm/unittests (note plural).</li><li>The makefile target to build and run the tests is "make unittest" (I wasn't sure if you could have a synthetic target that was the same as the name of a directory, so I made it "unittest" instead of "unittests")</li>
</ul></div></blockquote><div>I don't think that's an issue -- to recurse into a directory, you have to say "make -C dir [target]".  <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div><ul><li></li>
<li>There's a common Makefile and a common TestMain.cpp in llvm/unittests. The individual tests are in llvm/unittests/<dir> where <dir> is the name of an LLVM library such as "ADT". Each of these subdirs has a tiny makefile which sets TESTNAME=<name> and then includes the common Makefile.unittest. Each subdir under unittests creates a separate executable. (I didn't use the default googletest main because I figured at some point we might want to customize main().)</li>

<li>I updated the LICENSE.txt and mkpatch, but I haven't done the HTML docs yet because I am still thinking about what to write.</li><li>I probably made some mistakes in setting up the makefile rules - that is what took the most time - so it will merit heightened scrutiny.</li>
</ul></div></blockquote><div><br>+++ unittests/Makefile.unittest    (revision 0)<br>[...]<br>+VPATH=$(PROJ_SRC_DIR) $(PROJ_SRC_ROOT)/unittests<br> <br>That doesn't look like a valid path to me.<br></div></div><br>