================
Comment at: cmake/Modules/HandleLibCXXABI.cmake:85
@@ -82,1 +84,3 @@
+ set(CXXABI_LIBNAME cxxabi_shared)
+ endif()
set(LIBCXX_LIBCPPABI_VERSION "2" PARENT_SCOPE)
----------------
Why do you change the name of the library, the name should stay the same, its
the extension that changes (and I believe should be implicit based on
add_library).
================
Comment at: lib/CMakeLists.txt:41
@@ -40,1 +40,3 @@
+set(libraries "")
+if (LIBCXX_ENABLE_STATIC_ABI_LIBRARY)
----------------
Can you add this in side of a check for the linker type? The options are very
GNU centric.
================
Comment at: test/CMakeLists.txt:47
@@ -46,1 +46,3 @@
pythonize_bool(LIBCXX_ENABLE_MONOTONIC_CLOCK)
+ # The tests shouldn't link to any ABI library when it has been linked into
+ # libc++ statically.
----------------
Can you explain why this is correct? The tests are meant to test libc++abi.
They should link to it either statically or dynamically as built. The libc++
tests however, should not.
http://reviews.llvm.org/D8017
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
