[PATCH] D22866: [CMake] Load LLVMConfig for standalone build of builtins
Jonas Hahnfeld via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 1 00:23:52 PDT 2016
Hahnfeld added a comment.
In https://reviews.llvm.org/D22866#500834, @beanz wrote:
> This isn't the right solution here. We should instead make the builtins build load LLVMConfig.cmake (if available) and construct the value from the TARGET_TRIPLE the same way the sanitizer build does. If LLVMConfig.cmake isn't available we should make the builtin build error out during configuration that it needs the TARGET_TRIPLE specified.
>
> -Chris
Hi Chris,
I'm absolutely no CMake expert, but the changes seem to work for me.
Thanks,
Jonas
https://reviews.llvm.org/D22866
More information about the llvm-commits
mailing list