[PATCH] D49584: [CMake] Install C++ ABI headers into the right location
Louis Dionne via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jul 20 06:43:27 PDT 2018
ldionne added a comment.
This change LGTM. However, I do have a question about the overall change (including r335809 and r337118). Why is it that we're not simply relying on the `CMAKE_INSTALL_PREFIX` being set to the path where we should install instead of using all these custom variables like `LIBCXX_INSTALL_HEADER_PREFIX`? I'm curious to understand what's special about your use case, since it does seem like we're doing things differently in a few places for Fuchsia.
Repository:
rCXX libc++
https://reviews.llvm.org/D49584
More information about the cfe-commits
mailing list