From a69b07636b541b6e01ee7550aad55e566b07637c Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Sat, 25 Apr 2009 19:11:05 +0000 Subject: remove a fixme that is already done. llvm-svn: 70064 --- clang/lib/Sema/SemaExpr.cpp | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) (limited to 'clang/lib/Sema/SemaExpr.cpp') diff --git a/clang/lib/Sema/SemaExpr.cpp b/clang/lib/Sema/SemaExpr.cpp index fc5b678f21b..474670dbb7a 100644 --- a/clang/lib/Sema/SemaExpr.cpp +++ b/clang/lib/Sema/SemaExpr.cpp @@ -4495,17 +4495,13 @@ Sema::ActOnStmtExpr(SourceLocation LPLoc, StmtArg substmt, CompoundStmt *Compound = cast(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; -- cgit v1.2.3