[PATCH] D20886: [libcxxabi] Allow target flags to affect configuration tests.
Saleem Abdulrasool via cfe-commits
cfe-commits at lists.llvm.org
Wed Jun 1 17:55:14 PDT 2016
compnerd added inline comments.
================
Comment at: CMakeLists.txt:201
@@ +200,3 @@
+ message(FATAL_ERROR "LIBCXXABI_BUILD_32_BITS=ON is not supported on this platform.")
+endif()
+
----------------
Why not handle this like libunwind? Alternative, we could duplicate this into libunwind. But we should behave similarly across the two I think.
================
Comment at: CMakeLists.txt:219
@@ +218,3 @@
+ list(APPEND LIBCXXABI_LINK_FLAGS ${var})
+endmacro()
+
----------------
No CMAKE_LINKER_FLAGS like libunwind/libc++?
================
Comment at: CMakeLists.txt:233
@@ +232,3 @@
+add_target_flags_if(LIBCXXABI_SYSROOT
+ "--sysroot ${LIBCXXABI_SYSROOT}")
+
----------------
This should be "--sysroot=${LIBCXXABI_SYSROOT}"
http://reviews.llvm.org/D20886
More information about the cfe-commits
mailing list