On Wed, Oct 8, 2025 at 12:00 PM Dimitar Dimitrov <[email protected]> wrote:
>
> The cselim pass is enabled only for targets that have conditional move
> instructions.  Since pru-unknown-elf doesn't have such instructions, the
> pass is not executed, and the test fails with:
>
>   gcc.dg/tree-ssa/cselim-2.c: dump file does not exist
>   UNRESOLVED: gcc.dg/tree-ssa/cselim-2.c scan-tree-dump cselim "if-then-else 
> store replacement: 3"
>
> Fix by explicitly enabling the cselim pass for this test.
>
> Ok for trunk?

LGTM. Based on the other testcases that look at cselim dump, this is
the same method of fixing it there.
Plus this was my oversight sorry I had missed that when adding the testcase.

Thanks,
Andrew

>
> gcc/testsuite/ChangeLog:
>
>         * gcc.dg/tree-ssa/cselim-2.c: Pass -ftree-cselim option.
>
> Signed-off-by: Dimitar Dimitrov <[email protected]>
> ---
>  gcc/testsuite/gcc.dg/tree-ssa/cselim-2.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gcc/testsuite/gcc.dg/tree-ssa/cselim-2.c 
> b/gcc/testsuite/gcc.dg/tree-ssa/cselim-2.c
> index 2964fcfe1ff..db58dd22e8a 100644
> --- a/gcc/testsuite/gcc.dg/tree-ssa/cselim-2.c
> +++ b/gcc/testsuite/gcc.dg/tree-ssa/cselim-2.c
> @@ -1,5 +1,5 @@
>  /* { dg-do compile } */
> -/* { dg-options "-O2 -fdump-tree-cselim-stats -fno-ssa-phiopt" } */
> +/* { dg-options "-O2 -ftree-cselim -fdump-tree-cselim-stats -fno-ssa-phiopt" 
> } */
>
>  struct Loc {
>      int x[3];
> --
> 2.51.0
>

Reply via email to