Skip to content

Commit 3d43789

Browse files
authored
[libc++] Re-enable Clang-tidy checks in the CI (#110026)
Now that we've gained control of our CI docker image again, we can re-enable the clang-tidy tests.
1 parent 94316dd commit 3d43789

File tree

3 files changed

+2
-5
lines changed

3 files changed

+2
-5
lines changed

libcxx/include/chrono

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -998,7 +998,6 @@ constexpr chrono::year operator ""y(unsigned lo
998998
# include <cstdint>
999999
# include <stdexcept>
10001000
# include <string_view>
1001-
# include <vector>
10021001
#endif
10031002

10041003
#if !defined(_LIBCPP_REMOVE_TRANSITIVE_INCLUDES) && _LIBCPP_STD_VER <= 20

libcxx/test/libcxx/clang_tidy.gen.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515

1616
import sys
1717
sys.path.append(sys.argv[1])
18-
from libcxx.header_information import lit_header_restrictions, public_headers
18+
from libcxx.header_information import lit_header_restrictions, lit_header_undeprecations, public_headers
1919

2020
for header in public_headers:
2121
print(f"""\
@@ -30,6 +30,7 @@
3030
// UNSUPPORTED: clang-17
3131
3232
{lit_header_restrictions.get(header, '')}
33+
{lit_header_undeprecations.get(header, '')}
3334
3435
// TODO: run clang-tidy with modules enabled once they are supported
3536
// RUN: %{{clang-tidy}} %s --warnings-as-errors=* -header-filter=.* --checks='-*,libcpp-*' --load=%{{test-tools-dir}}/clang_tidy_checks/libcxx-tidy.plugin -- %{{compile_flags}} -fno-modules

libcxx/test/tools/clang_tidy_checks/CMakeLists.txt

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,3 @@
1-
# TODO: Re-enable the tests once the CI is back under control
2-
return()
3-
41
# The find_package changes these variables. This leaves the build in an odd
52
# state. Calling cmake a second time tries to write site config information in
63
# the system's libc++. Restoring these setting after testing fixes this issue.

0 commit comments

Comments
 (0)