Skip to content

Commit 4598a3c

Browse files
committed
[libc++] Move the long_tests Lit feature to the DSL
1 parent c1a83c3 commit 4598a3c

File tree

2 files changed

+4
-13
lines changed

2 files changed

+4
-13
lines changed

libcxx/utils/libcxx/test/config.py

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,6 @@ def __init__(self, lit_config, config):
7474
self.use_target = False
7575
self.use_system_cxx_lib = self.get_lit_bool('use_system_cxx_lib', False)
7676
self.use_clang_verify = False
77-
self.long_tests = None
7877

7978
def get_lit_conf(self, name, default=None):
8079
val = self.lit_config.params.get(name, None)
@@ -285,18 +284,6 @@ def configure_features(self):
285284
self.config.available_features.add('availability=%s' % name)
286285
self.config.available_features.add('availability=%s%s' % (name, version))
287286

288-
# Simulator testing can take a really long time for some of these tests
289-
# so add a feature check so we can REQUIRES: long_tests in them
290-
self.long_tests = self.get_lit_bool('long_tests')
291-
if self.long_tests is None:
292-
# Default to running long tests.
293-
self.long_tests = True
294-
self.lit_config.note(
295-
"inferred long_tests as: %r" % self.long_tests)
296-
297-
if self.long_tests:
298-
self.config.available_features.add('long_tests')
299-
300287
if self.target_info.is_windows():
301288
if self.cxx_stdlib_under_test == 'libc++':
302289
# LIBCXX-WINDOWS-FIXME is the feature name used to XFAIL the

libcxx/utils/libcxx/test/params.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,4 +33,8 @@
3333
help="Whether to enable tests for the C++ <filesystem> library.",
3434
feature=lambda filesystem: None if filesystem else
3535
Feature(name='c++filesystem-disabled')),
36+
37+
Parameter(name='long_tests', choices=[True, False], type=bool, default=True,
38+
help="Whether to tests that take longer to run. This can be useful when running on a very slow device.",
39+
feature=lambda enabled: Feature(name='long_tests') if enabled else None),
3640
]

0 commit comments

Comments
 (0)