[PATCH] D29573: [libunwind][CMake] Use libc++ headers when available
Chris Bieneman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 8 11:29:36 PST 2017
beanz added inline comments.
================
Comment at: CMakeLists.txt:294
+ IS_DIRECTORY "${LIBUNWIND_LIBCXX_INCLUDES_INTERNAL}")
+ set(LIBUNWIND_CXX_INCLUDE_PATHS "${LIBUNWIND_LIBCXX_INCLUDES_INTERNAL}")
+endif()
----------------
I would name this variable with `_DEFAULT` on the end so that it is clear that you're setting the default value for a cached variable.
================
Comment at: CMakeLists.txt:300
+
+include_directories("${LIBUNWIND_CXX_INCLUDE_PATHS}")
+
----------------
Pretty sure `include_directories` can't be called with an empty string, so in the case where find_path fails, this could go wrong.
Repository:
rL LLVM
https://reviews.llvm.org/D29573
More information about the llvm-commits
mailing list