[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