<div dir="auto"><div><div class="gmail_quote"><div dir="ltr">On Mon, Nov 12, 2018, 20:53 George Karpenkov via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">george.karpenkov added a comment.<br>
<br>
I don't quite understand the need for this patch.<br>
If we are talking about a binary built from source, wouldn't it make more sense to build libcxx from source as well?<br></blockquote></div></div><div dir="auto">I'm not sure what you mean by "built from source" - all programs are?</div><div dir="auto">The intent is to distribute as binary (I think), and not distribute libcxx. It's important the system stdlib is used, as that's the one consistent with the CDB and the actual build.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
If libcxx is built from source in the same repo, clang-based tools do find it.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D54310" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D54310</a><br>
<br>
<br>
<br>
</blockquote></div></div></div>