[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