diff options
author | Benjamin Kramer <benny.kra@googlemail.com> | 2011-07-12 14:11:05 +0000 |
---|---|---|
committer | Benjamin Kramer <benny.kra@googlemail.com> | 2011-07-12 14:11:05 +0000 |
commit | a4fb836f06807fc8056262d3e853414e73115b66 (patch) | |
tree | 011d128ae17bc83921af1655090d5da1b0d0d619 /clang | |
parent | b804a2b751ac0d1a5673ec395c4cecb326e73dfb (diff) | |
download | bcm5719-llvm-a4fb836f06807fc8056262d3e853414e73115b66.tar.gz bcm5719-llvm-a4fb836f06807fc8056262d3e853414e73115b66.zip |
Pop block scope after reading from it.
Found by valgrind.
llvm-svn: 134983
Diffstat (limited to 'clang')
-rw-r--r-- | clang/lib/Sema/SemaExpr.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/clang/lib/Sema/SemaExpr.cpp b/clang/lib/Sema/SemaExpr.cpp index 1e04ac734e6..9f91052e4cb 100644 --- a/clang/lib/Sema/SemaExpr.cpp +++ b/clang/lib/Sema/SemaExpr.cpp @@ -8496,10 +8496,6 @@ ExprResult Sema::ActOnBlockStmtExpr(SourceLocation CaretLoc, BSI->TheDecl->setBody(cast<CompoundStmt>(Body)); - BlockExpr *Result = new (Context) BlockExpr(BSI->TheDecl, BlockTy); - - const AnalysisBasedWarnings::Policy &WP = AnalysisWarnings.getDefaultPolicy(); - PopFunctionOrBlockScope(&WP, Result->getBlockDecl(), Result); for (BlockDecl::capture_const_iterator ci = BSI->TheDecl->capture_begin(), ce = BSI->TheDecl->capture_end(); ci != ce; ++ci) { const VarDecl *variable = ci->getVariable(); @@ -8509,6 +8505,10 @@ ExprResult Sema::ActOnBlockStmtExpr(SourceLocation CaretLoc, getCurFunction()->setHasBranchProtectedScope(); } + BlockExpr *Result = new (Context) BlockExpr(BSI->TheDecl, BlockTy); + const AnalysisBasedWarnings::Policy &WP = AnalysisWarnings.getDefaultPolicy(); + PopFunctionOrBlockScope(&WP, Result->getBlockDecl(), Result); + return Owned(Result); } |