2008/12/31 Chris Lattner <span dir="ltr"><<a href="mailto:clattner@apple.com">clattner@apple.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;">
<div style=""><div><div class="Ih2E3d"><div>On Dec 30, 2008, at 2:54 PM, Talin wrote:</div><blockquote type="cite"><div>OK changes made and new patch attached.</div></blockquote></div></div><div><div style="margin: 0px;">
+++ utils/unittest/Makefile<span style="white-space: pre;">     </span>(revision 0)</div><div><font face="Monaco" size="2"><span style="font-size: 10px;"><br></span></font></div><div><font face="Monaco" size="2"><span style="font-size: 10px;">...</span></font></div>
<div><font face="Monaco" size="2"><span style="font-size: 10px;"><div style="margin: 0px;">+# This has to come after Makefile.common, since it doesn't allow us to</div><div style="margin: 0px;">+# override the VPATH value unless we set PROJECT_NAME, which we don't want</div>
<div style="margin: 0px;">+# to do.</div><div style="margin: 0px;">+VPATH = $(LLVM_SRC_ROOT)/utils/unittest/googletest/src/</div><div><br></div><div><br></div><div>Why play with VPATH here?  What is this doing?  Can this be handled a different way?  Similarly in makefile.unittest.</div>
</span></font></div></div></div></blockquote><div><br>Removed VPATH in this file by moving GTest up one directory, and added VPATH-less Makefiles:<br><a href="http://llvm.org/viewvc/llvm-project?view=rev&revision=61539">http://llvm.org/viewvc/llvm-project?view=rev&revision=61539</a><br>
<a href="http://llvm.org/viewvc/llvm-project?view=rev&revision=61540">http://llvm.org/viewvc/llvm-project?view=rev&revision=61540</a> (minor revision)<br> <br>Also removed VPATH from Makefile.unittest (see link below).<br>
<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 style=""><div><div><font face="Monaco" size="2"><span style="font-size: 10px;"><div>
</div><div style="margin: 0px;">+++ unittests/ADT/DenseMapTest.cpp<span style="white-space: pre;">        </span>(revision 0)</div><div><div><div style="margin: 0px;">+namespace {</div><div style="margin: 0px;">+</div><div style="margin: 0px;">
+using namespace llvm;</div><div><br></div><div><br></div><div>Please put the 'using' right after the #includes.</div></div></div></span></font></div></div></div></blockquote><div><br>Done.<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 style=""><div><div><font face="Monaco" size="2"><span style="font-size: 10px;"><div><div><div></div><div>Otherwise, the patch looks good to me.  Please verify that it works with both a SRCDIR=OBJDIR and SRCDIR!=OBJDIR build.</div>
</div></div></span></font></div></div></div></blockquote></div><br>Done for both; passes both with flying colors.<br>Submitted: <a href="http://llvm.org/viewvc/llvm-project?view=rev&revision=61541">http://llvm.org/viewvc/llvm-project?view=rev&revision=61541</a><br>
<br>Thanks to Talin for the original patch.<br><br>Now everyone can go write some unittests!<br><br>Misha<br><br>P.S.  Why are you still reading?  Go!  Write!  Test!<br>