This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
RE: [patch, testsuite] Fix another short-enums testsuite error
- From: "Ian Bolton" <ian dot bolton at arm dot com>
- To: "'Joseph Myers'" <joseph at codesourcery dot com>
- Cc: <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 18 Jan 2011 09:46:28 -0000
- Subject: RE: [patch, testsuite] Fix another short-enums testsuite error
- References: <000001cbb3f8$8351ab60$89f50220$@bolton@arm.com> <Pine.LNX.4.64.1101141723290.18651@digraph.polyomino.org.uk>
> 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. */