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

Chris Bieneman via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Mar 30 11:00:15 PDT 2017

beanz added a comment.

Please revert your patch. It is *not* the right solution and is masking underlying problems.

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`. I suspect the problem is that when built as a DSO ExecutionEngine has no unresolved symbols against RuntimeDyld, and the linker probably isn't including the reference to RuntimeDyld in the produced ExecutionEngine DSO.

Regardless of the underlying cause, this is not the correct solution, it merely hides a problem that could occur in other consumers of ExecutionEngine.



More information about the lldb-commits mailing list