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

Dan Albert danalbert at google.com
Thu Jan 22 11:42:13 PST 2015


LGTM with at least the doc fix and the change to `sys.path.insert` (either change I mentioned is fine).


================
Comment at: test/lit.cfg:33
@@ +32,3 @@
+libcxx_test_src_root = os.path.join(libcxx_src_root, 'test')
+if os.path.isdir(libcxx_test_src_root):
+    sys.path.insert(0, libcxx_test_src_root)
----------------
`os.path.isfile(os.path.join(libcxx_test_src_root, 'libcxx', '__init__.py'))` is slightly more correct.

================
Comment at: test/lit.cfg:34
@@ -35,1 +33,3 @@
+if os.path.isdir(libcxx_test_src_root):
+    sys.path.insert(0, libcxx_test_src_root)
 else:
----------------
Apparently should use 1 rather than 0: http://stackoverflow.com/a/10097543/632035

Though in this case there's really no reason you can't just append it (or use `site.addsitedir`).

================
Comment at: www/index.html:93
@@ -92,2 +92,3 @@
   <ul>
+  <li>Check out the libcxx source tree.</li>
   <li><code>cd libcxxabi</code></li>
----------------
Make a link to the instructions on the libcxx page.

http://reviews.llvm.org/D7130

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






More information about the cfe-commits mailing list