llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang-tools-extra @llvm/pr-subscribers-clang-tidy Author: Baranov Victor (vbvictor) <details> <summary>Changes</summary> Follow up to https://github.com/llvm/llvm-project/pull/174295. --- Full diff: https://github.com/llvm/llvm-project/pull/174299.diff 10 Files Affected: - (modified) clang-tools-extra/clang-tidy/bugprone/RedundantBranchConditionCheck.cpp (+6-4) - (modified) clang-tools-extra/clang-tidy/cppcoreguidelines/PreferMemberInitializerCheck.cpp (+2-2) - (modified) clang-tools-extra/clang-tidy/cppcoreguidelines/VirtualClassDestructorCheck.cpp (+1-1) - (modified) clang-tools-extra/clang-tidy/llvm/UseNewMLIROpBuilderCheck.cpp (+4-4) - (modified) clang-tools-extra/clang-tidy/modernize/ConcatNestedNamespacesCheck.cpp (+2-3) - (modified) clang-tools-extra/clang-tidy/modernize/ReplaceDisallowCopyAndAssignMacroCheck.cpp (+2-1) - (modified) clang-tools-extra/clang-tidy/readability/AvoidUnconditionalPreprocessorIfCheck.cpp (+5-3) - (modified) clang-tools-extra/clang-tidy/readability/RedundantInlineSpecifierCheck.cpp (+3-2) - (modified) clang-tools-extra/clang-tidy/utils/FormatStringConverter.cpp (+3-1) - (modified) clang-tools-extra/clang-tidy/utils/LexerUtils.cpp (+1-1) ``````````diff diff --git a/clang-tools-extra/clang-tidy/bugprone/RedundantBranchConditionCheck.cpp b/clang-tools-extra/clang-tidy/bugprone/RedundantBranchConditionCheck.cpp index 528c254dbe17e..705b8e41f9ab7 100644 --- a/clang-tools-extra/clang-tidy/bugprone/RedundantBranchConditionCheck.cpp +++ b/clang-tools-extra/clang-tidy/bugprone/RedundantBranchConditionCheck.cpp @@ -8,6 +8,7 @@ #include "RedundantBranchConditionCheck.h" #include "../utils/Aliasing.h" +#include "../utils/LexerUtils.h" #include "clang/AST/ASTContext.h" #include "clang/ASTMatchers/ASTMatchFinder.h" #include "clang/Analysis/Analyses/ExprMutationAnalyzer.h" @@ -135,8 +136,8 @@ void RedundantBranchConditionCheck::check( const SourceLocation BeforeOtherSide = OtherSide->getBeginLoc().getLocWithOffset(-1); const SourceLocation AfterOtherSide = - Lexer::findNextToken(OtherSide->getEndLoc(), *Result.SourceManager, - getLangOpts()) + utils::lexer::findNextTokenSkippingComments( + OtherSide->getEndLoc(), *Result.SourceManager, getLangOpts()) ->getLocation(); Diag << FixItHint::CreateRemoval( CharSourceRange::getTokenRange(IfBegin, BeforeOtherSide)) @@ -167,8 +168,9 @@ void RedundantBranchConditionCheck::check( CondOp->getLHS()->getBeginLoc(), BeforeRHS)); } else { const SourceLocation AfterLHS = - Lexer::findNextToken(CondOp->getLHS()->getEndLoc(), - *Result.SourceManager, getLangOpts()) + utils::lexer::findNextTokenSkippingComments( + CondOp->getLHS()->getEndLoc(), *Result.SourceManager, + getLangOpts()) ->getLocation(); Diag << FixItHint::CreateRemoval(CharSourceRange::getTokenRange( AfterLHS, CondOp->getRHS()->getEndLoc())); diff --git a/clang-tools-extra/clang-tidy/cppcoreguidelines/PreferMemberInitializerCheck.cpp b/clang-tools-extra/clang-tidy/cppcoreguidelines/PreferMemberInitializerCheck.cpp index f3e8d66a15102..ab34eb805aad9 100644 --- a/clang-tools-extra/clang-tidy/cppcoreguidelines/PreferMemberInitializerCheck.cpp +++ b/clang-tools-extra/clang-tidy/cppcoreguidelines/PreferMemberInitializerCheck.cpp @@ -7,10 +7,10 @@ //===----------------------------------------------------------------------===// #include "PreferMemberInitializerCheck.h" +#include "../utils/LexerUtils.h" #include "clang/AST/ASTContext.h" #include "clang/AST/Decl.h" #include "clang/ASTMatchers/ASTMatchFinder.h" -#include "clang/Lex/Lexer.h" #include "llvm/ADT/DenseMap.h" using namespace clang::ast_matchers; @@ -269,7 +269,7 @@ void PreferMemberInitializerCheck::check( } SourceLocation SemiColonEnd; - if (auto NextToken = Lexer::findNextToken( + if (auto NextToken = utils::lexer::findNextTokenSkippingComments( S->getEndLoc(), *Result.SourceManager, getLangOpts())) SemiColonEnd = NextToken->getEndLoc(); else diff --git a/clang-tools-extra/clang-tidy/cppcoreguidelines/VirtualClassDestructorCheck.cpp b/clang-tools-extra/clang-tidy/cppcoreguidelines/VirtualClassDestructorCheck.cpp index bfa9ce301f02e..e6b06d3ec4dcb 100644 --- a/clang-tools-extra/clang-tidy/cppcoreguidelines/VirtualClassDestructorCheck.cpp +++ b/clang-tools-extra/clang-tidy/cppcoreguidelines/VirtualClassDestructorCheck.cpp @@ -65,7 +65,7 @@ getVirtualKeywordRange(const CXXDestructorDecl &Destructor, /// Range ends with \c StartOfNextToken so that any whitespace after \c /// virtual is included. std::optional<Token> NextToken = - Lexer::findNextToken(VirtualEndLoc, SM, LangOpts); + utils::lexer::findNextTokenSkippingComments(VirtualEndLoc, SM, LangOpts); if (!NextToken) return std::nullopt; const SourceLocation StartOfNextToken = NextToken->getLocation(); diff --git a/clang-tools-extra/clang-tidy/llvm/UseNewMLIROpBuilderCheck.cpp b/clang-tools-extra/clang-tidy/llvm/UseNewMLIROpBuilderCheck.cpp index 4b60086a7edc5..e4564149066bd 100644 --- a/clang-tools-extra/clang-tidy/llvm/UseNewMLIROpBuilderCheck.cpp +++ b/clang-tools-extra/clang-tidy/llvm/UseNewMLIROpBuilderCheck.cpp @@ -7,9 +7,9 @@ //===----------------------------------------------------------------------===// #include "UseNewMLIROpBuilderCheck.h" +#include "../utils/LexerUtils.h" #include "clang/ASTMatchers/ASTMatchers.h" #include "clang/Basic/LLVM.h" -#include "clang/Lex/Lexer.h" #include "clang/Tooling/Transformer/RangeSelector.h" #include "clang/Tooling/Transformer/RewriteRule.h" #include "clang/Tooling/Transformer/Stencil.h" @@ -52,11 +52,11 @@ static EditGenerator rewrite(RangeSelector Call, RangeSelector Builder) { return CurrentToken; if (CurrentToken->is(clang::tok::eof)) return std::optional<Token>(); - return clang::Lexer::findNextToken(CurrentToken->getLocation(), SM, - LangOpts); + return utils::lexer::findNextTokenSkippingComments( + CurrentToken->getLocation(), SM, LangOpts); }; std::optional<Token> LessToken = - clang::Lexer::findNextToken(Begin, SM, LangOpts); + utils::lexer::findNextTokenSkippingComments(Begin, SM, LangOpts); while (LessToken && LessToken->getKind() != clang::tok::less) LessToken = NextToken(LessToken); if (!LessToken) { diff --git a/clang-tools-extra/clang-tidy/modernize/ConcatNestedNamespacesCheck.cpp b/clang-tools-extra/clang-tidy/modernize/ConcatNestedNamespacesCheck.cpp index 7c82e9ef029ba..f95b9349ff4a3 100644 --- a/clang-tools-extra/clang-tidy/modernize/ConcatNestedNamespacesCheck.cpp +++ b/clang-tools-extra/clang-tidy/modernize/ConcatNestedNamespacesCheck.cpp @@ -34,9 +34,8 @@ std::optional<SourceRange> NS::getCleanedNamespaceFrontRange(const SourceManager &SM, const LangOptions &LangOpts) const { // Front from namespace tp '{' - std::optional<Token> Tok = - ::clang::tidy::utils::lexer::findNextTokenSkippingComments( - back()->getLocation(), SM, LangOpts); + std::optional<Token> Tok = utils::lexer::findNextTokenSkippingComments( + back()->getLocation(), SM, LangOpts); if (!Tok) return std::nullopt; while (Tok->getKind() != tok::TokenKind::l_brace) { diff --git a/clang-tools-extra/clang-tidy/modernize/ReplaceDisallowCopyAndAssignMacroCheck.cpp b/clang-tools-extra/clang-tidy/modernize/ReplaceDisallowCopyAndAssignMacroCheck.cpp index be5e21dce3ba1..b10e2cc1c2e2e 100644 --- a/clang-tools-extra/clang-tidy/modernize/ReplaceDisallowCopyAndAssignMacroCheck.cpp +++ b/clang-tools-extra/clang-tidy/modernize/ReplaceDisallowCopyAndAssignMacroCheck.cpp @@ -7,6 +7,7 @@ //===----------------------------------------------------------------------===// #include "ReplaceDisallowCopyAndAssignMacroCheck.h" +#include "../utils/LexerUtils.h" #include "clang/Frontend/CompilerInstance.h" #include "clang/Lex/MacroArgs.h" #include "clang/Lex/PPCallbacks.h" @@ -59,7 +60,7 @@ const {0} &operator=(const {0} &) = delete{1})cpp", /// \returns \c true if the next token after the given \p MacroLoc is \b not a /// semicolon. bool shouldAppendSemi(SourceRange MacroLoc) { - std::optional<Token> Next = Lexer::findNextToken( + std::optional<Token> Next = utils::lexer::findNextTokenSkippingComments( MacroLoc.getEnd(), PP.getSourceManager(), PP.getLangOpts()); return !(Next && Next->is(tok::semi)); } diff --git a/clang-tools-extra/clang-tidy/readability/AvoidUnconditionalPreprocessorIfCheck.cpp b/clang-tools-extra/clang-tidy/readability/AvoidUnconditionalPreprocessorIfCheck.cpp index 9fc81850fa22e..a962c4659cfbf 100644 --- a/clang-tools-extra/clang-tidy/readability/AvoidUnconditionalPreprocessorIfCheck.cpp +++ b/clang-tools-extra/clang-tidy/readability/AvoidUnconditionalPreprocessorIfCheck.cpp @@ -7,6 +7,7 @@ //===----------------------------------------------------------------------===// #include "AvoidUnconditionalPreprocessorIfCheck.h" +#include "../utils/LexerUtils.h" #include "clang/AST/ASTContext.h" #include "clang/Lex/PPCallbacks.h" #include "clang/Lex/Preprocessor.h" @@ -45,7 +46,8 @@ struct AvoidUnconditionalPreprocessorIfPPCallbacks : public PPCallbacks { Token Tok; if (Lexer::getRawToken(Loc, Tok, SM, LangOpts, true)) { - std::optional<Token> TokOpt = Lexer::findNextToken(Loc, SM, LangOpts); + std::optional<Token> TokOpt = + utils::lexer::findNextTokenSkippingComments(Loc, SM, LangOpts); if (!TokOpt || TokOpt->getLocation().isMacroID()) return false; Tok = *TokOpt; @@ -55,8 +57,8 @@ struct AvoidUnconditionalPreprocessorIfPPCallbacks : public PPCallbacks { if (!isImmutableToken(Tok)) return false; - std::optional<Token> TokOpt = - Lexer::findNextToken(Tok.getLocation(), SM, LangOpts); + std::optional<Token> TokOpt = utils::lexer::findNextTokenSkippingComments( + Tok.getLocation(), SM, LangOpts); if (!TokOpt || TokOpt->getLocation().isMacroID()) return false; Tok = *TokOpt; diff --git a/clang-tools-extra/clang-tidy/readability/RedundantInlineSpecifierCheck.cpp b/clang-tools-extra/clang-tidy/readability/RedundantInlineSpecifierCheck.cpp index 7b6eb3443d27a..e3ee9247b380d 100644 --- a/clang-tools-extra/clang-tidy/readability/RedundantInlineSpecifierCheck.cpp +++ b/clang-tools-extra/clang-tidy/readability/RedundantInlineSpecifierCheck.cpp @@ -7,6 +7,7 @@ //===----------------------------------------------------------------------===// #include "RedundantInlineSpecifierCheck.h" +#include "../utils/LexerUtils.h" #include "clang/AST/ASTContext.h" #include "clang/AST/Decl.h" #include "clang/AST/DeclCXX.h" @@ -64,8 +65,8 @@ static SourceLocation getInlineTokenLocation(SourceRange RangeLocation, CurrentToken->getRawIdentifier() == "inline") return CurrentToken->getLocation(); - CurrentToken = - Lexer::findNextToken(CurrentToken->getLocation(), Sources, LangOpts); + CurrentToken = utils::lexer::findNextTokenSkippingComments( + CurrentToken->getLocation(), Sources, LangOpts); } return {}; } diff --git a/clang-tools-extra/clang-tidy/utils/FormatStringConverter.cpp b/clang-tools-extra/clang-tidy/utils/FormatStringConverter.cpp index 5e879e940f3fc..cbf852ea7afc3 100644 --- a/clang-tools-extra/clang-tidy/utils/FormatStringConverter.cpp +++ b/clang-tools-extra/clang-tidy/utils/FormatStringConverter.cpp @@ -14,6 +14,7 @@ #include "FormatStringConverter.h" #include "../utils/FixItHintUtils.h" +#include "../utils/LexerUtils.h" #include "clang/AST/Expr.h" #include "clang/ASTMatchers/ASTMatchFinder.h" #include "clang/Basic/LangOptions.h" @@ -800,7 +801,8 @@ void FormatStringConverter::applyFixes(DiagnosticBuilder &Diag, for (const auto &[ArgIndex, Replacement] : ArgFixes) { const SourceLocation AfterOtherSide = - Lexer::findNextToken(Args[ArgIndex]->getEndLoc(), SM, LangOpts) + utils::lexer::findNextTokenSkippingComments(Args[ArgIndex]->getEndLoc(), + SM, LangOpts) ->getLocation(); Diag << FixItHint::CreateInsertion(Args[ArgIndex]->getBeginLoc(), diff --git a/clang-tools-extra/clang-tidy/utils/LexerUtils.cpp b/clang-tools-extra/clang-tidy/utils/LexerUtils.cpp index 6e93871289ed9..bff91b6aa267f 100644 --- a/clang-tools-extra/clang-tidy/utils/LexerUtils.cpp +++ b/clang-tools-extra/clang-tidy/utils/LexerUtils.cpp @@ -100,7 +100,7 @@ bool rangeContainsExpansionsOrDirectives(SourceRange Range, if (Loc.isMacroID()) return true; - std::optional<Token> Tok = Lexer::findNextToken(Loc, SM, LangOpts); + std::optional<Token> Tok = findNextTokenSkippingComments(Loc, SM, LangOpts); if (!Tok) return true; `````````` </details> https://github.com/llvm/llvm-project/pull/174299 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
