[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 10:06:01 PDT 2017


beanz added a comment.

This is definitely not the right fix. Please revert.

ExecutionEngine's LLVMBuild.txt file explicitly lists that RuntimeDyld is a required library of ExecutionEngine (as well as a few others). LLVM's CMake should be connecting that as an explicit dependency, and for some reason it isn't. Adding over-specified dependencies in LLDB to workaround bugs in LLVM is not the right approach, and masks problems instead of resolving them.


Repository:
  rL LLVM

https://reviews.llvm.org/D31367





More information about the lldb-commits mailing list