From a19e66d448223ccb1570a37d8838480011c8ecf8 Mon Sep 17 00:00:00 2001 From: Anders Carlsson Date: Sat, 30 May 2009 22:24:16 +0000 Subject: [PATCH] It's OK for a full expr to be null. This fixes the failing test cases. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@72642 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Sema/SemaExprCXX.cpp | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/lib/Sema/SemaExprCXX.cpp b/lib/Sema/SemaExprCXX.cpp index 18382bab21..2fde49caaf 100644 --- a/lib/Sema/SemaExprCXX.cpp +++ b/lib/Sema/SemaExprCXX.cpp @@ -1544,9 +1544,8 @@ Sema::OwningExprResult Sema::MaybeBindToTemporary(Expr *E) { Sema::OwningExprResult Sema::ActOnFinishFullExpr(ExprArg Arg) { Expr *FullExpr = Arg.takeAs(); - assert(FullExpr && "Null full expr!"); - - if (!ExprTemporaries.empty()) { + + if (FullExpr && !ExprTemporaries.empty()) { // Create a cleanup expr. FullExpr = new (Context) CXXExprWithTemporaries(FullExpr, &ExprTemporaries[0], -- 2.50.1