[PATCH] D36211: [cmake] Expose the dependencies of ExecutionEngine as PUBLIC
Chris Bieneman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 3 13:56:46 PDT 2017
beanz added a comment.
@chapuni, the reason that I wanted to have this done with `PUBLIC` interface specifications is because lldbExpression does't actually use anything from the RuntimeDYLD library. The problem is LLVMExecutionEngine's headers are not clean, and they reference RuntimeDYLD directly and result in needing to bring it in.
Longer term we should work with Lang to figure out a better solution, but this is the most reasonable way to allow downstream users of ExecutionEngine to only reference the libraries that they are directly using.
Repository:
rL LLVM
https://reviews.llvm.org/D36211
More information about the llvm-commits
mailing list