On May 28, 2010, at 3:34 PM, Eli Friedman wrote: > On Fri, May 28, 2010 at 3:23 PM, Fariborz Jahanian <[email protected] > > wrote: >> Author: fjahanian >> Date: Fri May 28 17:23:22 2010 >> New Revision: 105011 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=105011&view=rev >> Log: >> Empty enum in c is now error to match gcc's behavior. >> (radar 8040068). >> >> >> Modified: >> cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td >> cfe/trunk/lib/Parse/ParseDecl.cpp >> cfe/trunk/test/CodeGen/typedef-func.c >> cfe/trunk/test/Sema/enum.c >> cfe/trunk/test/Sema/function-redecl.c >> >> Modified: cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td >> URL: >> http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td?rev=105011&r1=105010&r2=105011&view=diff >> = >> = >> = >> = >> = >> = >> = >> = >> = >> ===================================================================== >> --- cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td (original) >> +++ cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td Fri May >> 28 17:23:22 2010 >> @@ -35,6 +35,7 @@ >> >> def ext_empty_struct_union_enum : Extension<"use of empty %0 >> extension">; >> >> +def error_empty_enum : Error<"use of empty union">; > > Umm, maybe change the text of the error message? It's kind of > confusing to complain about a nonexistent union :)
Oops. Thanks. Fixed in r105013. - Fariborz _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
