[libcxx-commits] [libcxx] 1580c76 - [libc++] Enable the new libc++ testing format by default

Louis Dionne via libcxx-commits libcxx-commits at lists.llvm.org
Sat Apr 4 13:58:10 PDT 2020


Author: Louis Dionne
Date: 2020-04-04T16:57:16-04:00
New Revision: 1580c76c4a08e24adc34ae0a4ff4f97969188268

URL: https://github.com/llvm/llvm-project/commit/1580c76c4a08e24adc34ae0a4ff4f97969188268
DIFF: https://github.com/llvm/llvm-project/commit/1580c76c4a08e24adc34ae0a4ff4f97969188268.diff

LOG: [libc++] Enable the new libc++ testing format by default

Both test formats are equivalent, so this *should* not be a problem.
However, I'm taking advantage of the week-end to test this and see if
there are any failures. If so, it should be fine to revert this until
the failures have been addressed.

For the time being, it is still possible to use the old format by passing
`--param=use_old_format=True` when running Lit.

Added: 
    

Modified: 
    libcxx/test/lit.cfg

Removed: 
    


################################################################################
diff  --git a/libcxx/test/lit.cfg b/libcxx/test/lit.cfg
index 8f42dd7c468b..86e806067a5b 100644
--- a/libcxx/test/lit.cfg
+++ b/libcxx/test/lit.cfg
@@ -52,9 +52,9 @@ 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:
     import libcxx.test.newformat
     config.test_format = libcxx.test.newformat.CxxStandardLibraryTest()
-else:
-    config.test_format = configuration.get_test_format()


        


More information about the libcxx-commits mailing list