on 2024/4/22 17:23, Alexandre Oliva wrote: > > These ppc lp64 tests check for errors or warnings on -mno-powerpc64. > On powerpc64-*-vxworks* we get the same errors as on most other > covered platforms, but the tests did not mark them as expected for > this target. On powerpc-*-vxworks*, the tests are skipped because > lp64 is not satisfied, so I'm naming powerpc*-*-vxworks* rather than > something more specific. > > Regstrapped on x86_64-linux-gnu and ppc64el-linux-gnu. Also tested with > gcc-13 on ppc64-vx7r2 and ppc-vx7r2. Ok to install?
OK, thanks! BR, Kewen > > > for gcc/testsuite/ChangeLog > > * gcc.target/powerpc/pr106680-1.c: Error on vxworks too. > * gcc.target/powerpc/pr106680-2.c: Likewise. > * gcc.target/powerpc/pr106680-3.c: Likewise. > --- > gcc/testsuite/gcc.target/powerpc/pr106680-1.c | 2 +- > gcc/testsuite/gcc.target/powerpc/pr106680-2.c | 2 +- > gcc/testsuite/gcc.target/powerpc/pr106680-3.c | 2 +- > 3 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/gcc/testsuite/gcc.target/powerpc/pr106680-1.c > b/gcc/testsuite/gcc.target/powerpc/pr106680-1.c > index d624d43230a7a..aadaa614cfeba 100644 > --- a/gcc/testsuite/gcc.target/powerpc/pr106680-1.c > +++ b/gcc/testsuite/gcc.target/powerpc/pr106680-1.c > @@ -8,6 +8,6 @@ int foo () > return 1; > } > > -/* { dg-error "'-m64' requires a PowerPC64 cpu" "PR106680" { target > powerpc*-*-linux* powerpc*-*-freebsd* powerpc-*-rtems* } 0 } */ > +/* { dg-error "'-m64' requires a PowerPC64 cpu" "PR106680" { target > powerpc*-*-linux* powerpc*-*-freebsd* powerpc-*-rtems* powerpc*-*-vxworks* } > 0 } */ > /* { dg-warning "'-m64' requires PowerPC64 architecture, enabling" > "PR106680" { target powerpc*-*-darwin* } 0 } */ > /* { dg-warning "'-maix64' requires PowerPC64 architecture remain enabled" > "PR106680" { target powerpc*-*-aix* } 0 } */ > diff --git a/gcc/testsuite/gcc.target/powerpc/pr106680-2.c > b/gcc/testsuite/gcc.target/powerpc/pr106680-2.c > index a9ed73726ef0c..f0758e303350a 100644 > --- a/gcc/testsuite/gcc.target/powerpc/pr106680-2.c > +++ b/gcc/testsuite/gcc.target/powerpc/pr106680-2.c > @@ -9,6 +9,6 @@ int foo () > return 1; > } > > -/* { dg-error "'-m64' requires a PowerPC64 cpu" "PR106680" { target > powerpc*-*-linux* powerpc*-*-freebsd* powerpc-*-rtems* } 0 } */ > +/* { dg-error "'-m64' requires a PowerPC64 cpu" "PR106680" { target > powerpc*-*-linux* powerpc*-*-freebsd* powerpc-*-rtems* powerpc*-*-vxworks* } > 0 } */ > /* { dg-warning "'-m64' requires PowerPC64 architecture, enabling" > "PR106680" { target powerpc*-*-darwin* } 0 } */ > /* { dg-warning "'-maix64' requires PowerPC64 architecture remain enabled" > "PR106680" { target powerpc*-*-aix* } 0 } */ > diff --git a/gcc/testsuite/gcc.target/powerpc/pr106680-3.c > b/gcc/testsuite/gcc.target/powerpc/pr106680-3.c > index b642d5c7a008d..bca012e2cf663 100644 > --- a/gcc/testsuite/gcc.target/powerpc/pr106680-3.c > +++ b/gcc/testsuite/gcc.target/powerpc/pr106680-3.c > @@ -8,6 +8,6 @@ int foo () > return 1; > } > > -/* { dg-error "'-m64' requires a PowerPC64 cpu" "PR106680" { target > powerpc*-*-linux* powerpc*-*-freebsd* powerpc-*-rtems* } 0 } */ > +/* { dg-error "'-m64' requires a PowerPC64 cpu" "PR106680" { target > powerpc*-*-linux* powerpc*-*-freebsd* powerpc-*-rtems* powerpc*-*-vxworks* } > 0 } */ > /* { dg-warning "'-m64' requires PowerPC64 architecture, enabling" > "PR106680" { target powerpc*-*-darwin* } 0 } */ > /* { dg-warning "'-maix64' requires PowerPC64 architecture remain enabled" > "PR106680" { target powerpc*-*-aix* } 0 } */ >