[PATCH] D85278: [lld] Support building shared libLLD.so

[Tom Stellard via Phabricator via llvm-commits]

tstellar added a comment.

In D85278#2434383 <https://reviews.llvm.org/D85278#2434383>, @thakis wrote:

> From what I understand, linking lld as a shared library is an explicit non-goal of lld. It trusts inputs and crashes on bad inputs, since it's designed to be run as a separate process.
>
> Can you say more about what you want this for? It seems like something we don't want to do.

We ship the lld libraries and headers in Fedora too, and this change would be an improvement (1 shared library is better than having many).  As @mgorny said there is at least one project already using lld this way.  If lld is really not meant to be used as a library, then this needs to be formalized some way with documentation and by removing the install targets for the libraries and headers.  I don't personally have an opinion about whether or not lld should be used as a library, but if we do have libraries I prefer libLLD.so over what we have now.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D85278/new/

https://reviews.llvm.org/D85278