[PATCH] D13131: Allow the builting of the builtins library with a forced compiler.

Alexey Samsonov via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 29 10:31:28 PDT 2015


samsonov added a comment.

In http://reviews.llvm.org/D13131#276341, @vkalintiris wrote:

> In http://reviews.llvm.org/D13131#272442, @samsonov wrote:
>
> > if arch we're testing is the same as `COMPILER_RT_DEFAULT_TARGET_ARCH`, and it was set explicitly, and we have CMAKE_C_COMPILER_FORCED, then we can add arch
>
>
> How can I test whether COMPILER_RT_DEFAULT_TARGET_ARCH was set explicitly? In my case, TARGET_TRIPLE = COMPILER_RT_DEFAULT_TARGET_TRIPLE and thus COMPILER_RT_HAS_EXPLICIT_DEFAULT_TARGET_TRIPLE=False (when I explicitly specify COMPILER_RT_DEFAULT_TARGET_TRIPLE=mips-mti-linux in the cmake command line).


Oh, ok. Then you probably can just allow arch in `test_target_arch` if it's the same as `COMPILER_RT_DEFAULT_TARGET_ARCH`, and `CMAKE_C_COMPILER_FORCED` is set.

> Other than that, I believe that what you suggested should be working for me.

> 

> > Will the build actually work without any standard libraries?

> 

> 

> Yes. For the builtins library, we only need the kernel headers and the headers from the C library.



http://reviews.llvm.org/D13131





More information about the llvm-commits mailing list