[PATCH] [libcxxabi] Add lit configs for libcxxabi tests.

Saleem Abdulrasool compnerd at compnerd.org
Mon Jul 7 19:19:59 PDT 2014


================
Comment at: CMakeLists.txt:104
@@ -103,2 +103,3 @@
 option(LIBCXXABI_ENABLE_WERROR "Fail and stop if a warning is triggered." OFF)
-option(LIBCXXABI_ENABLE_SHARED "Build libc++abi as a shared library." OFF)
+option(LIBCXXABI_ENABLE_SHARED "Build libc++abi as a shared library." ON)
+
----------------
Did you intend to make this shared by default?

================
Comment at: CMakeLists.txt:110
@@ +109,3 @@
+  else()
+    set(LIBCXXABI_LIBCXX_INCLUDES "${LLVM_INCLUDE_DIR}/c++/v1")
+  endif()
----------------
I think that you should use a separate variable (LIBCXX_INCLUDE_DIR perhaps?) that is set to ${CMAKE_CURRENT_BINARY_DIR}/include rather than relying on LLVM's variable.

I realize that this is a standalone build, and therefore LLVM_INCLUDE_DIR *should* point to the same location, but it makes it something that you need to reason out (at least, I had to think about it for a moment).

================
Comment at: test/lit.cfg:1
@@ +1,2 @@
+# -*- Python -*- vim: set syntax=python tabstop=4 expandtab cc=80:
+
----------------
Is this the format that the rest of the python code in LLVM uses?  I don't really care at the end of the day, just curious.

http://reviews.llvm.org/D4414






More information about the cfe-commits mailing list