Author: Jens Massberg Date: 2023-06-07T18:06:19+02:00 New Revision: 0a7ca80360584ff35cb58716e08855a913a3f4d0
URL: https://github.com/llvm/llvm-project/commit/0a7ca80360584ff35cb58716e08855a913a3f4d0 DIFF: https://github.com/llvm/llvm-project/commit/0a7ca80360584ff35cb58716e08855a913a3f4d0.diff LOG: Fix parameter name in Sema::addInitCapture to ByRef. Rename parameter in Sema::addInitCapture as proposed in review of Sema::addInitCapture. Sorry, that I have missed the comment there! Reviewed By: ilya-biryukov Differential Revision: https://reviews.llvm.org/D139541 Added: Modified: clang/include/clang/Sema/Sema.h clang/lib/Sema/SemaLambda.cpp Removed: ################################################################################ diff --git a/clang/include/clang/Sema/Sema.h b/clang/include/clang/Sema/Sema.h index cf25f39de0d19..a15846952fd01 100644 --- a/clang/include/clang/Sema/Sema.h +++ b/clang/include/clang/Sema/Sema.h @@ -7171,8 +7171,7 @@ class Sema final { IdentifierInfo *Id, unsigned InitStyle, Expr *Init, DeclContext *DeclCtx); /// Add an init-capture to a lambda scope. - void addInitCapture(sema::LambdaScopeInfo *LSI, VarDecl *Var, - bool isReferenceType); + void addInitCapture(sema::LambdaScopeInfo *LSI, VarDecl *Var, bool ByRef); /// Note that we have finished the explicit captures for the /// given lambda. diff --git a/clang/lib/Sema/SemaLambda.cpp b/clang/lib/Sema/SemaLambda.cpp index 730f70e732579..c1d70ad8b08f9 100644 --- a/clang/lib/Sema/SemaLambda.cpp +++ b/clang/lib/Sema/SemaLambda.cpp @@ -798,12 +798,11 @@ VarDecl *Sema::createLambdaInitCaptureVarDecl( return NewVD; } -void Sema::addInitCapture(LambdaScopeInfo *LSI, VarDecl *Var, - bool isReferenceType) { +void Sema::addInitCapture(LambdaScopeInfo *LSI, VarDecl *Var, bool ByRef) { assert(Var->isInitCapture() && "init capture flag should be set"); - LSI->addCapture(Var, /*isBlock*/ false, isReferenceType, - /*isNested*/ false, Var->getLocation(), SourceLocation(), - Var->getType(), /*Invalid*/ false); + LSI->addCapture(Var, /*isBlock=*/false, ByRef, + /*isNested=*/false, Var->getLocation(), SourceLocation(), + Var->getType(), /*Invalid=*/false); } // Unlike getCurLambda, getCurrentLambdaScopeUnsafe doesn't _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits