Patchwork [testsuite] Use long enums for case foldconst-3.c for target ARM EABI.

login
register
mail settings
Submitter Terry Guo
Date Sept. 16, 2011, 8:28 a.m.
Message ID <000101cc744a$ae81dd10$0b859730$@guo@arm.com>
Download mbox | patch
Permalink /patch/114901/
State New
Headers show

Comments

Terry Guo - Sept. 16, 2011, 8:28 a.m.
> 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.

Agree. Here is the updated one.

BR,
Terry

2011-09-16  Terry Guo  <terry.guo@arm.com>

        * gcc.dg/tree-ssa/foldconst-3.c: Don't use short enums.
Richard Earnshaw - Sept. 16, 2011, 8:30 a.m.
On 16/09/11 09:28, Terry Guo wrote:
>> 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.
> 
> Agree. Here is the updated one.
> 
> BR,
> Terry
> 
> 2011-09-16  Terry Guo  <terry.guo@arm.com>
> 
>         * gcc.dg/tree-ssa/foldconst-3.c: Don't use short enums.
> 
> 

OK.

R.

Patch

diff --git a/gcc/testsuite/gcc.dg/tree-ssa/foldconst-3.c
b/gcc/testsuite/gcc.dg/tree-ssa/foldcons
index 6132362..9f10886 100644
--- a/gcc/testsuite/gcc.dg/tree-ssa/foldconst-3.c
+++ b/gcc/testsuite/gcc.dg/tree-ssa/foldconst-3.c
@@ -1,5 +1,5 @@ 
 /* { dg-do compile } */
-/* { dg-options "-O2 -fdump-tree-optimized" } */
+/* { dg-options "-O2 -fdump-tree-optimized -fno-short-enums" } */
 typedef const union tree_node *const_tree;  typedef struct  {