Re: [PATCH] RISC-V: testsuite: Drop march-string in cpymemsi-1.c

[Christoph Müllner]

On Thu, May 16, 2024 at 10:03 AM Kito Cheng <kito.ch...@gmail.com> wrote:
>
> Just one minor question
>
> > diff --git a/gcc/testsuite/gcc.target/riscv/cpymemsi-1.c 
> > b/gcc/testsuite/gcc.target/riscv/cpymemsi-1.c
> > index 983b564ccaf..aee54d9aa00 100644
> > --- a/gcc/testsuite/gcc.target/riscv/cpymemsi-1.c
> > +++ b/gcc/testsuite/gcc.target/riscv/cpymemsi-1.c
> > @@ -1,6 +1,5 @@
> >  /* { dg-do run } */
> > -/* { dg-options "-march=rv32gc -save-temps -g0 -fno-lto" { target { rv32 } 
> > } } */
> > -/* { dg-options "-march=rv64gc -save-temps -g0 -fno-lto" { target { rv64 } 
> > } } */
> > +/* { dg-options "-save-temps -g0 -fno-lto" } */
>
> I know -save-temps -g0 already exists, but I am wondering why we need
> those 2 options here?

I copied from gcc/testsuite/gcc.dg/torture/inline-mem-cmp-1.c.
But you are right, we could drop these as well
(gcc/testsuite/gcc.target/powerpc/block-cmp-8.c also did so).
I'll retest/resend without the whole dg-options line.