[PATCH] D35038: [libunwind] Add a test harness

Eric Fiselier via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 5 17:49:24 PDT 2017


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

This LGTM. I'm assuming it works but we can iron out the kinks once it lands. We should probably add builders that run the tests as well.



================
Comment at: test/lit.cfg:46
+    libcxx.test.config.loadSiteConfig(
+        lit_config, config, 'libunwind_site_config', 'LIBCXXABI_SITE_CONFIG')
+    obj_root = getattr(config, 'libunwind_obj_root', None)
----------------
s/`LIBCXXABI_SITE_CONFIG`/`LIBUNWIND_SITE_CONFIG`.


https://reviews.llvm.org/D35038





More information about the cfe-commits mailing list