[cfe-commits] r70064 - /cfe/trunk/lib/Sema/SemaExpr.cpp

[Chris Lattner]

Author: lattner
Date: Sat Apr 25 14:11:05 2009
New Revision: 70064

URL: http://llvm.org/viewvc/llvm-project?rev=70064&view=rev
Log:
remove a fixme that is already done.

Modified:
    cfe/trunk/lib/Sema/SemaExpr.cpp

Modified: cfe/trunk/lib/Sema/SemaExpr.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaExpr.cpp?rev=70064&r1=70063&r2=70064&view=diff

==============================================================================
--- cfe/trunk/lib/Sema/SemaExpr.cpp (original)
+++ cfe/trunk/lib/Sema/SemaExpr.cpp Sat Apr 25 14:11:05 2009
@@ -4495,17 +4495,13 @@
   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;