[PATCH] [libcxxabi] Teach CMake better ways to find the libc++ source directory (and misc cleanup).

Jonathan Roelofs jonathan at codesourcery.com
Thu Jan 22 10:08:45 PST 2015


================
Comment at: CMakeLists.txt:149
@@ +148,3 @@
+if (LIBCXXABI_LIBCXX_PATH STREQUAL "LIBCXXABI_LIBCXX_PATH-NOTFOUND")
+  message(WARNING "LIBCXXABI_LIBCXX_PATH was not specified and couldn't be infered.")
+  set(LIBCXXABI_LIBCXX_PATH "")
----------------
shouldn't this be an error?

================
Comment at: test/lit.cfg:29
@@ -28,4 +28,3 @@
 # and libcxxabi source directories are sibling directories.
-libcxx_source_root = getattr(config, 'libcxx_source_root',
-                             os.path.join(config.test_source_root,
-                                          '../../libcxx'))
+libcxx_source_root = getattr(config, 'libcxx_src_root', None)
+if not libcxx_source_root:
----------------
may as well s/_source_/_src_/ while you're here.

================
Comment at: test/lit.cfg:51
@@ -50,3 +50,3 @@
     site_cfg = lit_config.params.get('libcxxabi_site_config',
-                                     os.environ.get('LIBCXX_SITE_CONFIG'))
     if not site_cfg:
----------------
oops, good catch.

http://reviews.llvm.org/D7130

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list