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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits