Re: [PATCH][www] Mention that libcxx is necessary on OS X

2015-09-07 Thread Nico Weber via cfe-commits
On Mon, Sep 7, 2015 at 9:17 AM, Kuba Brecka wrote: > > > On 07 Sep 2015, at 17:21, Nico Weber wrote: > > > > libcxx is needed to run compiler-rt tests, but it shouldn't be needed to > run clang's tests. And to _use_ your self-built compiler, you need to add > `-isysroot $(xcrun -show-sdk-path)`

Re: [PATCH][www] Mention that libcxx is necessary on OS X

2015-09-07 Thread Kuba Brecka via cfe-commits
> On 07 Sep 2015, at 17:21, Nico Weber wrote: > > libcxx is needed to run compiler-rt tests, but it shouldn't be needed to run > clang's tests. And to _use_ your self-built compiler, you need to add > `-isysroot $(xcrun -show-sdk-path)` to the compile flags to let the compiler > find C header

Re: [PATCH][www] Mention that libcxx is necessary on OS X

2015-09-07 Thread Nico Weber via cfe-commits
libcxx is needed to run compiler-rt tests, but it shouldn't be needed to run clang's tests. And to _use_ your self-built compiler, you need to add `-isysroot $(xcrun -show-sdk-path)` to the compile flags to let the compiler find C headers anyway, which also makes it find the libc++ headers. For co

Re: [PATCH][www] Mention that libcxx is necessary on OS X

2015-09-07 Thread Kuba Brecka via cfe-commits
I’m also curious what exact test failed for you with the "can't find iostream” error. AFAIK, you should currently receive the explaining message instead. But anyway, I’d also like to see libcxx[abi] mentioned in the Getting started page. Kuba > On 04 Sep 2015, at 23:07, Yuri Gribov

Re: [PATCH][www] Mention that libcxx is necessary on OS X

2015-09-04 Thread Yuri Gribov via cfe-commits
Oh, nice so this was actually done by Kuba to fix sanitizer unittests: "echo '#include ' | ${COMPILER_RT_TEST_COMPILER} -E -x c++ - > /dev/null" "if [ $? != 0 ] " " then echo" " echo 'Your just-built clang cannot find C++ headers, which are needed to build and run compile

Re: [PATCH][www] Mention that libcxx is necessary on OS X

2015-09-04 Thread Jonathan Roelofs via cfe-commits
On 9/4/15 2:55 PM, Yuri Gribov wrote: On Fri, Sep 4, 2015 at 11:49 PM, Jonathan Roelofs mailto:jonat...@codesourcery.com>> wrote: On 9/4/15 2:45 PM, Yuri Gribov via cfe-commits wrote: Hi all, According to https://llvm.org/bugs/show_bug.cgi?id=17821 , when Clang

Re: [PATCH][www] Mention that libcxx is necessary on OS X

2015-09-04 Thread Yuri Gribov via cfe-commits
On Fri, Sep 4, 2015 at 11:49 PM, Jonathan Roelofs wrote: > On 9/4/15 2:45 PM, Yuri Gribov via cfe-commits wrote: > >> Hi all, >> >> According to https://llvm.org/bugs/show_bug.cgi?id=17821 , when Clang >> gets built on OS X we need to build it with libcxx and libcxxabi >> (otherwise tests will fa

Re: [PATCH][www] Mention that libcxx is necessary on OS X

2015-09-04 Thread Jonathan Roelofs via cfe-commits
On 9/4/15 2:45 PM, Yuri Gribov via cfe-commits wrote: Hi all, According to https://llvm.org/bugs/show_bug.cgi?id=17821 , when Clang gets built on OS X we need to build it with libcxx and libcxxabi (otherwise tests will fail to link with "can't find iostream". This patch adds a note to getting

[PATCH][www] Mention that libcxx is necessary on OS X

2015-09-04 Thread Yuri Gribov via cfe-commits
Hi all, According to https://llvm.org/bugs/show_bug.cgi?id=17821 , when Clang gets built on OS X we need to build it with libcxx and libcxxabi (otherwise tests will fail to link with "can't find iostream". This patch adds a note to getting started page. Ok to apply? Best regards, Yury Gribov li