From: Anders Carlsson <andersca@mac.com>
Date: Fri, 15 May 2009 20:10:48 +0000 (+0000)
Subject: No need to null check the expr, Sema::SemaRef.InstantiateExpr handles all that.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=a52aac4851d379403aae4994f16307bf1a219a1c;p=clang

No need to null check the expr, Sema::SemaRef.InstantiateExpr handles all that.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@71885 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Sema/SemaTemplateInstantiateStmt.cpp b/lib/Sema/SemaTemplateInstantiateStmt.cpp
index 957402ac6f..8bdb42b58f 100644
--- a/lib/Sema/SemaTemplateInstantiateStmt.cpp
+++ b/lib/Sema/SemaTemplateInstantiateStmt.cpp
@@ -98,13 +98,10 @@ Sema::OwningStmtResult TemplateStmtInstantiator::VisitGotoStmt(GotoStmt *S) {
 
 Sema::OwningStmtResult
 TemplateStmtInstantiator::VisitReturnStmt(ReturnStmt *S) {
-  Sema::OwningExprResult Result = SemaRef.ExprEmpty();
-  if (Expr *E = S->getRetValue()) {
-    Result = SemaRef.InstantiateExpr(E, TemplateArgs);
-    
-    if (Result.isInvalid())
-      return SemaRef.StmtError();
-  }
+  Sema::OwningExprResult Result = 
+        SemaRef.InstantiateExpr(S->getRetValue(), TemplateArgs);
+  if (Result.isInvalid())
+    return SemaRef.StmtError();
   
   return SemaRef.ActOnReturnStmt(S->getReturnLoc(), move(Result));
 }