vsapsai marked an inline comment as done.
vsapsai added inline comments.

================
Comment at: clang/test/Sema/enum.c:128
+// PR28903
+struct PR28903 { // expected-warning {{empty struct is a GNU extension}}
+  enum {
----------------
rnk wrote:
> This is a simpler test case that fixes the extraneous diagnostics:
>   struct PR28903 {
>     enum {
>       E1 = (enum {  // expected-error-re {{...}}
>               E2, E3 = E2
>             })0
>     } e;
>   };
> 
Removed extraneous diagnostics in slightly different way. Keep enum constants 
prefixed with PR28903 because in C struct doesn't create a scope for enum 
constants and I want to avoid polluting test namespace with common names like 
E1, E2, E3.


https://reviews.llvm.org/D37089



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to