diff --git a/clang-tools-extra/clang-tidy/modernize/LoopConvertUtils.cpp b/clang-tools-extra/clang-tidy/modernize/LoopConvertUtils.cpp --- a/clang-tools-extra/clang-tidy/modernize/LoopConvertUtils.cpp +++ b/clang-tools-extra/clang-tidy/modernize/LoopConvertUtils.cpp @@ -785,8 +785,8 @@ const LambdaCapture *C, Expr *Init) { if (C->capturesVariable()) { - const VarDecl *VDecl = C->getCapturedVar(); - if (areSameVariable(IndexVar, cast(VDecl))) { + const ValueDecl *VDecl = C->getCapturedVar(); + if (areSameVariable(IndexVar, VDecl)) { // FIXME: if the index is captured, it will count as an usage and the // alias (if any) won't work, because it is only used in case of having // exactly one usage. diff --git a/clang/docs/ReleaseNotes.rst b/clang/docs/ReleaseNotes.rst --- a/clang/docs/ReleaseNotes.rst +++ b/clang/docs/ReleaseNotes.rst @@ -431,6 +431,9 @@ that can be used for such compatibility. The demangler now demangles symbols with named module attachment. +- Support capturing structured bindings in lambdas + (`P1091R3 `_ and `P1381R1 `) + C++2b Feature Support ^^^^^^^^^^^^^^^^^^^^^ diff --git a/clang/include/clang/AST/DeclCXX.h b/clang/include/clang/AST/DeclCXX.h --- a/clang/include/clang/AST/DeclCXX.h +++ b/clang/include/clang/AST/DeclCXX.h @@ -1057,8 +1057,9 @@ /// /// \note No entries will be added for init-captures, as they do not capture /// variables. - void getCaptureFields(llvm::DenseMap &Captures, - FieldDecl *&ThisCapture) const; + void + getCaptureFields(llvm::DenseMap &Captures, + FieldDecl *&ThisCapture) const; using capture_const_iterator = const LambdaCapture *; using capture_const_range = llvm::iterator_range; diff --git a/clang/include/clang/AST/LambdaCapture.h b/clang/include/clang/AST/LambdaCapture.h --- a/clang/include/clang/AST/LambdaCapture.h +++ b/clang/include/clang/AST/LambdaCapture.h @@ -71,7 +71,7 @@ /// capture that is a pack expansion, or an invalid source /// location to indicate that this is not a pack expansion. LambdaCapture(SourceLocation Loc, bool Implicit, LambdaCaptureKind Kind, - VarDecl *Var = nullptr, + ValueDecl *Var = nullptr, SourceLocation EllipsisLoc = SourceLocation()); /// Determine the kind of capture. @@ -86,7 +86,7 @@ /// Determine whether this capture handles a variable. bool capturesVariable() const { - return isa_and_nonnull(DeclAndBits.getPointer()); + return isa_and_nonnull(DeclAndBits.getPointer()); } /// Determine whether this captures a variable length array bound @@ -101,9 +101,9 @@ /// /// This operation is only valid if this capture is a variable capture /// (other than a capture of \c this). - VarDecl *getCapturedVar() const { + ValueDecl *getCapturedVar() const { assert(capturesVariable() && "No variable available for capture"); - return static_cast(DeclAndBits.getPointer()); + return static_cast(DeclAndBits.getPointer()); } /// Determine whether this was an implicit capture (not diff --git a/clang/include/clang/AST/Stmt.h b/clang/include/clang/AST/Stmt.h --- a/clang/include/clang/AST/Stmt.h +++ b/clang/include/clang/AST/Stmt.h @@ -59,6 +59,7 @@ class StringLiteral; class Token; class VarDecl; +class ValueDecl; //===----------------------------------------------------------------------===// // AST classes for statements. diff --git a/clang/include/clang/ASTMatchers/ASTMatchers.h b/clang/include/clang/ASTMatchers/ASTMatchers.h --- a/clang/include/clang/ASTMatchers/ASTMatchers.h +++ b/clang/include/clang/ASTMatchers/ASTMatchers.h @@ -4690,7 +4690,7 @@ /// In the matcher /// lambdaExpr(hasAnyCapture(lambdaCapture(capturesVar(hasName("x")))), /// capturesVar(hasName("x")) matches `x` and `x = 1`. -AST_MATCHER_P(LambdaCapture, capturesVar, internal::Matcher, +AST_MATCHER_P(LambdaCapture, capturesVar, internal::Matcher, InnerMatcher) { auto *capturedVar = Node.getCapturedVar(); return capturedVar && InnerMatcher.matches(*capturedVar, Finder, Builder); diff --git a/clang/include/clang/Basic/DiagnosticSemaKinds.td b/clang/include/clang/Basic/DiagnosticSemaKinds.td --- a/clang/include/clang/Basic/DiagnosticSemaKinds.td +++ b/clang/include/clang/Basic/DiagnosticSemaKinds.td @@ -9016,6 +9016,16 @@ def err_reference_to_local_in_enclosing_context : Error< "reference to local %select{variable|binding}1 %0 declared in enclosing " "%select{%3|block literal|lambda expression|context}2">; +def err_bitfield_capture_by_ref : Error< + "cannot capture a bit-field by reference">; +def err_capture_binding_openmp : Error< + "capturing a structured binding is not yet supported in OpenMP">; +def ext_capture_binding : ExtWarn< + "captured structured bindings are a C++20 extension">, InGroup; +def warn_cxx17_compat_capture_binding : Warning< + "captured structured bindings are incompatible with " + "C++ standards before C++20">, + InGroup, DefaultIgnore; def err_static_data_member_not_allowed_in_local_class : Error< "static data member %0 not allowed in local %sub{select_tag_type_kind}2 %1">; diff --git a/clang/include/clang/Sema/ScopeInfo.h b/clang/include/clang/Sema/ScopeInfo.h --- a/clang/include/clang/Sema/ScopeInfo.h +++ b/clang/include/clang/Sema/ScopeInfo.h @@ -548,7 +548,7 @@ const VariableArrayType *CapturedVLA; /// Otherwise, the captured variable (if any). - VarDecl *CapturedVar; + ValueDecl *CapturedVar; }; /// The source location at which the first capture occurred. @@ -584,12 +584,13 @@ unsigned Invalid : 1; public: - Capture(VarDecl *Var, bool Block, bool ByRef, bool IsNested, + Capture(ValueDecl *Var, bool Block, bool ByRef, bool IsNested, SourceLocation Loc, SourceLocation EllipsisLoc, QualType CaptureType, bool Invalid) : CapturedVar(Var), Loc(Loc), EllipsisLoc(EllipsisLoc), - CaptureType(CaptureType), - Kind(Block ? Cap_Block : ByRef ? Cap_ByRef : Cap_ByCopy), + CaptureType(CaptureType), Kind(Block ? Cap_Block + : ByRef ? Cap_ByRef + : Cap_ByCopy), Nested(IsNested), CapturesThis(false), ODRUsed(false), NonODRUsed(false), Invalid(Invalid) {} @@ -634,7 +635,7 @@ NonODRUsed = true; } - VarDecl *getVariable() const { + ValueDecl *getVariable() const { assert(isVariableCapture()); return CapturedVar; } @@ -673,7 +674,7 @@ : FunctionScopeInfo(Diag), ImpCaptureStyle(Style) {} /// CaptureMap - A map of captured variables to (index+1) into Captures. - llvm::DenseMap CaptureMap; + llvm::DenseMap CaptureMap; /// CXXThisCaptureIndex - The (index+1) of the capture of 'this'; /// zero if 'this' is not captured. @@ -690,7 +691,7 @@ /// or null if unknown. QualType ReturnType; - void addCapture(VarDecl *Var, bool isBlock, bool isByref, bool isNested, + void addCapture(ValueDecl *Var, bool isBlock, bool isByref, bool isNested, SourceLocation Loc, SourceLocation EllipsisLoc, QualType CaptureType, bool Invalid) { Captures.push_back(Capture(Var, isBlock, isByref, isNested, Loc, @@ -717,23 +718,21 @@ } /// Determine whether the given variable has been captured. - bool isCaptured(VarDecl *Var) const { - return CaptureMap.count(Var); - } + bool isCaptured(ValueDecl *Var) const { return CaptureMap.count(Var); } /// Determine whether the given variable-array type has been captured. bool isVLATypeCaptured(const VariableArrayType *VAT) const; /// Retrieve the capture of the given variable, if it has been /// captured already. - Capture &getCapture(VarDecl *Var) { + Capture &getCapture(ValueDecl *Var) { assert(isCaptured(Var) && "Variable has not been captured"); return Captures[CaptureMap[Var] - 1]; } - const Capture &getCapture(VarDecl *Var) const { - llvm::DenseMap::const_iterator Known - = CaptureMap.find(Var); + const Capture &getCapture(ValueDecl *Var) const { + llvm::DenseMap::const_iterator Known = + CaptureMap.find(Var); assert(Known != CaptureMap.end() && "Variable has not been captured"); return Captures[Known->second - 1]; } diff --git a/clang/include/clang/Sema/Sema.h b/clang/include/clang/Sema/Sema.h --- a/clang/include/clang/Sema/Sema.h +++ b/clang/include/clang/Sema/Sema.h @@ -5210,23 +5210,23 @@ /// /// \returns true if an error occurred (i.e., the variable cannot be /// captured) and false if the capture succeeded. - bool tryCaptureVariable(VarDecl *Var, SourceLocation Loc, TryCaptureKind Kind, - SourceLocation EllipsisLoc, bool BuildAndDiagnose, - QualType &CaptureType, + bool tryCaptureVariable(ValueDecl *Var, SourceLocation Loc, + TryCaptureKind Kind, SourceLocation EllipsisLoc, + bool BuildAndDiagnose, QualType &CaptureType, QualType &DeclRefType, const unsigned *const FunctionScopeIndexToStopAt); /// Try to capture the given variable. - bool tryCaptureVariable(VarDecl *Var, SourceLocation Loc, + bool tryCaptureVariable(ValueDecl *Var, SourceLocation Loc, TryCaptureKind Kind = TryCapture_Implicit, SourceLocation EllipsisLoc = SourceLocation()); /// Checks if the variable must be captured. - bool NeedToCaptureVariable(VarDecl *Var, SourceLocation Loc); + bool NeedToCaptureVariable(ValueDecl *Var, SourceLocation Loc); /// Given a variable, determine the type that a reference to that /// variable will have in the given scope. - QualType getCapturedDeclRefType(VarDecl *Var, SourceLocation Loc); + QualType getCapturedDeclRefType(ValueDecl *Var, SourceLocation Loc); /// Mark all of the declarations referenced within a particular AST node as /// referenced. Used when template instantiation instantiates a non-dependent diff --git a/clang/lib/AST/ASTImporter.cpp b/clang/lib/AST/ASTImporter.cpp --- a/clang/lib/AST/ASTImporter.cpp +++ b/clang/lib/AST/ASTImporter.cpp @@ -1007,7 +1007,7 @@ template <> Expected ASTNodeImporter::import(const LambdaCapture &From) { - VarDecl *Var = nullptr; + ValueDecl *Var = nullptr; if (From.capturesVariable()) { if (auto VarOrErr = import(From.getCapturedVar())) Var = *VarOrErr; diff --git a/clang/lib/AST/DeclCXX.cpp b/clang/lib/AST/DeclCXX.cpp --- a/clang/lib/AST/DeclCXX.cpp +++ b/clang/lib/AST/DeclCXX.cpp @@ -1556,8 +1556,8 @@ } void CXXRecordDecl::getCaptureFields( - llvm::DenseMap &Captures, - FieldDecl *&ThisCapture) const { + llvm::DenseMap &Captures, + FieldDecl *&ThisCapture) const { Captures.clear(); ThisCapture = nullptr; diff --git a/clang/lib/AST/ExprCXX.cpp b/clang/lib/AST/ExprCXX.cpp --- a/clang/lib/AST/ExprCXX.cpp +++ b/clang/lib/AST/ExprCXX.cpp @@ -1087,7 +1087,7 @@ : Expr(SC, Empty), NumArgs(NumArgs) {} LambdaCapture::LambdaCapture(SourceLocation Loc, bool Implicit, - LambdaCaptureKind Kind, VarDecl *Var, + LambdaCaptureKind Kind, ValueDecl *Var, SourceLocation EllipsisLoc) : DeclAndBits(Var, 0), Loc(Loc), EllipsisLoc(EllipsisLoc) { unsigned Bits = 0; @@ -1211,7 +1211,8 @@ } bool LambdaExpr::isInitCapture(const LambdaCapture *C) const { - return (C->capturesVariable() && C->getCapturedVar()->isInitCapture() && + return (C->capturesVariable() && isa(C->getCapturedVar()) && + cast(C->getCapturedVar())->isInitCapture() && (getCallOperator() == C->getCapturedVar()->getDeclContext())); } diff --git a/clang/lib/AST/ExprConstant.cpp b/clang/lib/AST/ExprConstant.cpp --- a/clang/lib/AST/ExprConstant.cpp +++ b/clang/lib/AST/ExprConstant.cpp @@ -578,7 +578,7 @@ /// LambdaCaptureFields - Mapping from captured variables/this to /// corresponding data members in the closure class. - llvm::DenseMap LambdaCaptureFields; + llvm::DenseMap LambdaCaptureFields; FieldDecl *LambdaThisCaptureField; CallStackFrame(EvalInfo &Info, SourceLocation CallLoc, diff --git a/clang/lib/AST/StmtPrinter.cpp b/clang/lib/AST/StmtPrinter.cpp --- a/clang/lib/AST/StmtPrinter.cpp +++ b/clang/lib/AST/StmtPrinter.cpp @@ -2072,7 +2072,7 @@ OS << "..."; if (Node->isInitCapture(C)) { - VarDecl *D = C->getCapturedVar(); + VarDecl *D = cast(C->getCapturedVar()); llvm::StringRef Pre; llvm::StringRef Post; diff --git a/clang/lib/Analysis/AnalysisDeclContext.cpp b/clang/lib/Analysis/AnalysisDeclContext.cpp --- a/clang/lib/Analysis/AnalysisDeclContext.cpp +++ b/clang/lib/Analysis/AnalysisDeclContext.cpp @@ -169,8 +169,8 @@ if (!LC.capturesVariable()) continue; - VarDecl *VD = LC.getCapturedVar(); - if (isSelfDecl(VD)) + ValueDecl *VD = LC.getCapturedVar(); + if (isSelfDecl(dyn_cast(VD))) return dyn_cast(VD); } diff --git a/clang/lib/CodeGen/CGDebugInfo.cpp b/clang/lib/CodeGen/CGDebugInfo.cpp --- a/clang/lib/CodeGen/CGDebugInfo.cpp +++ b/clang/lib/CodeGen/CGDebugInfo.cpp @@ -1495,7 +1495,7 @@ if (C.capturesVariable()) { SourceLocation Loc = C.getLocation(); assert(!Field->isBitField() && "lambdas don't have bitfield members!"); - VarDecl *V = C.getCapturedVar(); + ValueDecl *V = C.getCapturedVar(); StringRef VName = V->getName(); llvm::DIFile *VUnit = getOrCreateFile(Loc); auto Align = getDeclAlignIfRequired(V, CGM.getContext()); diff --git a/clang/lib/CodeGen/CGExpr.cpp b/clang/lib/CodeGen/CGExpr.cpp --- a/clang/lib/CodeGen/CGExpr.cpp +++ b/clang/lib/CodeGen/CGExpr.cpp @@ -2930,8 +2930,14 @@ // FIXME: While we're emitting a binding from an enclosing scope, all other // DeclRefExprs we see should be implicitly treated as if they also refer to // an enclosing scope. - if (const auto *BD = dyn_cast(ND)) + if (const auto *BD = dyn_cast(ND)) { + if (E->refersToEnclosingVariableOrCapture()) { + auto *FD = LambdaCaptureFields.lookup(BD); + assert(FD); + return EmitCapturedFieldLValue(*this, FD, CXXABIThisValue); + } return EmitLValue(BD->getBinding()); + } // We can form DeclRefExprs naming GUID declarations when reconstituting // non-type template parameters into expressions. diff --git a/clang/lib/CodeGen/CGOpenMPRuntime.cpp b/clang/lib/CodeGen/CGOpenMPRuntime.cpp --- a/clang/lib/CodeGen/CGOpenMPRuntime.cpp +++ b/clang/lib/CodeGen/CGOpenMPRuntime.cpp @@ -409,7 +409,7 @@ /// RAII for emitting code of OpenMP constructs. class InlinedOpenMPRegionRAII { CodeGenFunction &CGF; - llvm::DenseMap LambdaCaptureFields; + llvm::DenseMap LambdaCaptureFields; FieldDecl *LambdaThisCaptureField = nullptr; const CodeGen::CGBlockInfo *BlockInfo = nullptr; bool NoInheritance = false; @@ -8949,7 +8949,7 @@ Address VDAddr(Arg, CGF.ConvertTypeForMem(VDType), CGF.getContext().getDeclAlign(VD)); LValue VDLVal = CGF.MakeAddrLValue(VDAddr, VDType); - llvm::DenseMap Captures; + llvm::DenseMap Captures; FieldDecl *ThisCapture = nullptr; RD->getCaptureFields(Captures, ThisCapture); if (ThisCapture) { @@ -8971,7 +8971,7 @@ for (const LambdaCapture &LC : RD->captures()) { if (!LC.capturesVariable()) continue; - const VarDecl *VD = LC.getCapturedVar(); + const VarDecl *VD = cast(LC.getCapturedVar()); if (LC.getCaptureKind() != LCK_ByRef && !VD->getType()->isPointerType()) continue; auto It = Captures.find(VD); diff --git a/clang/lib/CodeGen/CGOpenMPRuntimeGPU.cpp b/clang/lib/CodeGen/CGOpenMPRuntimeGPU.cpp --- a/clang/lib/CodeGen/CGOpenMPRuntimeGPU.cpp +++ b/clang/lib/CodeGen/CGOpenMPRuntimeGPU.cpp @@ -3810,7 +3810,7 @@ else VDLVal = CGF.MakeAddrLValue( VDAddr, VD->getType().getCanonicalType().getNonReferenceType()); - llvm::DenseMap Captures; + llvm::DenseMap Captures; FieldDecl *ThisCapture = nullptr; RD->getCaptureFields(Captures, ThisCapture); if (ThisCapture && CGF.CapturedStmtInfo->isCXXThisExprCaptured()) { @@ -3822,13 +3822,15 @@ for (const LambdaCapture &LC : RD->captures()) { if (LC.getCaptureKind() != LCK_ByRef) continue; - const VarDecl *VD = LC.getCapturedVar(); - if (!CS->capturesVariable(VD)) + const ValueDecl *VD = LC.getCapturedVar(); + // FIXME: For now VD is always a VarDecl because OpenMP does not support + // capturing structured bindings in lambdas yet. + if (!CS->capturesVariable(cast(VD))) continue; auto It = Captures.find(VD); assert(It != Captures.end() && "Found lambda capture without field."); LValue VarLVal = CGF.EmitLValueForFieldInitialization(VDLVal, It->second); - Address VDAddr = CGF.GetAddrOfLocalVar(VD); + Address VDAddr = CGF.GetAddrOfLocalVar(cast(VD)); if (VD->getType().getCanonicalType()->isReferenceType()) VDAddr = CGF.EmitLoadOfReferenceLValue(VDAddr, VD->getType().getCanonicalType()) diff --git a/clang/lib/CodeGen/CodeGenFunction.h b/clang/lib/CodeGen/CodeGenFunction.h --- a/clang/lib/CodeGen/CodeGenFunction.h +++ b/clang/lib/CodeGen/CodeGenFunction.h @@ -609,7 +609,7 @@ const CodeGen::CGBlockInfo *BlockInfo = nullptr; llvm::Value *BlockPointer = nullptr; - llvm::DenseMap LambdaCaptureFields; + llvm::DenseMap LambdaCaptureFields; FieldDecl *LambdaThisCaptureField = nullptr; /// A mapping from NRVO variables to the flags used to indicate diff --git a/clang/lib/Sema/ScopeInfo.cpp b/clang/lib/Sema/ScopeInfo.cpp --- a/clang/lib/Sema/ScopeInfo.cpp +++ b/clang/lib/Sema/ScopeInfo.cpp @@ -220,7 +220,8 @@ bool Capture::isInitCapture() const { // Note that a nested capture of an init-capture is not itself an // init-capture. - return !isNested() && isVariableCapture() && getVariable()->isInitCapture(); + return !isNested() && isVariableCapture() && isa(getVariable()) && + cast(getVariable())->isInitCapture(); } bool CapturingScopeInfo::isVLATypeCaptured(const VariableArrayType *VAT) const { diff --git a/clang/lib/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp --- a/clang/lib/Sema/SemaDecl.cpp +++ b/clang/lib/Sema/SemaDecl.cpp @@ -14514,9 +14514,11 @@ auto I = LambdaClass->field_begin(); for (const auto &C : LambdaClass->captures()) { if (C.capturesVariable()) { - VarDecl *VD = C.getCapturedVar(); - if (VD->isInitCapture()) - S.CurrentInstantiationScope->InstantiatedLocal(VD, VD); + ValueDecl *VD = C.getCapturedVar(); + if (VarDecl *Var = dyn_cast(VD)) { + if (Var->isInitCapture()) + S.CurrentInstantiationScope->InstantiatedLocal(VD, VD); + } const bool ByRef = C.getCaptureKind() == LCK_ByRef; LSI->addCapture(VD, /*IsBlock*/false, ByRef, /*RefersToEnclosingVariableOrCapture*/true, C.getLocation(), diff --git a/clang/lib/Sema/SemaExpr.cpp b/clang/lib/Sema/SemaExpr.cpp --- a/clang/lib/Sema/SemaExpr.cpp +++ b/clang/lib/Sema/SemaExpr.cpp @@ -2082,9 +2082,8 @@ NestedNameSpecifierLoc NNS, NamedDecl *FoundD, SourceLocation TemplateKWLoc, const TemplateArgumentListInfo *TemplateArgs) { - bool RefersToCapturedVariable = - isa(D) && - NeedToCaptureVariable(cast(D), NameInfo.getLoc()); + bool RefersToCapturedVariable = isa(D) && + NeedToCaptureVariable(D, NameInfo.getLoc()); DeclRefExpr *E = DeclRefExpr::Create( Context, NNS, TemplateKWLoc, D, RefersToCapturedVariable, NameInfo, Ty, @@ -3415,13 +3414,13 @@ // These are always lvalues. valueKind = VK_LValue; type = type.getNonReferenceType(); - // FIXME: Support lambda-capture of BindingDecls, once CWG actually - // decides how that's supposed to work. auto *BD = cast(VD); - if (BD->getDeclContext() != CurContext) { - auto *DD = dyn_cast_or_null(BD->getDecomposedDecl()); - if (DD && DD->hasLocalStorage()) - diagnoseUncapturableValueReference(*this, Loc, BD); + auto *DD = dyn_cast_or_null(BD->getDecomposedDecl()); + if (DD && DD->hasLocalStorage()) { + if ((!LangOpts.CPlusPlus || !isLambdaCallOperator(CurContext)) && + BD->getDeclContext() != CurContext) { + diagnoseUncapturableValueReference(*this, Loc, VD); + } } break; } @@ -16312,7 +16311,7 @@ if (Cap.isInvalid() || Cap.isThisCapture()) continue; - VarDecl *Var = Cap.getVariable(); + VarDecl *Var = cast(Cap.getVariable()); Expr *CopyExpr = nullptr; if (getLangOpts().CPlusPlus && Cap.isCopyCapture()) { if (const RecordType *Record = @@ -18143,8 +18142,8 @@ MarkVarDeclODRUsed(Capture, Loc, *this, &CapturingScopeIndex); } -static void diagnoseUncapturableValueReference(Sema &S, SourceLocation loc, - ValueDecl *var) { +void diagnoseUncapturableValueReference(Sema &S, SourceLocation loc, + ValueDecl *var) { DeclContext *VarDC = var->getDeclContext(); // If the parameter still belongs to the translation unit, then @@ -18184,12 +18183,12 @@ // capture. } - -static bool isVariableAlreadyCapturedInScopeInfo(CapturingScopeInfo *CSI, VarDecl *Var, - bool &SubCapturesAreNested, - QualType &CaptureType, - QualType &DeclRefType) { - // Check whether we've already captured it. +static bool isVariableAlreadyCapturedInScopeInfo(CapturingScopeInfo *CSI, + ValueDecl *Var, + bool &SubCapturesAreNested, + QualType &CaptureType, + QualType &DeclRefType) { + // Check whether we've already captured it. if (CSI->CaptureMap.count(Var)) { // If we found a capture, any subcaptures are nested. SubCapturesAreNested = true; @@ -18216,14 +18215,16 @@ // Only block literals, captured statements, and lambda expressions can // capture; other scopes don't work. -static DeclContext *getParentOfCapturingContextOrNull(DeclContext *DC, VarDecl *Var, - SourceLocation Loc, - const bool Diagnose, Sema &S) { +static DeclContext *getParentOfCapturingContextOrNull(DeclContext *DC, + ValueDecl *Var, + SourceLocation Loc, + const bool Diagnose, + Sema &S) { if (isa(DC) || isa(DC) || isLambdaCallOperator(DC)) return getLambdaAwareParentOfDeclContext(DC); - else if (Var->hasLocalStorage()) { - if (Diagnose) - diagnoseUncapturableValueReference(S, Loc, Var); + else if (VarDecl *VD = dyn_cast(Var)) { + if (VD->hasLocalStorage() && Diagnose) + diagnoseUncapturableValueReference(S, Loc, VD); } return nullptr; } @@ -18231,9 +18232,12 @@ // Certain capturing entities (lambdas, blocks etc.) are not allowed to capture // certain types of variables (unnamed, variably modified types etc.) // so check for eligibility. -static bool isVariableCapturable(CapturingScopeInfo *CSI, VarDecl *Var, - SourceLocation Loc, - const bool Diagnose, Sema &S) { +static bool isVariableCapturable(CapturingScopeInfo *CSI, ValueDecl *Var, + SourceLocation Loc, const bool Diagnose, + Sema &S) { + + assert((isa(Var) || isa(Var)) && + "Only variables and structured bindings can be captured"); bool IsBlock = isa(CSI); bool IsLambda = isa(CSI); @@ -18290,17 +18294,26 @@ return false; } + if (isa(Var)) { + if (!IsLambda || !S.getLangOpts().CPlusPlus) { + return false; + } else if (Diagnose && S.getLangOpts().CPlusPlus) { + S.Diag(Loc, S.LangOpts.CPlusPlus20 + ? diag::warn_cxx17_compat_capture_binding + : diag::ext_capture_binding) + << Var; + S.Diag(Var->getLocation(), diag::note_entity_declared_at) << Var; + } + } + return true; } // Returns true if the capture by block was successful. -static bool captureInBlock(BlockScopeInfo *BSI, VarDecl *Var, - SourceLocation Loc, - const bool BuildAndDiagnose, - QualType &CaptureType, - QualType &DeclRefType, - const bool Nested, - Sema &S, bool Invalid) { +static bool captureInBlock(BlockScopeInfo *BSI, ValueDecl *Var, + SourceLocation Loc, const bool BuildAndDiagnose, + QualType &CaptureType, QualType &DeclRefType, + const bool Nested, Sema &S, bool Invalid) { bool ByRef = false; // Blocks are not allowed to capture arrays, excepting OpenCL. @@ -18364,10 +18377,9 @@ return !Invalid; } - /// Capture the given variable in the captured region. static bool captureInCapturedRegion( - CapturedRegionScopeInfo *RSI, VarDecl *Var, SourceLocation Loc, + CapturedRegionScopeInfo *RSI, ValueDecl *Var, SourceLocation Loc, const bool BuildAndDiagnose, QualType &CaptureType, QualType &DeclRefType, const bool RefersToCapturedVariable, Sema::TryCaptureKind Kind, bool IsTopScope, Sema &S, bool Invalid) { @@ -18406,16 +18418,12 @@ } /// Capture the given variable in the lambda. -static bool captureInLambda(LambdaScopeInfo *LSI, - VarDecl *Var, - SourceLocation Loc, - const bool BuildAndDiagnose, - QualType &CaptureType, - QualType &DeclRefType, +static bool captureInLambda(LambdaScopeInfo *LSI, ValueDecl *Var, + SourceLocation Loc, const bool BuildAndDiagnose, + QualType &CaptureType, QualType &DeclRefType, const bool RefersToCapturedVariable, const Sema::TryCaptureKind Kind, - SourceLocation EllipsisLoc, - const bool IsTopScope, + SourceLocation EllipsisLoc, const bool IsTopScope, Sema &S, bool Invalid) { // Determine whether we are capturing by reference or by value. bool ByRef = false; @@ -18424,6 +18432,43 @@ } else { ByRef = (LSI->ImpCaptureStyle == LambdaScopeInfo::ImpCap_LambdaByref); } + // C++20 : [expr.prim.lambda.capture]p12 + // A bit-field or a member of an anonymous union shall + // not be captured by reference. + MemberExpr *ME = nullptr; + BindingDecl *BD = nullptr; + if (VarDecl *V = dyn_cast_or_null(Var)) { + if (V->getInit()) + ME = dyn_cast(V->getInit()->IgnoreImplicit()); + } else if ((BD = dyn_cast(Var))) { + ME = dyn_cast_or_null(BD->getBinding()); + } + + // Capturing a bitfield by reference is not allowed + // except in OpenMP mode + if (ByRef && ME && + (isa(Var) || !S.LangOpts.OpenMP || + !S.isOpenMPCapturedDecl(Var))) { + FieldDecl *FD = dyn_cast_or_null(ME->getMemberDecl()); + if (FD && + (FD->isBitField() || (FD->getParent()->isAnonymousStructOrUnion() && + FD->getParent()->isUnion()))) { + if (BuildAndDiagnose) { + S.Diag(Loc, diag::err_bitfield_capture_by_ref) << Var; + S.Diag(Var->getLocation(), diag::note_entity_declared_at) << Var; + S.Diag(FD->getLocation(), diag::note_bitfield_decl) << FD; + } + Invalid = true; + } + } + // FIXME: We should support capturing structured bindings in OpenMP + if (!Invalid && BD && S.LangOpts.OpenMP) { + if (BuildAndDiagnose) { + S.Diag(Loc, diag::err_capture_binding_openmp) << Var; + S.Diag(Var->getLocation(), diag::note_entity_declared_at) << Var; + } + Invalid = true; + } // Compute the type of the field that will capture this variable. if (ByRef) { @@ -18505,7 +18550,8 @@ return !Invalid; } -static bool canCaptureVariableByCopy(VarDecl *Var, const ASTContext &Context) { +static bool canCaptureVariableByCopy(ValueDecl *Var, + const ASTContext &Context) { // Offer a Copy fix even if the type is dependent. if (Var->getType()->isDependentType()) return true; @@ -18531,7 +18577,7 @@ /// standard, for example we can't emit a default copy capture fix-it if we /// already explicitly copy capture capture another variable. static void buildLambdaCaptureFixit(Sema &Sema, LambdaScopeInfo *LSI, - VarDecl *Var) { + ValueDecl *Var) { assert(LSI->ImpCaptureStyle == CapturingScopeInfo::ImpCap_None); // Don't offer Capture by copy of default capture by copy fixes if Var is // known not to be copy constructible. @@ -18607,14 +18653,21 @@ } bool Sema::tryCaptureVariable( - VarDecl *Var, SourceLocation ExprLoc, TryCaptureKind Kind, + ValueDecl *Var, SourceLocation ExprLoc, TryCaptureKind Kind, SourceLocation EllipsisLoc, bool BuildAndDiagnose, QualType &CaptureType, QualType &DeclRefType, const unsigned *const FunctionScopeIndexToStopAt) { // An init-capture is notionally from the context surrounding its // declaration, but its parent DC is the lambda class. DeclContext *VarDC = Var->getDeclContext(); - if (Var->isInitCapture()) - VarDC = VarDC->getParent(); + VarDecl *VD = dyn_cast(Var); + if (VD) { + if (VD && VD->isInitCapture()) + VarDC = VarDC->getParent(); + } else { + VD = dyn_cast( + cast(Var)->getDecomposedDecl()); + } + assert(VD && "Cannot capture a null variable"); DeclContext *DC = CurContext; const unsigned MaxFunctionScopesIndex = FunctionScopeIndexToStopAt @@ -18636,12 +18689,14 @@ // Capture global variables if it is required to use private copy of this // variable. - bool IsGlobal = !Var->hasLocalStorage(); + bool IsGlobal = !VD->hasLocalStorage(); if (IsGlobal && !(LangOpts.OpenMP && isOpenMPCapturedDecl(Var, /*CheckScopeInfo=*/true, MaxFunctionScopesIndex))) return true; - Var = Var->getCanonicalDecl(); + + if (isa(Var)) + Var = cast(Var->getCanonicalDecl()); // Walk up the stack to determine whether we can capture the variable, // performing the "simple" checks that don't depend on type. We stop when @@ -18845,7 +18900,7 @@ return Invalid; } -bool Sema::tryCaptureVariable(VarDecl *Var, SourceLocation Loc, +bool Sema::tryCaptureVariable(ValueDecl *Var, SourceLocation Loc, TryCaptureKind Kind, SourceLocation EllipsisLoc) { QualType CaptureType; QualType DeclRefType; @@ -18854,7 +18909,7 @@ DeclRefType, nullptr); } -bool Sema::NeedToCaptureVariable(VarDecl *Var, SourceLocation Loc) { +bool Sema::NeedToCaptureVariable(ValueDecl *Var, SourceLocation Loc) { QualType CaptureType; QualType DeclRefType; return !tryCaptureVariable(Var, Loc, TryCapture_Implicit, SourceLocation(), @@ -18862,7 +18917,7 @@ DeclRefType, nullptr); } -QualType Sema::getCapturedDeclRefType(VarDecl *Var, SourceLocation Loc) { +QualType Sema::getCapturedDeclRefType(ValueDecl *Var, SourceLocation Loc) { QualType CaptureType; QualType DeclRefType; @@ -19458,6 +19513,24 @@ } } +static void DoMarkBindingDeclReferenced(Sema &SemaRef, SourceLocation Loc, + BindingDecl *BD) { + BD->setReferenced(); + + if (BD->isInvalidDecl()) + return; + + OdrUseContext OdrUse = isOdrUseContext(SemaRef); + if (OdrUse == OdrUseContext::Used) { + QualType CaptureType, DeclRefType; + SemaRef.tryCaptureVariable(BD, Loc, Sema::TryCapture_Implicit, + /*EllipsisLoc*/ SourceLocation(), + /*BuildAndDiagnose*/ true, CaptureType, + DeclRefType, + /*FunctionScopeIndexToStopAt*/ nullptr); + } +} + /// Mark a variable referenced, and check whether it is odr-used /// (C++ [basic.def.odr]p2, C99 6.9p3). Note that this should not be /// used directly for normal expressions referring to VarDecl. @@ -19477,6 +19550,11 @@ return; } + if (BindingDecl *Decl = dyn_cast(D)) { + DoMarkBindingDeclReferenced(SemaRef, Loc, Decl); + return; + } + SemaRef.MarkAnyDeclReferenced(Loc, D, MightBeOdrUse); // If this is a call to a method via a cast, also mark the method in the diff --git a/clang/lib/Sema/SemaInit.cpp b/clang/lib/Sema/SemaInit.cpp --- a/clang/lib/Sema/SemaInit.cpp +++ b/clang/lib/Sema/SemaInit.cpp @@ -7815,9 +7815,11 @@ break; // FIXME: We can't easily tell apart an init-capture from a nested // capture of an init-capture. - const VarDecl *VD = Elem.Capture->getCapturedVar(); + const ValueDecl *VD = Elem.Capture->getCapturedVar(); + bool InitCapture = + isa(VD) && cast(VD)->isInitCapture(); Diag(Elem.Capture->getLocation(), diag::note_lambda_capture_initializer) - << VD << VD->isInitCapture() << Elem.Capture->isExplicit() + << VD << InitCapture << Elem.Capture->isExplicit() << (Elem.Capture->getCaptureKind() == LCK_ByRef) << VD << nextPathEntryRange(Path, I + 1, L); break; diff --git a/clang/lib/Sema/SemaLambda.cpp b/clang/lib/Sema/SemaLambda.cpp --- a/clang/lib/Sema/SemaLambda.cpp +++ b/clang/lib/Sema/SemaLambda.cpp @@ -1088,7 +1088,7 @@ if (C->Init.isInvalid()) continue; - VarDecl *Var = nullptr; + ValueDecl *Var = nullptr; if (C->Init.isUsable()) { Diag(C->Loc, getLangOpts().CPlusPlus14 ? diag::warn_cxx11_compat_init_capture @@ -1166,7 +1166,10 @@ continue; } - Var = R.getAsSingle(); + if (auto BD = R.getAsSingle()) + Var = BD; + else + Var = R.getAsSingle(); if (Var && DiagnoseUseOfDecl(Var, C->Loc)) continue; } @@ -1200,7 +1203,14 @@ if (Var->isInvalidDecl()) continue; - if (!Var->hasLocalStorage()) { + VarDecl *Underlying; + if (isa(Var)) { + Underlying = dyn_cast_or_null( + cast(Var)->getDecomposedDecl()); + } else + Underlying = cast(Var); + + if (!Underlying->hasLocalStorage()) { Diag(C->Loc, diag::err_capture_non_automatic_variable) << C->Id; Diag(Var->getLocation(), diag::note_previous_decl) << C->Id; continue; @@ -1224,7 +1234,7 @@ } if (C->Init.isUsable()) { - addInitCapture(LSI, Var); + addInitCapture(LSI, cast(Var)); } else { TryCaptureKind Kind = C->Kind == LCK_ByRef ? TryCapture_ExplicitByRef : TryCapture_ExplicitByVal; @@ -1574,7 +1584,7 @@ // An init-capture is initialized directly from its stored initializer. if (Cap.isInitCapture()) - return Cap.getVariable()->getInit(); + return cast(Cap.getVariable())->getInit(); // For anything else, build an initialization expression. For an implicit // capture, the capture notionally happens at the capture-default, so use @@ -1605,7 +1615,7 @@ Init = This; } else { assert(Cap.isVariableCapture() && "unknown kind of capture"); - VarDecl *Var = Cap.getVariable(); + ValueDecl *Var = Cap.getVariable(); Name = Var->getIdentifier(); Init = BuildDeclarationNameExpr( CXXScopeSpec(), DeclarationNameInfo(Var->getDeclName(), Loc), Var); @@ -1654,7 +1664,7 @@ bool Sema::CaptureHasSideEffects(const Capture &From) { if (From.isInitCapture()) { - Expr *Init = From.getVariable()->getInit(); + Expr *Init = cast(From.getVariable())->getInit(); if (Init && Init->HasSideEffects(Context)) return true; } @@ -1704,9 +1714,9 @@ TypeSourceInfo *TSI = nullptr; if (Capture.isVariableCapture()) { - auto *Var = Capture.getVariable(); - if (Var->isInitCapture()) - TSI = Capture.getVariable()->getTypeSourceInfo(); + VarDecl *Var = llvm::dyn_cast_or_null(Capture.getVariable()); + if (Var && Var->isInitCapture()) + TSI = Var->getTypeSourceInfo(); } // FIXME: Should we really be doing this? A null TypeSourceInfo seems more @@ -1854,7 +1864,7 @@ return LambdaCapture(From.getLocation(), IsImplicit, LCK_VLAType); } else { assert(From.isVariableCapture() && "unknown kind of capture"); - VarDecl *Var = From.getVariable(); + ValueDecl *Var = From.getVariable(); LambdaCaptureKind Kind = From.isCopyCapture() ? LCK_ByCopy : LCK_ByRef; return LambdaCapture(From.getLocation(), IsImplicit, Kind, Var, diff --git a/clang/lib/Sema/SemaOpenMP.cpp b/clang/lib/Sema/SemaOpenMP.cpp --- a/clang/lib/Sema/SemaOpenMP.cpp +++ b/clang/lib/Sema/SemaOpenMP.cpp @@ -4521,12 +4521,12 @@ DSAStack->setForceCaptureByReferenceInTargetExecutable( /*V=*/true); if (RD->isLambda()) { - llvm::DenseMap Captures; + llvm::DenseMap Captures; FieldDecl *ThisCapture; RD->getCaptureFields(Captures, ThisCapture); for (const LambdaCapture &LC : RD->captures()) { if (LC.getCaptureKind() == LCK_ByRef) { - VarDecl *VD = LC.getCapturedVar(); + VarDecl *VD = cast(LC.getCapturedVar()); DeclContext *VDC = VD->getDeclContext(); if (!VDC->Encloses(CurContext)) continue; diff --git a/clang/lib/Sema/SemaStmt.cpp b/clang/lib/Sema/SemaStmt.cpp --- a/clang/lib/Sema/SemaStmt.cpp +++ b/clang/lib/Sema/SemaStmt.cpp @@ -4689,11 +4689,11 @@ if (S.getLangOpts().OpenMP && RSI->CapRegionKind == CR_OpenMP) S.setOpenMPCaptureKind(Field, Cap.getVariable(), RSI->OpenMPLevel); - Captures.push_back(CapturedStmt::Capture(Cap.getLocation(), - Cap.isReferenceCapture() - ? CapturedStmt::VCK_ByRef - : CapturedStmt::VCK_ByCopy, - Cap.getVariable())); + Captures.push_back(CapturedStmt::Capture( + Cap.getLocation(), + Cap.isReferenceCapture() ? CapturedStmt::VCK_ByRef + : CapturedStmt::VCK_ByCopy, + cast(Cap.getVariable()))); } CaptureInits.push_back(Init.get()); } diff --git a/clang/lib/Sema/TreeTransform.h b/clang/lib/Sema/TreeTransform.h --- a/clang/lib/Sema/TreeTransform.h +++ b/clang/lib/Sema/TreeTransform.h @@ -12931,7 +12931,7 @@ continue; TransformedInitCapture &Result = InitCaptures[C - E->capture_begin()]; - VarDecl *OldVD = C->getCapturedVar(); + VarDecl *OldVD = cast(C->getCapturedVar()); auto SubstInitCapture = [&](SourceLocation EllipsisLoc, Optional NumExpansions) { @@ -12948,7 +12948,8 @@ getSema().buildLambdaInitCaptureInitialization( C->getLocation(), OldVD->getType()->isReferenceType(), EllipsisLoc, NumExpansions, OldVD->getIdentifier(), - C->getCapturedVar()->getInitStyle() != VarDecl::CInit, + cast(C->getCapturedVar())->getInitStyle() != + VarDecl::CInit, NewExprInit); Result.Expansions.push_back( InitCaptureInfoTy(NewExprInit, NewInitCaptureType)); @@ -13115,7 +13116,7 @@ if (E->isInitCapture(C)) { TransformedInitCapture &NewC = InitCaptures[C - E->capture_begin()]; - VarDecl *OldVD = C->getCapturedVar(); + VarDecl *OldVD = cast(C->getCapturedVar()); llvm::SmallVector NewVDs; for (InitCaptureInfoTy &Info : NewC.Expansions) { @@ -13170,7 +13171,7 @@ // The transform has determined that we should perform an expansion; // transform and capture each of the arguments. // expansion of the pattern. Do so. - VarDecl *Pack = C->getCapturedVar(); + VarDecl *Pack = cast(C->getCapturedVar()); for (unsigned I = 0; I != *NumExpansions; ++I) { Sema::ArgumentPackSubstitutionIndexRAII SubstIndex(getSema(), I); VarDecl *CapturedVar diff --git a/clang/lib/Serialization/ASTWriter.cpp b/clang/lib/Serialization/ASTWriter.cpp --- a/clang/lib/Serialization/ASTWriter.cpp +++ b/clang/lib/Serialization/ASTWriter.cpp @@ -5781,7 +5781,7 @@ break; case LCK_ByCopy: case LCK_ByRef: - VarDecl *Var = + ValueDecl *Var = Capture.capturesVariable() ? Capture.getCapturedVar() : nullptr; AddDeclRef(Var); AddSourceLocation(Capture.isPackExpansion() ? Capture.getEllipsisLoc() diff --git a/clang/lib/StaticAnalyzer/Checkers/DeadStoresChecker.cpp b/clang/lib/StaticAnalyzer/Checkers/DeadStoresChecker.cpp --- a/clang/lib/StaticAnalyzer/Checkers/DeadStoresChecker.cpp +++ b/clang/lib/StaticAnalyzer/Checkers/DeadStoresChecker.cpp @@ -504,7 +504,7 @@ // Treat local variables captured by reference in C++ lambdas as escaped. void findLambdaReferenceCaptures(const LambdaExpr *LE) { const CXXRecordDecl *LambdaClass = LE->getLambdaClass(); - llvm::DenseMap CaptureFields; + llvm::DenseMap CaptureFields; FieldDecl *ThisCaptureField; LambdaClass->getCaptureFields(CaptureFields, ThisCaptureField); @@ -512,14 +512,14 @@ if (!C.capturesVariable()) continue; - VarDecl *VD = C.getCapturedVar(); + ValueDecl *VD = C.getCapturedVar(); const FieldDecl *FD = CaptureFields[VD]; - if (!FD) + if (!FD || !isa(VD)) continue; // If the capture field is a reference type, it is capture-by-reference. if (FD->getType()->isReferenceType()) - Escaped.insert(VD); + Escaped.insert(cast(VD)); } } }; diff --git a/clang/lib/StaticAnalyzer/Checkers/WebKit/UncountedLambdaCapturesChecker.cpp b/clang/lib/StaticAnalyzer/Checkers/WebKit/UncountedLambdaCapturesChecker.cpp --- a/clang/lib/StaticAnalyzer/Checkers/WebKit/UncountedLambdaCapturesChecker.cpp +++ b/clang/lib/StaticAnalyzer/Checkers/WebKit/UncountedLambdaCapturesChecker.cpp @@ -57,7 +57,7 @@ void visitLambdaExpr(LambdaExpr *L) const { for (const LambdaCapture &C : L->captures()) { if (C.capturesVariable()) { - VarDecl *CapturedVar = C.getCapturedVar(); + ValueDecl *CapturedVar = C.getCapturedVar(); if (auto *CapturedVarType = CapturedVar->getType().getTypePtrOrNull()) { Optional IsUncountedPtr = isUncountedPtr(CapturedVarType); if (IsUncountedPtr && *IsUncountedPtr) { @@ -68,7 +68,7 @@ } } - void reportBug(const LambdaCapture &Capture, VarDecl *CapturedVar, + void reportBug(const LambdaCapture &Capture, ValueDecl *CapturedVar, const Type *T) const { assert(CapturedVar); diff --git a/clang/lib/StaticAnalyzer/Core/ExprEngine.cpp b/clang/lib/StaticAnalyzer/Core/ExprEngine.cpp --- a/clang/lib/StaticAnalyzer/Core/ExprEngine.cpp +++ b/clang/lib/StaticAnalyzer/Core/ExprEngine.cpp @@ -2548,7 +2548,7 @@ MD->getParent()->isLambda()) { // Lookup the field of the lambda. const CXXRecordDecl *CXXRec = MD->getParent(); - llvm::DenseMap LambdaCaptureFields; + llvm::DenseMap LambdaCaptureFields; FieldDecl *LambdaThisCaptureField; CXXRec->getCaptureFields(LambdaCaptureFields, LambdaThisCaptureField); diff --git a/clang/lib/StaticAnalyzer/Core/LoopUnrolling.cpp b/clang/lib/StaticAnalyzer/Core/LoopUnrolling.cpp --- a/clang/lib/StaticAnalyzer/Core/LoopUnrolling.cpp +++ b/clang/lib/StaticAnalyzer/Core/LoopUnrolling.cpp @@ -175,7 +175,7 @@ const CXXRecordDecl *LambdaCXXRec = MD->getParent(); // Lookup the fields of the lambda - llvm::DenseMap LambdaCaptureFields; + llvm::DenseMap LambdaCaptureFields; FieldDecl *LambdaThisCaptureField; LambdaCXXRec->getCaptureFields(LambdaCaptureFields, LambdaThisCaptureField); diff --git a/clang/test/CodeGenCXX/cxx20-decomposition.cpp b/clang/test/CodeGenCXX/cxx20-decomposition.cpp new file mode 100644 --- /dev/null +++ b/clang/test/CodeGenCXX/cxx20-decomposition.cpp @@ -0,0 +1,42 @@ +// RUN: %clang_cc1 -std=c++20 -triple x86_64-linux-gnu -emit-llvm %s -o - | FileCheck %s + +struct S { + int i; + int j; +}; + + + +int f() { + auto [i, j] = S{1, 42}; + return [j, &i] { + i++; + return i + j; + }(); +} + +// CHECK-LABEL: define{{.*}} i32 @_Z1fv() +// CHECK: call void @llvm.memcpy.p0.p0.i64(ptr align 4 %0, ptr align 4 @__const._Z1fv., i64 8, i1 false) +// CHECK: %1 = getelementptr inbounds %class.anon, ptr %ref.tmp, i32 0, i32 0 +// CHECK: %j = getelementptr inbounds %struct.S, ptr %0, i32 0, i32 1 +// CHECK: %2 = load i32, ptr %j, align 4 +// CHECK: store i32 %2, ptr %1, align 8 +// CHECK: %3 = getelementptr inbounds %class.anon, ptr %ref.tmp, i32 0, i32 1 +// CHECK: %i = getelementptr inbounds %struct.S, ptr %0, i32 0, i32 0 +// CHECK: store ptr %i, ptr %3, align 8 +// CHECK: %call = call noundef i32 @"_ZZ1fvENK3$_0clEv"(ptr noundef nonnull align 8 dereferenceable(16) %ref.tmp) +// CHECK: ret i32 %call + +// CHECK-LABEL: define{{.*}} i32 @"_ZZ1fvENK3$_0clEv"( +// CHECK: %0 = getelementptr inbounds %class.anon, ptr %this1, i32 0, i32 1 +// CHECK: %1 = load ptr, ptr %0, align 8 +// CHECK: %2 = load i32, ptr %1, align 4 +// CHECK: %inc = add nsw i32 %2, 1 +// CHECK: store i32 %inc, ptr %1, align 4 +// CHECK: %3 = getelementptr inbounds %class.anon, ptr %this1, i32 0, i32 1 +// CHECK: %4 = load ptr, ptr %3, align 8 +// CHECK: %5 = load i32, ptr %4, align 4 +// CHECK: %6 = getelementptr inbounds %class.anon, ptr %this1, i32 0, i32 0 +// CHECK: %7 = load i32, ptr %6, align 8 +// CHECK: %add = add nsw i32 %5, %7 +// CHECK: ret i32 %add diff --git a/clang/test/SemaCXX/cxx1z-decomposition.cpp b/clang/test/SemaCXX/cxx1z-decomposition.cpp --- a/clang/test/SemaCXX/cxx1z-decomposition.cpp +++ b/clang/test/SemaCXX/cxx1z-decomposition.cpp @@ -1,4 +1,5 @@ -// RUN: %clang_cc1 -std=c++17 -verify %s +// RUN: %clang_cc1 -std=c++17 -Wc++20-extensions -verify=expected %s +// RUN: %clang_cc1 -std=c++20 -Wpre-c++20-compat -verify=expected %s void use_from_own_init() { auto [a] = a; // expected-error {{binding 'a' cannot appear in the initializer of its own decomposition declaration}} @@ -46,25 +47,58 @@ } static_assert(f({1, 2}) == 12); -constexpr bool g(S &&s) { +constexpr bool g(S &&s) { auto &[a, b] = s; return &a == &s.a && &b == &s.b && &a != &b; } static_assert(g({1, 2})); -auto [outer1, outer2] = S{1, 2}; +struct S1 { + int a, b; +}; +struct S2 { + int a : 1; // expected-note 2{{bit-field is declared here}} + int b; +}; + +auto [outer1, outer2] = S1{1, 2}; +auto [outerbit1, outerbit2] = S1{1, 2}; // expected-note {{declared here}} + void enclosing() { struct S { int a = outer1; }; - auto [n] = S(); // expected-note 2{{'n' declared here}} + auto [n] = S(); // expected-note 3{{'n' declared here}} + + struct Q { + int f() { return n; } // expected-error {{reference to local binding 'n' declared in enclosing function 'enclosing'}} + }; + + (void)[&] { return n; }; // expected-warning {{C++20}} + (void)[n] { return n; }; // expected-warning {{C++20}} - struct Q { int f() { return n; } }; // expected-error {{reference to local binding 'n' declared in enclosing function}} - (void) [&] { return n; }; // expected-error {{reference to local binding 'n' declared in enclosing function}} - (void) [n] {}; // expected-error {{'n' in capture list does not name a variable}} + static auto [m] = S(); // expected-note {{'m' declared here}} \ + // expected-warning {{C++20}} - static auto [m] = S(); // expected-warning {{extension}} struct R { int f() { return m; } }; (void) [&] { return m; }; - (void) [m] {}; // expected-error {{'m' in capture list does not name a variable}} + (void)[m]{}; // expected-error {{'m' cannot be captured because it does not have automatic storage duration}} + + (void)[outerbit1]{}; // expected-error {{'outerbit1' cannot be captured because it does not have automatic storage duration}} + + auto [bit, var] = S2{1, 1}; // expected-note 4{{'bit' declared here}} + + (void)[&bit] { // expected-error {{cannot capture a bit-field by reference}} \ + // expected-warning {{C++20}} + return bit; + }; + + union { // expected-note {{declared here}} + int u; + }; + + (void)[&] { return bit + u; } // expected-error {{unnamed variable cannot be implicitly captured in a lambda expression}} \ + // expected-error {{cannot capture a bit-field by reference}} \ + // expected-warning {{C++20}} + (); } void bitfield() { @@ -98,7 +132,7 @@ struct PR37352 { int n; - void f() { static auto [a] = *this; } // expected-warning {{C++20 extension}} + void f() { static auto [a] = *this; } // expected-warning {{C++20}} }; namespace instantiate_template { diff --git a/clang/test/SemaCXX/cxx20-decomposition.cpp b/clang/test/SemaCXX/cxx20-decomposition.cpp new file mode 100644 --- /dev/null +++ b/clang/test/SemaCXX/cxx20-decomposition.cpp @@ -0,0 +1,46 @@ +// RUN: %clang_cc1 -fsyntax-only -std=c++20 -verify %s +// expected-no-diagnostics + +template +constexpr bool is_same = false; +template +constexpr bool is_same = true; + +struct S { + int i; + int &j; +}; + +void check_category() { + int a = 42; + { + auto [v, r] = S{1, a}; + (void)[ v, r ] { + static_assert(is_same); + static_assert(is_same); + }; + } + { + auto [v, r] = S{1, a}; + (void)[&v, &r ] { + static_assert(is_same); + static_assert(is_same); + }; + } + { + S s{1, a}; + const auto &[v, r] = s; + (void)[ v, r ] { + static_assert(is_same); + static_assert(is_same); + }; + } + { + S s{1, a}; + const auto &[v, r] = s; + (void)[&v, &r ] { + static_assert(is_same); + static_assert(is_same); + }; + } +} diff --git a/clang/test/SemaCXX/decomposition-blocks.cpp b/clang/test/SemaCXX/decomposition-blocks.cpp new file mode 100644 --- /dev/null +++ b/clang/test/SemaCXX/decomposition-blocks.cpp @@ -0,0 +1,14 @@ +// RUN: %clang_cc1 -std=c++17 -fsyntax-only -verify %s -fblocks + +struct S { + int i : 1; + int j; +}; + +void run(void (^)()); +void test() { + auto [i, j] = S{1, 42}; // expected-note {{'i' declared here}} + run(^{ + (void)i; // expected-error {{reference to local binding 'i' declared in enclosing function 'test'}} + }); +} diff --git a/clang/test/SemaCXX/decomposition-openmp.cpp b/clang/test/SemaCXX/decomposition-openmp.cpp new file mode 100644 --- /dev/null +++ b/clang/test/SemaCXX/decomposition-openmp.cpp @@ -0,0 +1,13 @@ + +// RUN: %clang_cc1 -fsyntax-only -verify -std=c++20 -fopenmp %s + +// FIXME: OpenMP should support capturing structured bindings +auto f() { + int i[2] = {}; + auto [a, b] = i; // expected-note 2{{declared here}} + return [=, &a] { + // expected-error@-1 {{capturing a structured binding is not yet supported in OpenMP}} + return a + b; + // expected-error@-1 {{capturing a structured binding is not yet supported in OpenMP}} + }; +} diff --git a/clang/tools/libclang/CIndex.cpp b/clang/tools/libclang/CIndex.cpp --- a/clang/tools/libclang/CIndex.cpp +++ b/clang/tools/libclang/CIndex.cpp @@ -3449,9 +3449,11 @@ C != CEnd; ++C) { if (!C->capturesVariable()) continue; - - if (Visit(MakeCursorVariableRef(C->getCapturedVar(), C->getLocation(), - TU))) + // TODO: hamdle structured bindings here ? + if (!isa(C->getCapturedVar())) + continue; + if (Visit(MakeCursorVariableRef(cast(C->getCapturedVar()), + C->getLocation(), TU))) return true; } // Visit init captures diff --git a/clang/www/cxx_status.html b/clang/www/cxx_status.html --- a/clang/www/cxx_status.html +++ b/clang/www/cxx_status.html @@ -1144,7 +1144,7 @@ Structured binding extensions P1091R3 - Partial + Clang 15 P1381R1