[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