On 11/09/11 02:39, Terry Guo wrote:
> Hello,
>
> This patch aims to disable short enums for arm eabi otherwise the case will
> fail to be compiled due to "width of 'code' exceeds its type". Is it OK to
> trunk?
>
> BR,
> Terry
>
> 2011-09-09 Terry Guo <terry....@arm.com>
>
> * gcc.dg/tree-ssa/foldconst-3.c: Use -fno-short-enums
> for ARM EABI target.
>
> diff --git a/gcc/testsuite/gcc.dg/tree-ssa/foldconst-3.c
> b/gcc/testsuite/gcc.dg/tree-ssa/foldconst-3.c
> index 6132362..e030f53 100644
> --- a/gcc/testsuite/gcc.dg/tree-ssa/foldconst-3.c
> +++ b/gcc/testsuite/gcc.dg/tree-ssa/foldconst-3.c
> @@ -1,5 +1,6 @@
> /* { dg-do compile } */
> /* { dg-options "-O2 -fdump-tree-optimized" } */
> +/* { dg-options "-O2 -fdump-tree-optimized -fno-short-enums" { target
> arm_eabi } } */
> typedef const union tree_node *const_tree;
> typedef struct
> {
>
>
This is a compile-only test, and -fno-short-enums is a global option
that all targets support, so I'd suggest just adding -fno-short-enums to
the standard list of options. It won't hurt those platforms where that
is already the default and it will fix the testcase problem for all
those platforms where it isn't.
R.
