[libc++] 2/N: Enable the new libc++ testing format by default
authorLouis Dionne <ldionne@apple.com>
Mon, 6 Apr 2020 22:30:19 +0000 (18:30 -0400)
committerLouis Dionne <ldionne@apple.com>
Mon, 6 Apr 2020 22:35:17 +0000 (18:35 -0400)
Both test formats are equivalent, so this *should* not be a problem.
We've fixed a couple of failures uncovered by the first time we tried
making the switch, so this new attempt should go even farther.

If failures are noticed, it should be fine to revert this commit, but
please give a heads up afterwards so we know to address the issues!

Also note that it is still possible to use the old format by passing
`--param=use_old_format=True` when running Lit for the time being.

libcxx/test/lit.cfg

index 89c465c..ce98c63 100644 (file)
@@ -55,9 +55,10 @@ config_module = __import__(config_module_name, fromlist=['Configuration'])
 configuration = config_module.Configuration(lit_config, config)
 configuration.configure()
 configuration.print_config_info()
-if lit_config.params.get('use_new_format', False):
-    lit_config.note("Using the experimental libc++ testing format")
+if lit_config.params.get('use_old_format', False):
+    lit_config.note("Using the old libc++ testing format")
+    config.test_format = configuration.get_test_format()
+else:
+    lit_config.note("Using the new libc++ testing format")
     import libcxx.test.newformat
     config.test_format = libcxx.test.newformat.CxxStandardLibraryTest()
-else:
-    config.test_format = configuration.get_test_format()