llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang-tools-extra Author: Victor Chernyakin (localspook) <details> <summary>Changes</summary> `dependentNameTypeLoc` is unused. `dependentScopeDeclRefExpr` has appeared in `ASTMatchers.h` since this code was written. --- Full diff: https://github.com/llvm/llvm-project/pull/155180.diff 1 Files Affected: - (modified) clang-tools-extra/clang-tidy/modernize/TypeTraitsCheck.cpp (-9) ``````````diff diff --git a/clang-tools-extra/clang-tidy/modernize/TypeTraitsCheck.cpp b/clang-tools-extra/clang-tidy/modernize/TypeTraitsCheck.cpp index de2d42d760550..472128201acc2 100644 --- a/clang-tools-extra/clang-tidy/modernize/TypeTraitsCheck.cpp +++ b/clang-tools-extra/clang-tidy/modernize/TypeTraitsCheck.cpp @@ -189,15 +189,6 @@ AST_MATCHER(TypeLoc, isType) { static constexpr char Bind[] = ""; void TypeTraitsCheck::registerMatchers(MatchFinder *Finder) { - const ast_matchers::internal::VariadicDynCastAllOfMatcher< - Stmt, - DependentScopeDeclRefExpr> - dependentScopeDeclRefExpr; // NOLINT(readability-identifier-naming) - const ast_matchers::internal::VariadicDynCastAllOfMatcher< - TypeLoc, - DependentNameTypeLoc> - dependentNameTypeLoc; // NOLINT(readability-identifier-naming) - // Only register matchers for trait<...>::value in c++17 mode. if (getLangOpts().CPlusPlus17) { Finder->addMatcher(mapAnyOf(declRefExpr, dependentScopeDeclRefExpr) `````````` </details> https://github.com/llvm/llvm-project/pull/155180 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
