[PATCH] D15033: tools/llvm-config: improve shared library support

Andrew Wilkins via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 4 01:04:24 PST 2015


axw added a comment.

In http://reviews.llvm.org/D15033#297984, @DiamondLovesYou wrote:

> It looks like if LLVM is built with LLVM_LINK_LLVM_DYLIB=ON and `--link-static` is provided and the static archives are removed, `llvm-config` doesn't error and still prints the shared lib with `--libs`.


I've updated it to cater for this. It's a bit more complicated in order to keep current behaviour.

I'd prefer if we didn't do this "if static exists use, otherwise use shared", and rather just use whatever the installation is supposed to as according to LLVM_LINK_LLVM_DYLIB. Richard, if you have no objections, I'll change it to do that.


http://reviews.llvm.org/D15033





More information about the llvm-commits mailing list