[PATCH] [Zorg] Modify LibCxxAndABIBuilder to support testing libcxx with multiple standard versions.

Eric Fiselier eric at efcs.ca
Mon Sep 1 18:46:39 PDT 2014


>>! In D5114#5, @gribozavr wrote:
> Could we add more targets instead, check-libcxx-cxx11, check-libcxx-cxx14 etc?
> 
> Then it would be easier to run them when developing libcxx itself.

I agree that we should add these Makefile targets, but I would like to keep invoking lit directly in this change for a couple of reasons:

1. We need a custom LIT invocation so the output is more "log" friendly (no progress bar).
2. More changes are going to follow that require custom invocations. (Testing debug mode, sanitizers, ect)
3. I don't want to add the makefile targets until I've actually added support for running the tests as C++03, C++14, ect. So I want to use the bots to test these changes.

If you don't have any strong objections I would like to keep the custom LIT invocations.

================
Comment at: zorg/buildbot/builders/LibcxxAndAbiBuilder.py:95
@@ +94,3 @@
+            name='test.libcxx.%s' % std,
+            command=[lit_exe, '-v', '--show-unsupported', '--show-xfail',
+                     properties.WithProperties('--threads=%(jobs)s'),
----------------
danalbert wrote:
> -s so we're not inundated with all the passing tests?
'-s` add the progress bar that is non-log friendly. I'll add `-s` and `--no-progress-bar`, but I'm personally not opposed to printing the passing tests.

================
Comment at: zorg/buildbot/builders/LibcxxAndAbiBuilder.py:99
@@ -94,1 +98,3 @@
+                    libcxx_test_path],
+            workdir='.'))
 
----------------
danalbert wrote:
> Is that not the default?
I'm not actually sure, but I also don't think the working dir matters that much anyway. I'll remove it.

http://reviews.llvm.org/D5114






More information about the llvm-commits mailing list