diff options
| author | Anders Carlsson <andersca@mac.com> | 2010-11-28 16:40:49 +0000 |
|---|---|---|
| committer | Anders Carlsson <andersca@mac.com> | 2010-11-28 16:40:49 +0000 |
| commit | f8b3015060096a946d91578d0f3b65d3097a4ebb (patch) | |
| tree | f926c1254430063c42dbad10a0d4c4ae67b5d950 /lib/AST/Expr.cpp | |
| parent | fbfdb202405df65e054db4ad2111393c1b581d2d (diff) | |
Look through parentheses when deciding whether an expr is a temporary object. Fixes PR8683.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@120247 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/AST/Expr.cpp')
| -rw-r--r-- | lib/AST/Expr.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/lib/AST/Expr.cpp b/lib/AST/Expr.cpp index 3e3792e21f..3a112a7e07 100644 --- a/lib/AST/Expr.cpp +++ b/lib/AST/Expr.cpp @@ -1804,7 +1804,7 @@ bool Expr::isDefaultArgument() const { /// \brief Skip over any no-op casts and any temporary-binding /// expressions. -static const Expr *skipTemporaryBindingsAndNoOpCasts(const Expr *E) { +static const Expr *skipTemporaryBindingsNoOpCastsAndParens(const Expr *E) { while (const ImplicitCastExpr *ICE = dyn_cast<ImplicitCastExpr>(E)) { if (ICE->getCastKind() == CK_NoOp) E = ICE->getSubExpr(); @@ -1821,8 +1821,8 @@ static const Expr *skipTemporaryBindingsAndNoOpCasts(const Expr *E) { else break; } - - return E; + + return E->IgnoreParens(); } /// isTemporaryObject - Determines if this expression produces a @@ -1831,7 +1831,7 @@ bool Expr::isTemporaryObject(ASTContext &C, const CXXRecordDecl *TempTy) const { if (!C.hasSameUnqualifiedType(getType(), C.getTypeDeclType(TempTy))) return false; - const Expr *E = skipTemporaryBindingsAndNoOpCasts(this); + const Expr *E = skipTemporaryBindingsNoOpCastsAndParens(this); // Temporaries are by definition pr-values of class type. if (!E->Classify(C).isPRValue()) { |