llvmorg-github-actions[bot] wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang-tools-extra Author: Baranov Victor (vbvictor) <details> <summary>Changes</summary> --- Patch is 35.02 KiB, truncated to 20.00 KiB below, full version: https://github.com/llvm/llvm-project/pull/203823.diff 26 Files Affected: - (modified) clang-tools-extra/clang-tidy/ClangTidyCheck.cpp (+1-1) - (modified) clang-tools-extra/clang-tidy/bugprone/MacroParenthesesCheck.cpp (+1-1) - (modified) clang-tools-extra/clang-tidy/bugprone/MoveForwardingReferenceCheck.cpp (+1-1) - (modified) clang-tools-extra/clang-tidy/bugprone/NotNullTerminatedResultCheck.cpp (+10-10) - (modified) clang-tools-extra/clang-tidy/bugprone/ParentVirtualCallCheck.cpp (+1-1) - (modified) clang-tools-extra/clang-tidy/bugprone/UseAfterMoveCheck.cpp (+1-1) - (modified) clang-tools-extra/clang-tidy/llvmlibc/InlineFunctionDeclCheck.cpp (+1-1) - (modified) clang-tools-extra/clang-tidy/misc/ConstCorrectnessCheck.cpp (+4-3) - (modified) clang-tools-extra/clang-tidy/misc/MisplacedConstCheck.cpp (+3-2) - (modified) clang-tools-extra/clang-tidy/misc/NoRecursionCheck.cpp (+2-2) - (modified) clang-tools-extra/clang-tidy/misc/UseInternalLinkageCheck.cpp (+1-1) - (modified) clang-tools-extra/clang-tidy/modernize/LoopConvertCheck.cpp (+3-3) - (modified) clang-tools-extra/clang-tidy/modernize/LoopConvertUtils.cpp (+6-4) - (modified) clang-tools-extra/clang-tidy/modernize/LoopConvertUtils.h (+2-1) - (modified) clang-tools-extra/clang-tidy/modernize/UseNullptrCheck.cpp (+1-1) - (modified) clang-tools-extra/clang-tidy/modernize/UseStringViewCheck.cpp (+2-2) - (modified) clang-tools-extra/clang-tidy/performance/MoveConstArgCheck.cpp (+2-1) - (modified) clang-tools-extra/clang-tidy/performance/UnnecessaryCopyInitializationCheck.cpp (+4-4) - (modified) clang-tools-extra/clang-tidy/readability/ConvertMemberFunctionsToStaticCheck.cpp (+2-2) - (modified) clang-tools-extra/clang-tidy/readability/ElseAfterReturnCheck.cpp (+12-10) - (modified) clang-tools-extra/clang-tidy/readability/EnumInitialValueCheck.cpp (+6-5) - (modified) clang-tools-extra/clang-tidy/readability/ImplicitBoolConversionCheck.cpp (+4-4) - (modified) clang-tools-extra/clang-tidy/utils/FileExtensionsUtils.cpp (+2-2) - (modified) clang-tools-extra/clang-tidy/utils/FileExtensionsUtils.h (+2-2) - (modified) clang-tools-extra/clang-tidy/utils/FormatStringConverter.cpp (+1-1) - (modified) clang-tools-extra/clang-tidy/utils/UseRangesCheck.cpp (+3-3) ``````````diff diff --git a/clang-tools-extra/clang-tidy/ClangTidyCheck.cpp b/clang-tools-extra/clang-tidy/ClangTidyCheck.cpp index 30ba36737571a..26c4234c1b0ee 100644 --- a/clang-tools-extra/clang-tidy/ClangTidyCheck.cpp +++ b/clang-tools-extra/clang-tidy/ClangTidyCheck.cpp @@ -65,7 +65,7 @@ ClangTidyCheck::OptionsView::get(StringRef LocalName) const { static ClangTidyOptions::OptionMap::const_iterator findPriorityOption(const ClangTidyOptions::OptionMap &Options, StringRef NamePrefix, StringRef LocalName, - ClangTidyContext *Context) { + const ClangTidyContext *Context) { llvm::StringSet<> *Collector = Context->getOptionsCollector(); if (Collector) { Collector->insert((NamePrefix + LocalName).str()); diff --git a/clang-tools-extra/clang-tidy/bugprone/MacroParenthesesCheck.cpp b/clang-tools-extra/clang-tidy/bugprone/MacroParenthesesCheck.cpp index e11b153e0250b..a51bce1484a42 100644 --- a/clang-tools-extra/clang-tidy/bugprone/MacroParenthesesCheck.cpp +++ b/clang-tools-extra/clang-tidy/bugprone/MacroParenthesesCheck.cpp @@ -99,7 +99,7 @@ static bool possibleVarDecl(const MacroInfo *MI, const Token *Tok) { isVarDeclKeyword(*Tok); } -static StringRef getMacroText(const MacroInfo *MI, Preprocessor *PP) { +static StringRef getMacroText(const MacroInfo *MI, const Preprocessor *PP) { if (MI->tokens_empty()) return {}; return Lexer::getSourceText( diff --git a/clang-tools-extra/clang-tidy/bugprone/MoveForwardingReferenceCheck.cpp b/clang-tools-extra/clang-tidy/bugprone/MoveForwardingReferenceCheck.cpp index deb17880aaf8d..7e545096aed67 100644 --- a/clang-tools-extra/clang-tidy/bugprone/MoveForwardingReferenceCheck.cpp +++ b/clang-tools-extra/clang-tidy/bugprone/MoveForwardingReferenceCheck.cpp @@ -24,7 +24,7 @@ AST_MATCHER(DeclRefExpr, refersToEnclosingVariableOrCapture) { static void replaceMoveWithForward(const UnresolvedLookupExpr *Callee, const ParmVarDecl *ParmVar, const TemplateTypeParmDecl *TypeParmDecl, - DiagnosticBuilder &Diag, + const DiagnosticBuilder &Diag, const ASTContext &Context) { const SourceManager &SM = Context.getSourceManager(); const LangOptions &LangOpts = Context.getLangOpts(); diff --git a/clang-tools-extra/clang-tidy/bugprone/NotNullTerminatedResultCheck.cpp b/clang-tools-extra/clang-tidy/bugprone/NotNullTerminatedResultCheck.cpp index 77b21d84529ce..8359a393917e5 100644 --- a/clang-tools-extra/clang-tidy/bugprone/NotNullTerminatedResultCheck.cpp +++ b/clang-tools-extra/clang-tidy/bugprone/NotNullTerminatedResultCheck.cpp @@ -293,7 +293,7 @@ isFixedGivenLengthAndUnknownSrc(const MatchFinder::MatchResult &Result) { static void lengthExprHandle(const Expr *LengthExpr, LengthHandleKind LengthHandle, const MatchFinder::MatchResult &Result, - DiagnosticBuilder &Diag) { + const DiagnosticBuilder &Diag) { LengthExpr = LengthExpr->IgnoreParenImpCasts(); // See whether we work with a macro. @@ -360,14 +360,14 @@ static void lengthExprHandle(const Expr *LengthExpr, static void lengthArgHandle(LengthHandleKind LengthHandle, const MatchFinder::MatchResult &Result, - DiagnosticBuilder &Diag) { + const DiagnosticBuilder &Diag) { const auto *LengthExpr = Result.Nodes.getNodeAs<Expr>(LengthExprName); lengthExprHandle(LengthExpr, LengthHandle, Result, Diag); } static void lengthArgPosHandle(unsigned ArgPos, LengthHandleKind LengthHandle, const MatchFinder::MatchResult &Result, - DiagnosticBuilder &Diag) { + const DiagnosticBuilder &Diag) { const auto *FunctionExpr = Result.Nodes.getNodeAs<CallExpr>(FunctionExprName); lengthExprHandle(FunctionExpr->getArg(ArgPos), LengthHandle, Result, Diag); } @@ -375,7 +375,7 @@ static void lengthArgPosHandle(unsigned ArgPos, LengthHandleKind LengthHandle, // The string handler functions are only operates with plain 'char'/'wchar_t' // without 'unsigned/signed', therefore we need to cast it. static bool isDestExprFix(const MatchFinder::MatchResult &Result, - DiagnosticBuilder &Diag) { + const DiagnosticBuilder &Diag) { const auto *Dest = Result.Nodes.getNodeAs<Expr>(DestExprName); if (!Dest) return false; @@ -392,7 +392,7 @@ static bool isDestExprFix(const MatchFinder::MatchResult &Result, // If the destination array is the same length as the given length we have to // increase the capacity by one to create space for the null terminator. static bool isDestCapacityFix(const MatchFinder::MatchResult &Result, - DiagnosticBuilder &Diag) { + const DiagnosticBuilder &Diag) { const bool IsOverflows = isDestCapacityOverflows(Result); if (IsOverflows) if (const Expr *CapacityExpr = getDestCapacityExpr(Result)) @@ -402,7 +402,7 @@ static bool isDestCapacityFix(const MatchFinder::MatchResult &Result, } static void removeArg(int ArgPos, const MatchFinder::MatchResult &Result, - DiagnosticBuilder &Diag) { + const DiagnosticBuilder &Diag) { // This is the following structure: (src, '\0', strlen(src)) // ArgToRemove: ~~~~~~~~~~~ // LHSArg: ~~~~ @@ -418,7 +418,7 @@ static void removeArg(int ArgPos, const MatchFinder::MatchResult &Result, static void renameFunc(StringRef NewFuncName, const MatchFinder::MatchResult &Result, - DiagnosticBuilder &Diag) { + const DiagnosticBuilder &Diag) { const auto *FunctionExpr = Result.Nodes.getNodeAs<CallExpr>(FunctionExprName); const int FuncNameLength = FunctionExpr->getDirectCallee()->getIdentifier()->getLength(); @@ -433,7 +433,7 @@ static void renameFunc(StringRef NewFuncName, static void renameMemcpy(StringRef Name, bool IsCopy, bool IsSafe, const MatchFinder::MatchResult &Result, - DiagnosticBuilder &Diag) { + const DiagnosticBuilder &Diag) { SmallString<10> NewFuncName; NewFuncName = (Name[0] != 'w') ? "str" : "wcs"; NewFuncName += IsCopy ? "cpy" : "ncpy"; @@ -443,7 +443,7 @@ static void renameMemcpy(StringRef Name, bool IsCopy, bool IsSafe, static void insertDestCapacityArg(bool IsOverflows, StringRef Name, const MatchFinder::MatchResult &Result, - DiagnosticBuilder &Diag) { + const DiagnosticBuilder &Diag) { const auto *FunctionExpr = Result.Nodes.getNodeAs<CallExpr>(FunctionExprName); SmallString<64> NewSecondArg; @@ -464,7 +464,7 @@ static void insertDestCapacityArg(bool IsOverflows, StringRef Name, static void insertNullTerminatorExpr(StringRef Name, const MatchFinder::MatchResult &Result, - DiagnosticBuilder &Diag) { + const DiagnosticBuilder &Diag) { const auto *FunctionExpr = Result.Nodes.getNodeAs<CallExpr>(FunctionExprName); const int FuncLocStartColumn = Result.SourceManager->getPresumedColumnNumber( FunctionExpr->getBeginLoc()); diff --git a/clang-tools-extra/clang-tidy/bugprone/ParentVirtualCallCheck.cpp b/clang-tools-extra/clang-tidy/bugprone/ParentVirtualCallCheck.cpp index 6c7e237331571..4184f6d89d768 100644 --- a/clang-tools-extra/clang-tidy/bugprone/ParentVirtualCallCheck.cpp +++ b/clang-tools-extra/clang-tidy/bugprone/ParentVirtualCallCheck.cpp @@ -65,7 +65,7 @@ static std::string getNameAsString(const NamedDecl *Decl) { // Returns E as written in the source code. Used to handle 'using' and // 'typedef'ed names of grand-parent classes. -static std::string getExprAsString(const Expr &E, ASTContext &AC) { +static std::string getExprAsString(const Expr &E, const ASTContext &AC) { std::string Text = tooling::fixit::getText(E, AC).str(); llvm::erase_if(Text, [](char C) { return llvm::isSpace(static_cast<unsigned char>(C)); diff --git a/clang-tools-extra/clang-tidy/bugprone/UseAfterMoveCheck.cpp b/clang-tools-extra/clang-tidy/bugprone/UseAfterMoveCheck.cpp index d3cf2923eaf83..f8555aa251165 100644 --- a/clang-tools-extra/clang-tidy/bugprone/UseAfterMoveCheck.cpp +++ b/clang-tools-extra/clang-tidy/bugprone/UseAfterMoveCheck.cpp @@ -509,7 +509,7 @@ static MoveType determineMoveType(const FunctionDecl *FuncDecl) { static void emitDiagnostic(const Expr *MovingCall, const DeclRefExpr *MoveArg, const UseAfterMove &Use, ClangTidyCheck *Check, - ASTContext *Context, MoveType Type, + const ASTContext *Context, MoveType Type, const FunctionDecl *MoveDecl) { const SourceLocation UseLoc = Use.DeclRef->getExprLoc(); const SourceLocation MoveLoc = MovingCall->getExprLoc(); diff --git a/clang-tools-extra/clang-tidy/llvmlibc/InlineFunctionDeclCheck.cpp b/clang-tools-extra/clang-tidy/llvmlibc/InlineFunctionDeclCheck.cpp index f00b07019d21e..69393781918e1 100644 --- a/clang-tools-extra/clang-tidy/llvmlibc/InlineFunctionDeclCheck.cpp +++ b/clang-tools-extra/clang-tidy/llvmlibc/InlineFunctionDeclCheck.cpp @@ -22,7 +22,7 @@ getLastTemplateParameterList(const FunctionDecl *FuncDecl) { FuncDecl->getDescribedTemplateParams(); if (!ReturnList) { - ArrayRef<TemplateParameterList *> TPLs = + const ArrayRef<TemplateParameterList *> TPLs = FuncDecl->getTemplateParameterLists(); if (!TPLs.empty()) diff --git a/clang-tools-extra/clang-tidy/misc/ConstCorrectnessCheck.cpp b/clang-tools-extra/clang-tidy/misc/ConstCorrectnessCheck.cpp index f07b49444f3e2..6964e1b3652ea 100644 --- a/clang-tools-extra/clang-tidy/misc/ConstCorrectnessCheck.cpp +++ b/clang-tools-extra/clang-tidy/misc/ConstCorrectnessCheck.cpp @@ -171,9 +171,10 @@ void ConstCorrectnessCheck::registerMatchers(MatchFinder *Finder) { } } -static void addConstFixits(DiagnosticBuilder &Diag, const VarDecl *Variable, - const FunctionDecl *Function, ASTContext &Context, - Qualifiers::TQ Qualifier, +static void addConstFixits(const DiagnosticBuilder &Diag, + const VarDecl *Variable, + const FunctionDecl *Function, + const ASTContext &Context, Qualifiers::TQ Qualifier, utils::fixit::QualifierTarget Target, utils::fixit::QualifierPolicy Policy) { // If this is a parameter, also add fixits for corresponding parameters in diff --git a/clang-tools-extra/clang-tidy/misc/MisplacedConstCheck.cpp b/clang-tools-extra/clang-tidy/misc/MisplacedConstCheck.cpp index c8c0cfd1c6ad5..a7daad4838c9d 100644 --- a/clang-tools-extra/clang-tidy/misc/MisplacedConstCheck.cpp +++ b/clang-tools-extra/clang-tidy/misc/MisplacedConstCheck.cpp @@ -30,7 +30,8 @@ void MisplacedConstCheck::registerMatchers(MatchFinder *Finder) { this); } -static QualType guessAlternateQualification(ASTContext &Context, QualType QT) { +static QualType guessAlternateQualification(const ASTContext &Context, + QualType QT) { // We're given a QualType from a typedef where the qualifiers apply to the // pointer instead of the pointee. Strip the const qualifier from the pointer // type and add it to the pointee instead. @@ -47,7 +48,7 @@ static QualType guessAlternateQualification(ASTContext &Context, QualType QT) { void MisplacedConstCheck::check(const MatchFinder::MatchResult &Result) { const auto *Var = Result.Nodes.getNodeAs<ValueDecl>("decl"); - ASTContext &Ctx = *Result.Context; + const ASTContext &Ctx = *Result.Context; const QualType CanQT = Var->getType().getCanonicalType(); SourceLocation AliasLoc; diff --git a/clang-tools-extra/clang-tidy/misc/NoRecursionCheck.cpp b/clang-tools-extra/clang-tidy/misc/NoRecursionCheck.cpp index 00b48ca59553c..ca269fd1d543c 100644 --- a/clang-tools-extra/clang-tidy/misc/NoRecursionCheck.cpp +++ b/clang-tools-extra/clang-tidy/misc/NoRecursionCheck.cpp @@ -40,10 +40,10 @@ static CallStackTy pathfindSomeCycle(ArrayRef<CallGraphNode *> SCC) { CallStackSet; // Arbitrarily take the first element of SCC as entry point. - CallGraphNode::CallRecord EntryNode(SCC.front(), /*CallExpr=*/nullptr); + const CallGraphNode::CallRecord EntryNode(SCC.front(), /*CallExpr=*/nullptr); // Continue recursing into subsequent callees that are part of this SCC, // and are thus known to be part of the call graph loop, until loop forms. - CallGraphNode::CallRecord *Node = &EntryNode; + const CallGraphNode::CallRecord *Node = &EntryNode; while (true) { // Did we see this node before? if (!CallStackSet.insert(*Node)) diff --git a/clang-tools-extra/clang-tidy/misc/UseInternalLinkageCheck.cpp b/clang-tools-extra/clang-tidy/misc/UseInternalLinkageCheck.cpp index db24f740148b1..adbe8d75ba5aa 100644 --- a/clang-tools-extra/clang-tidy/misc/UseInternalLinkageCheck.cpp +++ b/clang-tools-extra/clang-tidy/misc/UseInternalLinkageCheck.cpp @@ -44,7 +44,7 @@ struct OptionEnumMapping<misc::UseInternalLinkageCheck::FixModeKind> { namespace clang::tidy::misc { -static bool isInMainFile(SourceLocation L, SourceManager &SM, +static bool isInMainFile(SourceLocation L, const SourceManager &SM, const FileExtensionsSet &HeaderFileExtensions) { for (;;) { if (utils::isExpansionLocInHeaderFile(L, SM, HeaderFileExtensions)) diff --git a/clang-tools-extra/clang-tidy/modernize/LoopConvertCheck.cpp b/clang-tools-extra/clang-tidy/modernize/LoopConvertCheck.cpp index c21a0d54159ec..b0adf971ea4a6 100644 --- a/clang-tools-extra/clang-tidy/modernize/LoopConvertCheck.cpp +++ b/clang-tools-extra/clang-tidy/modernize/LoopConvertCheck.cpp @@ -432,8 +432,8 @@ getContainerFromBeginEndCall(const Expr *Init, bool IsBegin, bool *IsArrow, /// /// BeginExpr must be a member call to a function named "begin()", and EndExpr /// must be a member. -static const Expr *findContainer(ASTContext *Context, const Expr *BeginExpr, - const Expr *EndExpr, +static const Expr *findContainer(const ASTContext *Context, + const Expr *BeginExpr, const Expr *EndExpr, bool *ContainerNeedsDereference, bool IsReverse) { // Now that we know the loop variable and test expression, make sure they are @@ -461,7 +461,7 @@ static const Expr *findContainer(ASTContext *Context, const Expr *BeginExpr, } /// Obtain the original source code text from a SourceRange. -static StringRef getStringFromRange(SourceManager &SourceMgr, +static StringRef getStringFromRange(const SourceManager &SourceMgr, const LangOptions &LangOpts, SourceRange Range) { if (SourceMgr.getFileID(Range.getBegin()) != diff --git a/clang-tools-extra/clang-tidy/modernize/LoopConvertUtils.cpp b/clang-tools-extra/clang-tidy/modernize/LoopConvertUtils.cpp index e9932ba444bc9..ae97a7e0a0266 100644 --- a/clang-tools-extra/clang-tidy/modernize/LoopConvertUtils.cpp +++ b/clang-tools-extra/clang-tidy/modernize/LoopConvertUtils.cpp @@ -186,7 +186,8 @@ const Expr *digThroughConstructorsConversions(const Expr *E) { } /// Returns true when two Exprs are equivalent. -bool areSameExpr(ASTContext *Context, const Expr *First, const Expr *Second) { +bool areSameExpr(const ASTContext *Context, const Expr *First, + const Expr *Second) { return utils::areStatementsIdentical(First, Second, *Context, true); } @@ -277,7 +278,8 @@ static bool isIndexInSubscriptExpr(const Expr *IndexExpr, /// (*container)[index] /// (*container).at(index) /// \endcode -static bool isIndexInSubscriptExpr(ASTContext *Context, const Expr *IndexExpr, +static bool isIndexInSubscriptExpr(const ASTContext *Context, + const Expr *IndexExpr, const VarDecl *IndexVar, const Expr *Obj, const Expr *SourceExpr, bool PermitDeref) { if (!SourceExpr || !Obj || !isIndexInSubscriptExpr(IndexExpr, IndexVar)) @@ -339,7 +341,7 @@ static bool isDereferenceOfUop(const UnaryOperator *Uop, /// // use t, do not use i /// } /// \endcode -static bool isAliasDecl(ASTContext *Context, const Decl *TheDecl, +static bool isAliasDecl(const ASTContext *Context, const Decl *TheDecl, const VarDecl *IndexVar) { const auto *VDecl = dyn_cast<VarDecl>(TheDecl); if (!VDecl) @@ -422,7 +424,7 @@ static bool isAliasDecl(ASTContext *Context, const Decl *TheDecl, /// for (int i = 0; i < N; ++i) { /* use arr[i] */ } /// for (int i = 0; i < arraysize(arr); ++i) { /* use arr[i] */ } /// \endcode -static bool arrayMatchesBoundExpr(ASTContext *Context, +static bool arrayMatchesBoundExpr(const ASTContext *Context, const QualType &ArrayType, const Expr *ConditionExpr) { if (!ConditionExpr || ConditionExpr->isValueDependent()) diff --git a/clang-tools-extra/clang-tidy/modernize/LoopConvertUtils.h b/clang-tools-extra/clang-tidy/modernize/LoopConvertUtils.h index 0c295fd5d0820..1415af6061783 100644 --- a/clang-tools-extra/clang-tidy/modernize/LoopConvertUtils.h +++ b/clang-tools-extra/clang-tidy/modernize/LoopConvertUtils.h @@ -268,7 +268,8 @@ using UsageResult = SmallVector<Usage, 8>; // General functions used by ForLoopIndexUseVisitor and LoopConvertCheck. const Expr *digThroughConstructorsConversions(const Expr *E); -bool areSameExpr(ASTContext *Context, const Expr *First, const Expr *Second); +bool areSameExpr(const ASTContext *Context, const Expr *First, + const Expr *Second); const DeclRefExpr *getDeclRef(const Expr *E); bool areSameVariable(const ValueDecl *First, const ValueDecl *Second); diff --git a/clang-tools-extra/clang-tidy/modernize/UseNullptrCheck.cpp b/clang-tools-extra/clang-tidy/modernize/UseNullptrCheck.cpp index 85224b573edeb..5fe62915c062e 100644 --- a/clang-tools-extra/clang-tidy/modernize/UseNullptrCheck.cpp +++ b/clang-tools-extra/clang-tidy/modernize/UseNullptrCheck.cpp @@ -91,7 +91,7 @@ static bool isReplaceableRange(SourceLocation StartLoc, SourceLocation EndLoc, /// Replaces the provided range with the text "nullptr", but only if /// the start and end location are both in main file. /// Returns true if and only if a replacement was made. -static void replaceWithNullptr(ClangTidyCheck &Check, SourceManager &SM, +static void replaceWithNullptr(ClangTidyCheck &Check, const SourceManager &SM, SourceLocation StartLoc, SourceLocation EndLoc) { const CharSourceRange Range(SourceRange(StartLoc, EndLoc), true); // Add a space if nullptr follows an alphanumeric character. This happens diff --git a/clang-tools-extra/clang-tidy/modernize/UseStringViewCheck.cpp b/clang-tools-extra/clang-tidy/modernize/UseStringViewCheck.cpp index 9892870279b55..5209846aca567 100644 --- a/clang-tools-extra/clang-tidy/modernize/UseStringViewCheck.cpp +++ b/clang-tools-extra/clang-tidy/modernize/UseStringViewCheck.cpp @@ -63,8 +63,8 @@ static auto getStringTypeMatcher(StringRef CharType) { return hasCanonicalType(hasDeclaration(cxxRecordDecl(hasName(CharType)))); } -static void fixReturns(const FunctionDecl *FuncDecl, DiagnosticBuilder &Diag, - ASTContext &Context) { +static void fixReturns(const FunctionDecl *FuncDecl, + const DiagnosticBuilder &Diag, ASTContext &Context) { auto Matches = match( findAll(returnStmt(hasReturnValue(ignoringParenImpCasts( cxxTemporaryObjectExpr(argumentCountIs(0)).bind("temp_obj_expr"))))), diff --git a/clang-tools-extra/clang-tidy/performance/MoveConstArgCheck.cpp b/clang-tools-extra/cl... [truncated] `````````` </details> https://github.com/llvm/llvm-project/pull/203823 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
