Here's the version of the unit test patch, incorporating the feedback I have received so far.<div><br></div><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>
<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>
<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>-- <br>-- Talin<br>
</div><div><br></div>