[PATCH] D12488: Enable linking tools, shared libraries against libLLVM

Chris Bieneman via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 1 09:33:28 PDT 2015


beanz added a comment.

I still disagree with how this patch works, and there are some problems with it functionally too. For example, if you try to build clang, lld, or lldb with this it doesn't work at all, and even if it did, it would screw up the target dependencies, because llvm-shlib isn't processed by CMake until after clang, lld, and lldb.

I've also been thinking more about how this patch works, and I think it is implemented wrong. We should push all of this down into the `llvm_config` CMake functionality, and we should probably find a way to bake this behavior into the llvm-config executable and installed CMake files so that people using distributions of LLVM get the right behavior too.


http://reviews.llvm.org/D12488





More information about the llvm-commits mailing list