Skip to content

Commit 84c74e0

Browse files
committed
[libc++][modules] Increase clang-tidy version used.
As suggested in #71438 we should use export import std; in the std.compat module. Testing this locally failed when building with the clang-tidy-17 plugin. The std module was considered corrupt in the test libcxx/test/libcxx/module_std_compat.gen.py however the test libcxx/test/libcxx/module_std.gen.py passed. Both test generated identical std.pcm files. Using the clang-tidy-18 plugin solves the issue.
1 parent d06ae33 commit 84c74e0

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

libcxx/test/tools/clang_tidy_checks/CMakeLists.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,9 @@
55
set(LLVM_DIR_SAVE ${LLVM_DIR})
66
set(Clang_DIR_SAVE ${Clang_DIR})
77

8-
find_package(Clang 17)
8+
find_package(Clang 18)
99
if (NOT Clang_FOUND)
10-
find_package(Clang 18)
10+
find_package(Clang 17)
1111
endif()
1212

1313
set(SOURCES

libcxx/utils/libcxx/test/features.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,8 @@ def _getSuitableClangTidy(cfg):
3131
return None
3232

3333
# TODO MODULES require ToT due module specific fixes.
34-
if runScriptExitCode(cfg, ['clang-tidy-17 --version']) == 0:
35-
return 'clang-tidy-17'
34+
if runScriptExitCode(cfg, ['clang-tidy-18 --version']) == 0:
35+
return 'clang-tidy-18'
3636

3737
# TODO This should be the last stable release.
3838
# LLVM RELEASE bump to latest stable version

0 commit comments

Comments
 (0)