[libcxx-commits] [PATCH] D116988: [libc++][libc++abi][libunwind] Dedup install path var definitions

John Ericson via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon Jan 10 19:12:27 PST 2022


Ericson2314 created this revision.
Ericson2314 added reviewers: phosek, beanz.
Herald added subscribers: libcxx-commits, mgorny.
Herald added a project: libunwind.
Herald added a reviewer: libunwind.
Ericson2314 requested review of this revision.
Herald added projects: libc++, libc++abi.
Herald added a reviewer: libc++.
Herald added a reviewer: libc++abi.

In D116873 <https://reviews.llvm.org/D116873> I did this for libunwind prior to defining a new install path
variable. But I think the change is good on its own, and libc++{,abi}
could also use it.

libc++ needed the base header var defined above the conditional part to
use it for the prefi+ed headers in the non-target-specific case. For
consistency, I therefore put the unconditional ones above for all 3
libs, which is why I touched the libunwind code (seeing that it had the
core change already)


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D116988

Files:
  libcxx/CMakeLists.txt
  libcxxabi/CMakeLists.txt
  libunwind/CMakeLists.txt

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D116988.398812.patch
Type: text/x-patch
Size: 7058 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20220111/08e8ca57/attachment-0001.bin>


More information about the libcxx-commits mailing list