
<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">(+Bogner, I think he originally added this)<div class=""><br class=""></div><div class="">My understanding is that we only did this copy step if you are building with a clang that isn't installed so that the non-installed clang could find these headers.</div><div class=""><br class=""></div><div class="">I'm pretty sure compiler-rt finds libcxx via the sources, not the build directory.</div><div class=""><br class=""></div><div class="">I'll put together a fix later today.</div><div class=""><br class=""></div><div class="">-Chris</div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Dec 14, 2016, at 11:49 PM, Eric Fiselier <<a href="mailto:eric@efcs.ca" class="">eric@efcs.ca</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class=""><div class="gmail_extra"><div class="gmail_quote"><div class=""> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Eric, this part of this change seems wrong to me. It causes the headers to be installed into the libcxx build directory instead of the LLVM one. If you build using the LLVM runtimes directory this puts the headers in the wrong place for clang to find them.<br class="">

<br class="">

Is there a reason you're copying them into the libcxx binary dir? I generally don't think the headers are needed there.<br class=""><br class=""></blockquote><div class=""><br class=""></div><div class="">If it seems wrong for building in the LLVM runtimes directory then it probably is. I don't think there is a good reason why we need to copy the headers into the build directory, although we should double check compiler-rt uses the headers from the source directory.</div><div class=""><br class=""></div><div class="">If you know how to fix this feel free to commit a patch. If not I'll look into this more on Friday.</div><div class=""><br class=""></div><div class="">@Dan, @Jonathan: Do you guys use, or see a need for, this behavior?</div><div class=""><br class=""></div></div></div></div>

</div></blockquote></div><br class=""></div></body></html>