diff options
author | Zhongxing Xu <xuzhongxing@gmail.com> | 2010-11-13 07:30:59 +0000 |
---|---|---|
committer | Zhongxing Xu <xuzhongxing@gmail.com> | 2010-11-13 07:30:59 +0000 |
commit | 1b038fa00fbd3822eeda1261491bb383625903b2 (patch) | |
tree | b6745a62588d935d01e9d0f9e550ac61df3eb9de /clang/lib/Analysis/CFG.cpp | |
parent | bc8b3788480f7648d34dad7aed292d37f681d750 (diff) | |
download | bcm5719-llvm-1b038fa00fbd3822eeda1261491bb383625903b2.tar.gz bcm5719-llvm-1b038fa00fbd3822eeda1261491bb383625903b2.zip |
Do not add implicit dtors for CXXBindTemporaryExpr with elidable
CXXConstructExpr.
llvm-svn: 118991
Diffstat (limited to 'clang/lib/Analysis/CFG.cpp')
-rw-r--r-- | clang/lib/Analysis/CFG.cpp | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/clang/lib/Analysis/CFG.cpp b/clang/lib/Analysis/CFG.cpp index ef67d3b9576..42e081628d8 100644 --- a/clang/lib/Analysis/CFG.cpp +++ b/clang/lib/Analysis/CFG.cpp @@ -2579,11 +2579,18 @@ CFGBlock *CFGBuilder::VisitBinaryOperatorForTemporaryDtors(BinaryOperator *E) { return RHSBlock ? RHSBlock : LHSBlock; } +static bool hasElidableCXXConstructExpr(CXXBindTemporaryExpr *E) { + if (CXXConstructExpr *CE = dyn_cast<CXXConstructExpr>(E->getSubExpr())) + if (CE->isElidable()) + return true; + return false; +} + CFGBlock *CFGBuilder::VisitCXXBindTemporaryExprForTemporaryDtors( CXXBindTemporaryExpr *E, bool BindToTemporary) { // First add destructors for temporaries in subexpression. CFGBlock *B = VisitForTemporaryDtors(E->getSubExpr()); - if (!BindToTemporary) { + if (!BindToTemporary && !hasElidableCXXConstructExpr(E)) { // If lifetime of temporary is not prolonged (by assigning to constant // reference) add destructor for it. autoCreateBlock(); |