Author: Kazu Hirata Date: 2025-05-17T12:59:47-07:00 New Revision: f9f69dac2a63afd9b317033bfe055a174bdd180f
URL: https://github.com/llvm/llvm-project/commit/f9f69dac2a63afd9b317033bfe055a174bdd180f DIFF: https://github.com/llvm/llvm-project/commit/f9f69dac2a63afd9b317033bfe055a174bdd180f.diff LOG: [clang] Remove redundant control flow statements (NFC) (#140359) Added: Modified: clang/lib/AST/DeclTemplate.cpp clang/lib/CodeGen/CGExprAgg.cpp clang/lib/CodeGen/CGOpenMPRuntimeGPU.cpp clang/lib/Driver/ToolChains/CommonArgs.cpp clang/lib/InstallAPI/DylibVerifier.cpp clang/lib/Sema/SemaDeclAttr.cpp clang/lib/Sema/SemaExpr.cpp clang/lib/Sema/SemaInit.cpp clang/lib/Sema/SemaStmtAttr.cpp Removed: ################################################################################ diff --git a/clang/lib/AST/DeclTemplate.cpp b/clang/lib/AST/DeclTemplate.cpp index b951e68b0a1b8..0f4ae368ae90d 100644 --- a/clang/lib/AST/DeclTemplate.cpp +++ b/clang/lib/AST/DeclTemplate.cpp @@ -359,7 +359,6 @@ void RedeclarableTemplateDecl::loadLazySpecializationsImpl( ExternalSource->LoadExternalSpecializations(this->getCanonicalDecl(), OnlyPartial); - return; } bool RedeclarableTemplateDecl::loadLazySpecializationsImpl( diff --git a/clang/lib/CodeGen/CGExprAgg.cpp b/clang/lib/CodeGen/CGExprAgg.cpp index 7917cdb7e12b6..e6f18505a7d4d 100644 --- a/clang/lib/CodeGen/CGExprAgg.cpp +++ b/clang/lib/CodeGen/CGExprAgg.cpp @@ -548,7 +548,6 @@ static void EmitHLSLScalarFlatCast(CodeGenFunction &CGF, Address DestVal, } CGF.Builder.CreateStore(Cast, StoreGEPList[I].first); } - return; } // emit a flat cast where the RHS is an aggregate diff --git a/clang/lib/CodeGen/CGOpenMPRuntimeGPU.cpp b/clang/lib/CodeGen/CGOpenMPRuntimeGPU.cpp index aa97422d54ede..6f819640a9521 100644 --- a/clang/lib/CodeGen/CGOpenMPRuntimeGPU.cpp +++ b/clang/lib/CodeGen/CGOpenMPRuntimeGPU.cpp @@ -1759,7 +1759,6 @@ void CGOpenMPRuntimeGPU::emitReduction( CGF.getTarget().getGridValue(), C.getLangOpts().OpenMPCUDAReductionBufNum, RTLoc)); CGF.Builder.restoreIP(AfterIP); - return; } const VarDecl * diff --git a/clang/lib/Driver/ToolChains/CommonArgs.cpp b/clang/lib/Driver/ToolChains/CommonArgs.cpp index 5c1bc090810a2..632027c4a944c 100644 --- a/clang/lib/Driver/ToolChains/CommonArgs.cpp +++ b/clang/lib/Driver/ToolChains/CommonArgs.cpp @@ -2576,8 +2576,6 @@ static void GetSDLFromOffloadArchive( InputInfo(&JA, C.getArgs().MakeArgString(OutputLib)))); CC1Args.push_back(DriverArgs.MakeArgString(OutputLib)); - - return; } // Wrapper function used by driver for adding SDLs during link phase. diff --git a/clang/lib/InstallAPI/DylibVerifier.cpp b/clang/lib/InstallAPI/DylibVerifier.cpp index 45c84c00d9236..d21c4a0754894 100644 --- a/clang/lib/InstallAPI/DylibVerifier.cpp +++ b/clang/lib/InstallAPI/DylibVerifier.cpp @@ -750,7 +750,6 @@ void DylibVerifier::visitSymbolInDylib(const Record &R, SymbolContext &SymCtx) { // Missing declarations are dropped for ErrorsOnly mode. It is the last // remaining mode. updateState(Result::Ignore); - return; } void DylibVerifier::visitGlobal(const GlobalRecord &R) { diff --git a/clang/lib/Sema/SemaDeclAttr.cpp b/clang/lib/Sema/SemaDeclAttr.cpp index 377595639bef1..4d7f0455444f1 100644 --- a/clang/lib/Sema/SemaDeclAttr.cpp +++ b/clang/lib/Sema/SemaDeclAttr.cpp @@ -3982,7 +3982,6 @@ static void handleFormatMatchesAttr(Sema &S, Decl *D, const ParsedAttr &AL) { S.Diag(AL.getLoc(), diag::err_format_nonliteral) << FormatStrExpr->getSourceRange(); - return; } /// Handle __attribute__((callback(CalleeIdx, PayloadIdx0, ...))) attributes. diff --git a/clang/lib/Sema/SemaExpr.cpp b/clang/lib/Sema/SemaExpr.cpp index 91e63c7cb8677..d1889100c382e 100644 --- a/clang/lib/Sema/SemaExpr.cpp +++ b/clang/lib/Sema/SemaExpr.cpp @@ -1639,7 +1639,6 @@ static void CheckUnicodeArithmeticConversions(Sema &SemaRef, Expr *LHS, SemaRef.Diag(Loc, diag::warn_arith_conv_mixed_unicode_types) << LHS->getSourceRange() << RHS->getSourceRange() << ACK << LHSType << RHSType; - return; } /// UsualArithmeticConversions - Performs various conversions that are common to diff --git a/clang/lib/Sema/SemaInit.cpp b/clang/lib/Sema/SemaInit.cpp index adce0a15bc320..9ee8603ff7811 100644 --- a/clang/lib/Sema/SemaInit.cpp +++ b/clang/lib/Sema/SemaInit.cpp @@ -9853,7 +9853,6 @@ static void CheckC23ConstexprInitStringLiteral(const StringLiteral *SE, return; } } - return; } //===----------------------------------------------------------------------===// diff --git a/clang/lib/Sema/SemaStmtAttr.cpp b/clang/lib/Sema/SemaStmtAttr.cpp index a09626c3a9a8c..cf1c597e90596 100644 --- a/clang/lib/Sema/SemaStmtAttr.cpp +++ b/clang/lib/Sema/SemaStmtAttr.cpp @@ -428,7 +428,6 @@ static void CheckForDuplicateLoopAttrs(Sema &S, ArrayRef<const Attr *> Attrs) { S.Diag((*FirstItr)->getLocation(), diag::note_previous_attribute); } } - return; } static Attr *handleMSConstexprAttr(Sema &S, Stmt *St, const ParsedAttr &A, _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
