[PATCH] D22469: [compiler-rt] Don't require c++ headers when configuring compiler-rt builds

Jonas Hahnfeld via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 1 01:13:47 PDT 2016


Hahnfeld added a comment.

In https://reviews.llvm.org/D22469#500936, @Hahnfeld wrote:

> In https://reviews.llvm.org/D22469#500819, @beanz wrote:
>
> > @hahnfeld, I thought that while the sanitizers are written in C++ they don't actually use libcxx.
> >
> > -Chris
>
>
> They may not, I don't know. But `clang` tries to link because no `-nostdlib` is provided


The solution seems to be that simple: I posted https://reviews.llvm.org/D23005 which solves the problem for me.

Back to this revision: I think we should check that we actually have a C++ compiler (maybe without C++ headers and library), but not only a C compiler. Otherwise I think the sanitizers will definitely not build...


Repository:
  rL LLVM

https://reviews.llvm.org/D22469





More information about the llvm-commits mailing list