diff --git a/libcxx/utils/libcxx/test/features.py b/libcxx/utils/libcxx/test/features.py --- a/libcxx/utils/libcxx/test/features.py +++ b/libcxx/utils/libcxx/test/features.py @@ -18,6 +18,20 @@ _isMSVC = lambda cfg: '_MSC_VER' in compilerMacros(cfg) _msvcVersion = lambda cfg: (int(compilerMacros(cfg)['_MSC_VER']) // 100, int(compilerMacros(cfg)['_MSC_VER']) % 100) +def _getSuitableClangTidy(cfg): + try: + if runScriptExitCode(cfg, ['stat %{test-tools}/clang_tidy_checks/libcxx-tidy.plugin']) != 0: + return '' + + if runScriptExitCode(cfg, ['clang-tidy-16 --version']) == 0: + return 'clang-tidy-16' + + if int(re.search('[0-9]+', commandOutput(cfg, ['clang-tidy --version'])).group()) >= 16: + return 'clang-tidy' + + except ConfigurationRuntimeError: + return '' + DEFAULT_FEATURES = [ Feature(name='fcoroutines-ts', when=lambda cfg: hasCompileFlag(cfg, '-fcoroutines-ts') and @@ -142,9 +156,8 @@ when=lambda cfg: runScriptExitCode(cfg, ['%{exec} bash -c \'bash --version\'']) != 0), Feature(name='has-clang-tidy', # TODO This should be the last stable release. - when=lambda cfg: runScriptExitCode(cfg, ['clang-tidy-16 --version']) == 0 and - runScriptExitCode(cfg, ['stat %{test-tools}/clang_tidy_checks/libcxx-tidy.plugin']) == 0, - actions=[AddSubstitution('%{clang-tidy}', 'clang-tidy-16')]), + when=lambda cfg: _getSuitableClangTidy(cfg) != '', + actions=[AddSubstitution('%{clang-tidy}', lambda cfg: _getSuitableClangTidy(cfg))]), Feature(name='has-clang-query', when=lambda cfg: runScriptExitCode(cfg, ['clang-query-15 --version']) == 0, actions=[AddSubstitution('%{clang-query}', 'clang-query-15')]),