[PATCH] D16545: [libcxxabi] Enable testing for static libc++abi

Eric Fiselier via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 15 20:54:05 PDT 2016


EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.

This part LGTM. Sorry about the delay.


================
Comment at: CMakeLists.txt:187
@@ +186,3 @@
+if (NOT LIBCXXABI_LIBCXX_LIBRARY_PATH)
+  set(LIBCXXABI_LIBCXX_LIBRARY_PATH "${LIBCXXABI_LIBRARY_DIR}" CACHE PATH
+      "The path to libc++ library.")
----------------
This variable should still probably be re-set even if it's given on the command line. By doing that you update the type and help text for the variable in the cache.


http://reviews.llvm.org/D16545





More information about the cfe-commits mailing list