[patch, testsuite] Fix another short-enums testsuite error

Ian Bolton ian.bolton@arm.com
Tue Jan 18 10:48:00 GMT 2011


> The normal approach for the warning would be to pass
> -Wl,--no-enum-size-warning on ARM eabi targets (or at least, that's the
> approach for tests passing -fshort-enums explicitly).
> 

Thanks for the input, Joseph.

I've updated the test to reflect this.  Assuming that other targets can
handle that linker option too, it should be good to go.  If not, I will
probably need to limit this patch to ARM eabi targets only.

Here's the new patch:


2011-01-18  Ian Bolton  <ian.bolton@arm.com>

	* testsuite/gcc.dg/20100906-1.c: Use -fno-short-enums
	option for "target short_enums".


Index: gcc/testsuite/gcc.dg/20100906-1.c
===================================================================
--- gcc/testsuite/gcc.dg/20100906-1.c   (revision 168543)
+++ gcc/testsuite/gcc.dg/20100906-1.c   (working copy)
@@ -1,5 +1,6 @@
 /* { dg-do run } */
 /* { dg-options "-O2" } */
+/* { dg-options "-O2 -fno-short-enums -Wl,--no-enum-size-warning" {target
short_enums} } */
 
 /* This testcase got misoptimized by combine due to a wrong setting of
    subst_low_luid in try_combine.  */





More information about the Gcc-patches mailing list