[libcxx-commits] [PATCH] D57646: [libcxx] Specify unwind lib before other system libraries.

Yuanfang Chen via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Fri May 3 11:21:20 PDT 2019


tabloid.adroit added a comment.

In D57646#1490180 <https://reviews.llvm.org/D57646#1490180>, @ldionne wrote:

> I'll admit that I find it very brittle that ordering matters. We can do this, however it would be better to find another solution that doesn't require depending on the ordering of `-l` parameters. I don't know what that would be, though.


`install_name_tool -change` could adjust the order for executable as a post build action. If there are additional dependent libraries, then reorder logic need to be updated. I'm not sure it would be an improvement than just put more comments in the existing code since overriding libSystem.B.dylib is a rare case.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D57646





More information about the libcxx-commits mailing list