From: Chris Lattner <sabre@nondot.org>
Date: Sat, 25 Apr 2009 19:11:05 +0000 (+0000)
Subject: remove a fixme that is already done.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=4a049f074117555e932a1256a3d41d8254e85866;p=clang

remove a fixme that is already done.


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

diff --git a/lib/Sema/SemaExpr.cpp b/lib/Sema/SemaExpr.cpp
index fc5b678f21..474670dbb7 100644
--- a/lib/Sema/SemaExpr.cpp
+++ b/lib/Sema/SemaExpr.cpp
@@ -4495,17 +4495,13 @@ Sema::ActOnStmtExpr(SourceLocation LPLoc, StmtArg substmt,
   CompoundStmt *Compound = cast<CompoundStmt>(SubStmt);
 
   bool isFileScope = getCurFunctionOrMethodDecl() == 0;
-  if (isFileScope) {
+  if (isFileScope)
     return ExprError(Diag(LPLoc, diag::err_stmtexpr_file_scope));
-  }
 
   // FIXME: there are a variety of strange constraints to enforce here, for
   // example, it is not possible to goto into a stmt expression apparently.
   // More semantic analysis is needed.
 
-  // FIXME: the last statement in the compount stmt has its value used.  We
-  // should not warn about it being unused.
-
   // If there are sub stmts in the compound stmt, take the type of the last one
   // as the type of the stmtexpr.
   QualType Ty = Context.VoidTy;