[PATCH] D42459: [cmake] [libcxx] Call llvm_setup_rpath() when adding shared libraries.

Louis Dionne via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 9 12:45:58 PST 2020


ldionne added a comment.

@hintonda  Is there a reason why this was done except for "standardizing the usage across projects"?

I'm asking because this change prevents us from being able to avoid having a rpath completely when building libc++. That is the behavior we want when building libc++ as a system library, in which case we specify the install name dir to be `/usr/lib`, and we don't want any rpath to appear.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D42459/new/

https://reviews.llvm.org/D42459



More information about the cfe-commits mailing list