[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