[Lldb-commits] [PATCH] D31367: Expression: add missing linkage to RuntimeDyld component

[Michał Górny via lldb-commits]

On czw, 2017-03-30 at 21:37 +0200, Joerg Sonnenberger wrote:
> On Thu, Mar 30, 2017 at 06:00:15PM +0000, Chris Bieneman via Phabricator wrote:
> > ExecutionEngine headers directly reference symbols from RuntimeDyld,
> > so we should enforce a requirement that anyone using ExeuctionEngine
> > also link RuntimeDyld. This works today as expected for static archive
> > builds. It is only broken with `BUILD_SHARED_LIBS`.
> 
> Welcome to the brave new world of newer ELF linkers. Just because a
> library pulls in a symbol into the namespace doesn't mean you can just
> use that symbol. They want you to explicitly specify the dependency...
> This does happen for static archives though.

This is expected and desired. Just because some random library you're
using is using zlib does not mean you should skip linking zlib when your
program is using it too.

-- 
Best regards,
Michał Górny
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 963 bytes
Desc: This is a digitally signed message part
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20170330/c4c41e62/attachment.sig>