On Wed, Sep 8, 2021 at 12:05 AM Joseph Myers <jos...@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 <jos...@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
> jos...@codesourcery.com
