[libcxx-commits] [PATCH] D131037: [libc++] Simplify how we define the linker script for libc++

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue Aug 2 17:33:56 PDT 2022


ldionne created this revision.
ldionne added a reviewer: dim.
Herald added a subscriber: mgorny.
Herald added a project: All.
ldionne requested review of this revision.
Herald added a project: libc++.
Herald added a subscriber: libcxx-commits.
Herald added a reviewer: libc++.

Trying to be generic didn't work properly because we had to special-case
some interface libraries that we didn't want in the linker script. Instead,
only look at the ABI and the unwinding libraries explicitly.

This should solve the issue reported by @dim in [1].

[1]: https://discourse.llvm.org/t/15-0-0-rc1-has-been-tagged/64174/22


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D131037

Files:
  libcxx/cmake/Modules/DefineLinkerScript.cmake
  libcxx/src/CMakeLists.txt

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D131037.449496.patch
Type: text/x-patch
Size: 5383 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20220803/175f6f78/attachment.bin>


More information about the libcxx-commits mailing list