[PATCH] D39020: Made shlib compile explicitly AFTER all tools.

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 17 14:40:56 PDT 2017


pcc added a comment.

Taking a step back, I don't think we even want the `llvm-cfi-verify` code in the shared library. That is because it is only intended to be used by one executable and the test suite. If you can avoid linking it in to the shlib, I guess you wouldn't need to do anything about the enumeration order of `llvm-shlib`.

Does it not work to declare your library with `add_library` instead of `add_llvm_library`? Alternatively it looks like `add_llvm_library` has a `BUILDTREE_ONLY` flag that appears to do what is needed here.


https://reviews.llvm.org/D39020





More information about the llvm-commits mailing list