Skip to content

Commit 0638549

Browse files
committed
[libc++] Refactor clang-query checks to clang-tidy checks to get less obscure error messages
Also remove clang-query related code, since it's unused now. Reviewed By: ldionne, Mordante, #libc Spies: libcxx-commits, arichardson Differential Revision: https://reviews.llvm.org/D141805
1 parent 7458908 commit 0638549

File tree

14 files changed

+148
-358
lines changed

14 files changed

+148
-358
lines changed

libcxx/docs/TestingLibcxx.rst

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,8 +98,7 @@ Additional tools
9898
The libc++ test suite uses a few optional tools to improve the code quality.
9999

100100
These tools are:
101-
- clang-query
102-
- clang-tidy
101+
- clang-tidy (you might need additional dev packages to compile libc++-specific clang-tidy checks)
103102

104103
Writing Tests
105104
-------------

libcxx/include/__memory_resource/unsynchronized_pool_resource.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@ class _LIBCPP_TYPE_VIS unsynchronized_pool_resource : public memory_resource {
8585

8686
void do_deallocate(void* __p, size_t __bytes, size_t __align) override;
8787

88-
_LIBCPP_HIDE_FROM_ABI bool do_is_equal(const memory_resource& __other) const _NOEXCEPT override {
88+
_LIBCPP_HIDE_FROM_ABI_VIRTUAL bool do_is_equal(const memory_resource& __other) const _NOEXCEPT override {
8989
return &__other == this;
9090
}
9191

libcxx/test/libcxx/clang_query.sh.cpp

Lines changed: 0 additions & 277 deletions
This file was deleted.

libcxx/test/libcxx/clang_query/README.md

Lines changed: 0 additions & 4 deletions
This file was deleted.

libcxx/test/libcxx/clang_query/abi_tag_on_virtual.query

Lines changed: 0 additions & 29 deletions
This file was deleted.

libcxx/test/libcxx/clang_query/hide_from_abi_or_visible.query

Lines changed: 0 additions & 37 deletions
This file was deleted.

libcxx/test/tools/clang_tidy_checks/CMakeLists.txt

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,13 @@ set(CMAKE_FIND_PACKAGE_SORT_DIRECTION DEC)
66
find_package(Clang 16)
77

88
set(SOURCES
9+
abi_tag_on_virtual.cpp
10+
hide_from_abi.cpp
911
robust_against_adl.cpp
10-
libcpp_module.cpp
1112
qualify_declval.cpp
12-
)
1313

14+
libcpp_module.cpp
15+
)
1416

1517
if(NOT Clang_FOUND)
1618
message(STATUS "Could not find a suitable version of the Clang development package;

0 commit comments

Comments
 (0)