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

Vasileios Kalintiris via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 2 10:35:48 PST 2015


vkalintiris added a comment.

In http://reviews.llvm.org/D13131#279131, @beanz wrote:

> The same basic problem you're having here was reported on IRC last week during the LLVM dev meeting. I've started a thread on llvm-dev to discuss this problem and possible solutions. I'm not completely opposed to this as a temporary solution, but it actually won't work for most cases.
>
> In the normal CMake cross-compiling process you won't get anywhere near this far through configuring the build because all of the `try_compile` and `check_cxx_compiler_flag` calls will fail. I assume the only reason that isn't biting you is because you're not actually cross-compiling via a CMake toolchain, you're relying on the hacked-in cross compiling capabilities in Compiler-RT's build.
>
> I do not want to block you from making progress, but I think it would be best for the project if we can at least discuss the long-term solution to this problem before we talk about temporary workarounds.
>
> I would encourage you to participate in the discussion, the thread is: http://lists.llvm.org/pipermail/llvm-dev/2015-November/091916.html
>
> Thanks,
> -Chris


The initial version of this review request is simple enough for me to maintain in our out-of-tree repositories. However, if the "right" solution to this problem is going to take too long, eg. more than say 6 months, it would be nice to have something upstream in order to avoid resolving conflicts manually. Also, given the intention to fix this, the initial version of the review request would be preferable to hacking the test_target_arch macro. More on the thread at llvm-dev :)


http://reviews.llvm.org/D13131





More information about the llvm-commits mailing list