From 33accdf602a9191caa0a42c4400241def6613af5 Mon Sep 17 00:00:00 2001 From: Eli Friedman Date: Thu, 3 Jan 2013 00:39:26 +0000 Subject: [PATCH] Don't assert/crash on reference variables in lambdas bound to a static local variable from the parent scope. PR14773. llvm-svn: 171433 --- clang/lib/CodeGen/CGExprConstant.cpp | 6 ++---- clang/lib/CodeGen/CodeGenFunction.h | 5 ----- clang/test/CodeGenCXX/lambda-expressions.cpp | 15 +++++++++++++-- 3 files changed, 15 insertions(+), 11 deletions(-) diff --git a/clang/lib/CodeGen/CGExprConstant.cpp b/clang/lib/CodeGen/CGExprConstant.cpp index f5bf2a2fe7ad..7a604013b795 100644 --- a/clang/lib/CodeGen/CGExprConstant.cpp +++ b/clang/lib/CodeGen/CGExprConstant.cpp @@ -905,10 +905,8 @@ public: if (!VD->hasLocalStorage()) { if (VD->isFileVarDecl() || VD->hasExternalStorage()) return CGM.GetAddrOfGlobalVar(VD); - else if (VD->isLocalVarDecl()) { - assert(CGF && "Can't access static local vars without CGF"); - return CGF->GetAddrOfStaticLocalVar(VD); - } + else if (VD->isLocalVarDecl()) + return CGM.getStaticLocalDeclAddress(VD); } } return 0; diff --git a/clang/lib/CodeGen/CodeGenFunction.h b/clang/lib/CodeGen/CodeGenFunction.h index e8f70555c5a4..acc12fe6b63d 100644 --- a/clang/lib/CodeGen/CodeGenFunction.h +++ b/clang/lib/CodeGen/CodeGenFunction.h @@ -1687,11 +1687,6 @@ public: /// then reuse it. void StartBlock(const char *N); - /// GetAddrOfStaticLocalVar - Return the address of a static local variable. - llvm::Constant *GetAddrOfStaticLocalVar(const VarDecl *BVD) { - return cast(GetAddrOfLocalVar(BVD)); - } - /// GetAddrOfLocalVar - Return the address of a local variable. llvm::Value *GetAddrOfLocalVar(const VarDecl *VD) { llvm::Value *Res = LocalDeclMap[VD]; diff --git a/clang/test/CodeGenCXX/lambda-expressions.cpp b/clang/test/CodeGenCXX/lambda-expressions.cpp index a8967dce8023..ce20399c5791 100644 --- a/clang/test/CodeGenCXX/lambda-expressions.cpp +++ b/clang/test/CodeGenCXX/lambda-expressions.cpp @@ -80,9 +80,20 @@ int g() { return [] { return r; } (); }; -// CHECK: define internal void @"_ZZ1hvEN3$_78__invokeEv"(%struct.A* noalias sret %agg.result) +// PR14773 +// CHECK: [[ARRVAL:%[0-9a-zA-Z]*]] = load i32* getelementptr inbounds ([0 x i32]* bitcast (<{}>* @_ZZ14staticarrayrefvE5array to [0 x i32]*), i32 0, i64 0), align 4 +// CHECK-NEXT: store i32 [[ARRVAL]] +void staticarrayref(){ + static int array[] = {}; + (void)[](){ + int (&xxx)[0] = array; + int y = xxx[0]; + }(); +} + +// CHECK: define internal void @"_ZZ1hvEN3$_88__invokeEv"(%struct.A* noalias sret %agg.result) // CHECK-NOT: = -// CHECK: call void @"_ZZ1hvENK3$_7clEv"(%struct.A* sret %agg.result, +// CHECK: call void @"_ZZ1hvENK3$_8clEv"(%struct.A* sret %agg.result, // CHECK-NEXT: ret void struct A { ~A(); }; void h() {