This is an archive of the discontinued LLVM Phabricator instance.

[CMake] Don't install c++abi headers in standalone libc++ build
ClosedPublic

Authored by phosek on Jul 24 2018, 1:13 PM.

Details

Summary

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 one in include/c++/v1. However, the
second copy is unnecessary when building libc++ standalone.

Diff Detail

Repository
rL LLVM

Event Timeline

phosek created this revision.Jul 24 2018, 1:13 PM

This was pointed out by @mclow.lists on IRC in our discussion related to D49711, I believe this change should resolve that issue.

With this change, only one copy of the c++abi headers gets copied - into include/c++/v1/.
LGTM.

This revision was not accepted when it landed; it landed in state Needs Review.Jul 25 2018, 3:57 PM
This revision was automatically updated to reflect the committed changes.