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

[Louis Dionne via libcxx-commits]

https://github.com/ldionne created https://github.com/llvm/llvm-project/pull/211314

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

>From 2f83ae7356e61f23f76e3f4c3b33a59da6a70479 Mon Sep 17 00:00:00 2001
From: Louis Dionne <ldionne.2 at gmail.com>
Date: Wed, 22 Jul 2026 10:10:26 -0400
Subject: [PATCH] [libc++] Remove workaround for Clang < 20 in clang-tidy
 plugin

The clang-tidy plugin is now always built with Clang >= 20, so the
workaround can be removed.
---
 .../robust_against_operator_ampersand.cpp                 | 8 --------
 1 file changed, 8 deletions(-)

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)