On Thu, Sep 01, 2011 at 09:34:20AM +0800, Terry Guo wrote: > Here is the patch, is it OK to commit?
Definitely not. struct-layout-1* already has support for short enums, see http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=110247 So, if that doesn't work for you, you need to debug why. > 2011-08-31 Terry Guo <terry....@arm.com> > > * gcc.dg/compat/struct-layout-1_main.c: Skip the case > if the target uses short enums. > > diff --git a/gcc/testsuite/gcc.dg/compat/struct-layout-1_main.c > b/gcc/testsuite/gcc.dg/compat/struct-layout-1_main.c > index b59453e..64275ea 100644 > --- a/gcc/testsuite/gcc.dg/compat/struct-layout-1_main.c > +++ b/gcc/testsuite/gcc.dg/compat/struct-layout-1_main.c > @@ -1,4 +1,5 @@ > /* { dg-prune-output ".*-Wno-abi.*" } */ > +/* { dg-skip-if "" { short_enums } } */ > > #include "struct-layout-1.h" > Jakub
