| Message ID | alpine.DEB.2.22.394.2109072204190.1731265@digraph.polyomino.org.uk |
|---|---|
| State | New |
| Headers | show |
| Series | testsuite: Use explicit -ftree-cselim in tests using -fdump-tree-cselim-details | expand |
On Wed, Sep 8, 2021 at 12:05 AM Joseph Myers <joseph@codesourcery.com> wrote: > > When testing for Nios II (gcc-testresults shows this for various other > targets as well), tests scanning cselim dumps produce an UNRESOLVED > result because those dumps do not exist. > > cselim is enabled conditionally by code in toplev.c: > > if (flag_tree_cselim == AUTODETECT_VALUE) > { > if (HAVE_conditional_move) > flag_tree_cselim = 1; > else > flag_tree_cselim = 0; > } > > Add explicit -ftree-cselim to dg-options in the affected tests (as > already used by some other tests of cselim dumps) so that this dump > exists on all architectures. > > Tested with no regressions with cross to nios2-elf, where this causes > the tests in question to PASS instead of being UNRESOLVED. OK to commit? OK. Richard. > 2021-09-07 Joseph Myers <joseph@codesourcery.com> > > * gcc.dg/tree-ssa/pr89430-1.c, gcc.dg/tree-ssa/pr89430-2.c, > gcc.dg/tree-ssa/pr89430-3.c, gcc.dg/tree-ssa/pr89430-4.c, > gcc.dg/tree-ssa/pr89430-5.c, gcc.dg/tree-ssa/pr89430-6.c, > gcc.dg/tree-ssa/pr89430-7-comp-ref.c, > gcc.dg/tree-ssa/pr89430-8-mem-ref-size.c, > gcc.dg/tree-ssa/pr99473-1.c: Use -ftree-cselim. > > diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-1.c b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-1.c > index 8ee1850ac63..d9fb2edf549 100644 > --- a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-1.c > +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-1.c > @@ -1,5 +1,5 @@ > /* { dg-do compile } */ > -/* { dg-options "-O2 -fdump-tree-cselim-details" } */ > +/* { dg-options "-O2 -ftree-cselim -fdump-tree-cselim-details" } */ > > unsigned test(unsigned k, unsigned b) { > unsigned a[2]; > diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-2.c b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-2.c > index 9b96875ac7a..bb39df2be8e 100644 > --- a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-2.c > +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-2.c > @@ -1,5 +1,5 @@ > /* { dg-do compile } */ > -/* { dg-options "-O2 -fdump-tree-cselim-details" } */ > +/* { dg-options "-O2 -ftree-cselim -fdump-tree-cselim-details" } */ > > int c; > unsigned test(unsigned k, unsigned b) { > diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-3.c b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-3.c > index 0fac9f9b9c7..00166373267 100644 > --- a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-3.c > +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-3.c > @@ -1,5 +1,5 @@ > /* { dg-do compile } */ > -/* { dg-options "-O2 -fdump-tree-cselim-details" } */ > +/* { dg-options "-O2 -ftree-cselim -fdump-tree-cselim-details" } */ > > unsigned a[2]; > unsigned test(unsigned k, unsigned b) { > diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-4.c b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-4.c > index 54b8c11a407..127cbdf3d10 100644 > --- a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-4.c > +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-4.c > @@ -1,5 +1,5 @@ > /* { dg-do compile } */ > -/* { dg-options "-O2 -fdump-tree-cselim-details" } */ > +/* { dg-options "-O2 -ftree-cselim -fdump-tree-cselim-details" } */ > > int *p; > unsigned test(unsigned k, unsigned b) { > diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-5.c b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-5.c > index b2d04119381..6a00f54b545 100644 > --- a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-5.c > +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-5.c > @@ -1,5 +1,5 @@ > /* { dg-do compile } */ > -/* { dg-options "-O2 -fdump-tree-cselim-details" } */ > +/* { dg-options "-O2 -ftree-cselim -fdump-tree-cselim-details" } */ > > int test(int b, int k) { > struct { > diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-6.c b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-6.c > index 8d3c4f7cc6a..ecc083ebebe 100644 > --- a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-6.c > +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-6.c > @@ -1,5 +1,5 @@ > /* { dg-do compile } */ > -/* { dg-options "-O2 -fdump-tree-cselim-details" } */ > +/* { dg-options "-O2 -ftree-cselim -fdump-tree-cselim-details" } */ > > int test(int b, int k) { > typedef struct { > diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-7-comp-ref.c b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-7-comp-ref.c > index c35a2afc70b..4fad2d1eb13 100644 > --- a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-7-comp-ref.c > +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-7-comp-ref.c > @@ -1,5 +1,5 @@ > /* { dg-do compile } */ > -/* { dg-options "-O2 -fdump-tree-cselim-details" } */ > +/* { dg-options "-O2 -ftree-cselim -fdump-tree-cselim-details" } */ > > typedef union { > int i; > diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-8-mem-ref-size.c b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-8-mem-ref-size.c > index f9e66aefb13..5f93112acf7 100644 > --- a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-8-mem-ref-size.c > +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-8-mem-ref-size.c > @@ -1,5 +1,5 @@ > /* { dg-do compile } */ > -/* { dg-options "-O2 -fdump-tree-cselim-details" } */ > +/* { dg-options "-O2 -ftree-cselim -fdump-tree-cselim-details" } */ > > int *t; > > diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr99473-1.c b/gcc/testsuite/gcc.dg/tree-ssa/pr99473-1.c > index a9fd5427694..0fda5663a80 100644 > --- a/gcc/testsuite/gcc.dg/tree-ssa/pr99473-1.c > +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr99473-1.c > @@ -1,5 +1,5 @@ > /* { dg-do compile } */ > -/* { dg-options "-O2 -fallow-store-data-races -fdump-tree-cselim-details" } */ > +/* { dg-options "-O2 -ftree-cselim -fallow-store-data-races -fdump-tree-cselim-details" } */ > > void f (int*); > > > -- > Joseph S. Myers > joseph@codesourcery.com
diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-1.c b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-1.c index 8ee1850ac63..d9fb2edf549 100644 --- a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-1.c +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-1.c @@ -1,5 +1,5 @@ /* { dg-do compile } */ -/* { dg-options "-O2 -fdump-tree-cselim-details" } */ +/* { dg-options "-O2 -ftree-cselim -fdump-tree-cselim-details" } */ unsigned test(unsigned k, unsigned b) { unsigned a[2]; diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-2.c b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-2.c index 9b96875ac7a..bb39df2be8e 100644 --- a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-2.c +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-2.c @@ -1,5 +1,5 @@ /* { dg-do compile } */ -/* { dg-options "-O2 -fdump-tree-cselim-details" } */ +/* { dg-options "-O2 -ftree-cselim -fdump-tree-cselim-details" } */ int c; unsigned test(unsigned k, unsigned b) { diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-3.c b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-3.c index 0fac9f9b9c7..00166373267 100644 --- a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-3.c +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-3.c @@ -1,5 +1,5 @@ /* { dg-do compile } */ -/* { dg-options "-O2 -fdump-tree-cselim-details" } */ +/* { dg-options "-O2 -ftree-cselim -fdump-tree-cselim-details" } */ unsigned a[2]; unsigned test(unsigned k, unsigned b) { diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-4.c b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-4.c index 54b8c11a407..127cbdf3d10 100644 --- a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-4.c +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-4.c @@ -1,5 +1,5 @@ /* { dg-do compile } */ -/* { dg-options "-O2 -fdump-tree-cselim-details" } */ +/* { dg-options "-O2 -ftree-cselim -fdump-tree-cselim-details" } */ int *p; unsigned test(unsigned k, unsigned b) { diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-5.c b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-5.c index b2d04119381..6a00f54b545 100644 --- a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-5.c +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-5.c @@ -1,5 +1,5 @@ /* { dg-do compile } */ -/* { dg-options "-O2 -fdump-tree-cselim-details" } */ +/* { dg-options "-O2 -ftree-cselim -fdump-tree-cselim-details" } */ int test(int b, int k) { struct { diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-6.c b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-6.c index 8d3c4f7cc6a..ecc083ebebe 100644 --- a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-6.c +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-6.c @@ -1,5 +1,5 @@ /* { dg-do compile } */ -/* { dg-options "-O2 -fdump-tree-cselim-details" } */ +/* { dg-options "-O2 -ftree-cselim -fdump-tree-cselim-details" } */ int test(int b, int k) { typedef struct { diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-7-comp-ref.c b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-7-comp-ref.c index c35a2afc70b..4fad2d1eb13 100644 --- a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-7-comp-ref.c +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-7-comp-ref.c @@ -1,5 +1,5 @@ /* { dg-do compile } */ -/* { dg-options "-O2 -fdump-tree-cselim-details" } */ +/* { dg-options "-O2 -ftree-cselim -fdump-tree-cselim-details" } */ typedef union { int i; diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-8-mem-ref-size.c b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-8-mem-ref-size.c index f9e66aefb13..5f93112acf7 100644 --- a/gcc/testsuite/gcc.dg/tree-ssa/pr89430-8-mem-ref-size.c +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr89430-8-mem-ref-size.c @@ -1,5 +1,5 @@ /* { dg-do compile } */ -/* { dg-options "-O2 -fdump-tree-cselim-details" } */ +/* { dg-options "-O2 -ftree-cselim -fdump-tree-cselim-details" } */ int *t; diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr99473-1.c b/gcc/testsuite/gcc.dg/tree-ssa/pr99473-1.c index a9fd5427694..0fda5663a80 100644 --- a/gcc/testsuite/gcc.dg/tree-ssa/pr99473-1.c +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr99473-1.c @@ -1,5 +1,5 @@ /* { dg-do compile } */ -/* { dg-options "-O2 -fallow-store-data-races -fdump-tree-cselim-details" } */ +/* { dg-options "-O2 -ftree-cselim -fallow-store-data-races -fdump-tree-cselim-details" } */ void f (int*);
When testing for Nios II (gcc-testresults shows this for various other targets as well), tests scanning cselim dumps produce an UNRESOLVED result because those dumps do not exist. cselim is enabled conditionally by code in toplev.c: if (flag_tree_cselim == AUTODETECT_VALUE) { if (HAVE_conditional_move) flag_tree_cselim = 1; else flag_tree_cselim = 0; } Add explicit -ftree-cselim to dg-options in the affected tests (as already used by some other tests of cselim dumps) so that this dump exists on all architectures. Tested with no regressions with cross to nios2-elf, where this causes the tests in question to PASS instead of being UNRESOLVED. OK to commit? 2021-09-07 Joseph Myers <joseph@codesourcery.com> * gcc.dg/tree-ssa/pr89430-1.c, gcc.dg/tree-ssa/pr89430-2.c, gcc.dg/tree-ssa/pr89430-3.c, gcc.dg/tree-ssa/pr89430-4.c, gcc.dg/tree-ssa/pr89430-5.c, gcc.dg/tree-ssa/pr89430-6.c, gcc.dg/tree-ssa/pr89430-7-comp-ref.c, gcc.dg/tree-ssa/pr89430-8-mem-ref-size.c, gcc.dg/tree-ssa/pr99473-1.c: Use -ftree-cselim.