<div dir="ltr">+llvmdev<br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jan 28, 2014 at 8:03 PM, Saleem Abdulrasool <span dir="ltr"><<a href="mailto:compnerd@compnerd.org" target="_blank">compnerd@compnerd.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">When building a standalone clang (which I believe most of us don't do, I certainly do not do this when developing), I noticed a build failure and fixed the particular issue in the build (SVN r200096).<div>

<br></div><div>I agree with David here that if we are supporting a standalone build, we should ensure that it works correctly and builds identically (as much as possible) to the non-standalone version so that we do not end up accidentally introducing subtle bugs into the build.</div>

<div><br></div><div>Digging further into the issue (thanks dblakie!) it seems that the problem is exhibited due to a small change to the bundled copy of google-test in the LLVM repository.  That change was made in 2009 by Jeffery (SVN r81093) to assist in printing out types more conveniently by adapting std::ostream to LLVM's raw_ostream.</div>

<div><br></div><div>We have a few options here:</div><div><br></div><div>1) have gtest be part of the installed LLVM image</div><div>2) duplicate gtest into clang</div><div>3a) remove the change to google-test and adapt the unit-tests appropriately (which is what I did in the previously mentioned change for the cases that I found).</div>

<div>3b) move the required machinery for supporting google-test into libSupport.</div><div><div><br></div><div>I wanted to bring this up on this list in order to facilitate a discussion to determine the best course of action.</div>

<div><br></div><div>My personal preference would be for 3a.  We should adapt the tests to perform the necessary conversions to print out information as necessary.  This would also help reduce divergence from googletest which is a bonus IMO.</div>
<span class="HOEnZb"><font color="#888888">
<div><br></div>-- <br>Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org
</font></span></div></div>
</blockquote></div><br></div></div>