[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