This revision was not accepted when it landed; it landed in state "Needs
Review".
This revision was automatically updated to reflect the committed changes.
Closed by commit rL337979: [CMake] Don't install c++abi headers in
standalone libc++ build (authored by phosek, committed by ).
Herald added
mclow.lists added a comment.
With this change, only one copy of the c++abi headers gets copied - into
`include/c++/v1/`.
LGTM.
Repository:
rCXX libc++
https://reviews.llvm.org/D49752
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http:/
phosek added a comment.
This was pointed out by @mclow.lists on IRC in our discussion related to
https://reviews.llvm.org/D49711, I believe this change should resolve that
issue.
Repository:
rCXX libc++
https://reviews.llvm.org/D49752
___
cfe-c
phosek created this revision.
phosek added reviewers: mclow.lists, EricWF, ldionne.
Herald added subscribers: cfe-commits, christof, mgorny.
This is a refinement on r337833. Previously we were installing two
copies of c++abi headers in libc++ build directory, one in
include/c++build and another on
