[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