Author: flovent Date: 2025-09-08T21:37:03+03:00 New Revision: 3b452e0565897d2e28dfd9d703fa358fad2bac76
URL: https://github.com/llvm/llvm-project/commit/3b452e0565897d2e28dfd9d703fa358fad2bac76 DIFF: https://github.com/llvm/llvm-project/commit/3b452e0565897d2e28dfd9d703fa358fad2bac76.diff LOG: [clang-tidy][NFC] Enable `modernize-use-auto` in clang-tidy config and fix warnings (#157468) Closes [#156154](https://github.com/llvm/llvm-project/issues/156154) Added: Modified: clang-tools-extra/clang-tidy/.clang-tidy clang-tools-extra/clang-tidy/modernize/UseIntegerSignComparisonCheck.cpp clang-tools-extra/clang-tidy/modernize/UseScopedLockCheck.cpp clang-tools-extra/clang-tidy/readability/UseStdMinMaxCheck.cpp Removed: ################################################################################ diff --git a/clang-tools-extra/clang-tidy/.clang-tidy b/clang-tools-extra/clang-tidy/.clang-tidy index 6bb526c50ff71..ae554c6668a84 100644 --- a/clang-tools-extra/clang-tidy/.clang-tidy +++ b/clang-tools-extra/clang-tidy/.clang-tidy @@ -11,7 +11,6 @@ Checks: > modernize-*, -modernize-avoid-c-arrays, -modernize-pass-by-value, - -modernize-use-auto, -modernize-use-nodiscard, -modernize-use-trailing-return-type, performance-*, diff --git a/clang-tools-extra/clang-tidy/modernize/UseIntegerSignComparisonCheck.cpp b/clang-tools-extra/clang-tidy/modernize/UseIntegerSignComparisonCheck.cpp index eeba5cce80da5..4726674be66fd 100644 --- a/clang-tools-extra/clang-tidy/modernize/UseIntegerSignComparisonCheck.cpp +++ b/clang-tools-extra/clang-tidy/modernize/UseIntegerSignComparisonCheck.cpp @@ -138,9 +138,9 @@ void UseIntegerSignComparisonCheck::check( return; const Expr *SubExprLHS = nullptr; const Expr *SubExprRHS = nullptr; - SourceRange R1 = SourceRange(LHS->getBeginLoc()); - SourceRange R2 = SourceRange(BinaryOp->getOperatorLoc()); - SourceRange R3 = SourceRange(Lexer::getLocForEndOfToken( + SourceRange R1(LHS->getBeginLoc()); + SourceRange R2(BinaryOp->getOperatorLoc()); + SourceRange R3(Lexer::getLocForEndOfToken( RHS->getEndLoc(), 0, *Result.SourceManager, getLangOpts())); if (const auto *LHSCast = llvm::dyn_cast<ExplicitCastExpr>(LHS)) { SubExprLHS = LHSCast->getSubExpr(); diff --git a/clang-tools-extra/clang-tidy/modernize/UseScopedLockCheck.cpp b/clang-tools-extra/clang-tidy/modernize/UseScopedLockCheck.cpp index c74db0ed861b4..4041c81526d2f 100644 --- a/clang-tools-extra/clang-tidy/modernize/UseScopedLockCheck.cpp +++ b/clang-tools-extra/clang-tidy/modernize/UseScopedLockCheck.cpp @@ -98,7 +98,7 @@ static SourceRange getLockGuardRange(const TypeSourceInfo *SourceInfo) { // Find the exact source range of the 'lock_guard' name token static SourceRange getLockGuardNameRange(const TypeSourceInfo *SourceInfo) { - const TemplateSpecializationTypeLoc TemplateLoc = + const auto TemplateLoc = SourceInfo->getTypeLoc().getAs<TemplateSpecializationTypeLoc>(); if (!TemplateLoc) return {}; diff --git a/clang-tools-extra/clang-tidy/readability/UseStdMinMaxCheck.cpp b/clang-tools-extra/clang-tidy/readability/UseStdMinMaxCheck.cpp index 718467ed02f0a..511256332cee9 100644 --- a/clang-tools-extra/clang-tidy/readability/UseStdMinMaxCheck.cpp +++ b/clang-tools-extra/clang-tidy/readability/UseStdMinMaxCheck.cpp @@ -62,7 +62,7 @@ static bool maxCondition(const BinaryOperator::Opcode Op, const Expr *CondLhs, static QualType getNonTemplateAlias(QualType QT) { while (true) { // cast to a TypedefType - if (const TypedefType *TT = dyn_cast<TypedefType>(QT)) { + if (const auto *TT = dyn_cast<TypedefType>(QT)) { // check if the typedef is a template and if it is dependent if (!TT->getDecl()->getDescribedTemplate() && !TT->getDecl()->getDeclContext()->isDependentContext()) _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
