[libcxx-commits] [libcxx] 882a138 - [libc++] Remove workaround for Clang < 20 in clang-tidy plugin (#211314)

via libcxx-commits libcxx-commits at lists.llvm.org
Thu Jul 23 06:21:49 PDT 2026


Author: Louis Dionne
Date: 2026-07-23T09:21:44-04:00
New Revision: 882a1381942c3bb4208136743ca7d743947a00a2

URL: https://github.com/llvm/llvm-project/commit/882a1381942c3bb4208136743ca7d743947a00a2
DIFF: https://github.com/llvm/llvm-project/commit/882a1381942c3bb4208136743ca7d743947a00a2.diff

LOG: [libc++] Remove workaround for Clang < 20 in clang-tidy plugin (#211314)

The clang-tidy plugin is now always built with Clang >= 20, so the
workaround can be removed.

Added: 
    

Modified: 
    libcxx/test/tools/clang_tidy_checks/robust_against_operator_ampersand.cpp

Removed: 
    


################################################################################
diff  --git a/libcxx/test/tools/clang_tidy_checks/robust_against_operator_ampersand.cpp b/libcxx/test/tools/clang_tidy_checks/robust_against_operator_ampersand.cpp
index 777dd6fe11df5..24089e33ca1c0 100644
--- a/libcxx/test/tools/clang_tidy_checks/robust_against_operator_ampersand.cpp
+++ b/libcxx/test/tools/clang_tidy_checks/robust_against_operator_ampersand.cpp
@@ -25,14 +25,6 @@
 // This is part of libc++'s policy
 // https://libcxx.llvm.org/CodingGuidelines.html#don-t-use-argument-dependent-lookup-unless-required-by-the-standard
 
-// TODO(LLVM-21) Remove dependentScopeDeclRefExpr
-// dependentScopeDeclRefExpr requires Clang 20, this uses the same definition as Clang
-#if defined(__clang_major__) && __clang_major__ < 20
-namespace clang::ast_matchers {
-const internal::VariadicDynCastAllOfMatcher<Stmt, DependentScopeDeclRefExpr> dependentScopeDeclRefExpr;
-} // namespace clang::ast_matchers
-#endif
-
 namespace libcpp {
 robust_against_operator_ampersand::robust_against_operator_ampersand(
     llvm::StringRef name, clang::tidy::ClangTidyContext* context)


        


More information about the libcxx-commits mailing list