Chris Bieneman <be...@apple.com> writes: > beanz created this revision. > beanz added reviewers: bogner, EricWF. > beanz added a subscriber: cfe-commits. > > This copy phase is only needed for in-tree builds, so we should be > copying to the LLVM build directory's include dir instead of the > sub-project include dir.
LGTM! > https://reviews.llvm.org/D24015 > > Files: > include/CMakeLists.txt > > Index: include/CMakeLists.txt > =================================================================== > --- include/CMakeLists.txt > +++ include/CMakeLists.txt > @@ -10,11 +10,13 @@ > ${LIBCXX_SUPPORT_HEADER_PATTERN} > ) > > -file(COPY . > - DESTINATION "${CMAKE_BINARY_DIR}/include/c++/v1" > - FILES_MATCHING > - ${LIBCXX_HEADER_PATTERN} > - ) > +if(LLVM_BINARY_DIR) > + file(COPY . > + DESTINATION "${LLVM_BINARY_DIR}/include/c++/v1" > + FILES_MATCHING > + ${LIBCXX_HEADER_PATTERN} > + ) > +endif() > > if (LIBCXX_INSTALL_HEADERS) > install(DIRECTORY . > _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
