| Message ID | 20200212222701.GB17695@tucnak |
|---|---|
| State | New |
| Headers | show |
| Series | [committed] testsuite: Fix up gcc.target/powerpc/pr93122.c test | expand |
On Wed, Feb 12, 2020 at 11:27:01PM +0100, Jakub Jelinek wrote: > On Mon, Feb 10, 2020 at 01:45:42PM -0500, Michael Meissner wrote: > > This patch renames the PowerPC internal switch -mprefixed-addr to be > > -mprefixed. > This change broke the gcc.target/powerpc/pr93122.c test, so it now > FAIL: gcc.target/powerpc/pr93122.c (test for excess errors) > Excess errors: > xgcc: error: unrecognized command-line option '-mprefixed-addr'; did you mean '-mprefixed'? > > Fixed thusly, bootstrapped/regtested on powerpc64le-linux, committed to > trunk as obvious. Thanks! Segher
On Wed, Feb 12, 2020 at 11:27:01PM +0100, Jakub Jelinek wrote: > On Mon, Feb 10, 2020 at 01:45:42PM -0500, Michael Meissner wrote: > > This patch renames the PowerPC internal switch -mprefixed-addr to be > > -mprefixed. > > --- gcc/config/rs6000/rs6000.opt > +++ gcc/config/rs6000/rs6000.opt > @@ -570,8 +570,8 @@ mfuture > Target Report Mask(FUTURE) Var(rs6000_isa_flags) > Use instructions for a future architecture. > > -mprefixed-addr > -Target Undocumented Mask(PREFIXED_ADDR) Var(rs6000_isa_flags) > +mprefixed > +Target Report Mask(PREFIXED) Var(rs6000_isa_flags) > Generate (do not generate) prefixed memory instructions. > > mpcrel > > This change broke the gcc.target/powerpc/pr93122.c test, so it now > FAIL: gcc.target/powerpc/pr93122.c (test for excess errors) > Excess errors: > xgcc: error: unrecognized command-line option '-mprefixed-addr'; did you mean '-mprefixed'? > > Fixed thusly, bootstrapped/regtested on powerpc64le-linux, committed to > trunk as obvious. Thanks. I don't think that test was in the trunk when I did the the bootstrap for the -mprefixed-addr to -mprefixed option. I was about to send a similar patch.
--- gcc/config/rs6000/rs6000.opt +++ gcc/config/rs6000/rs6000.opt @@ -570,8 +570,8 @@ mfuture Target Report Mask(FUTURE) Var(rs6000_isa_flags) Use instructions for a future architecture. -mprefixed-addr -Target Undocumented Mask(PREFIXED_ADDR) Var(rs6000_isa_flags) +mprefixed +Target Report Mask(PREFIXED) Var(rs6000_isa_flags) Generate (do not generate) prefixed memory instructions. mpcrel This change broke the gcc.target/powerpc/pr93122.c test, so it now FAIL: gcc.target/powerpc/pr93122.c (test for excess errors) Excess errors: xgcc: error: unrecognized command-line option '-mprefixed-addr'; did you mean '-mprefixed'? Fixed thusly, bootstrapped/regtested on powerpc64le-linux, committed to trunk as obvious. 2020-02-12 Jakub Jelinek <jakub@redhat.com> * gcc.target/powerpc/pr93122.c: Use -mprefixed instead of -mprefixed-addr in dg-options. --- gcc/testsuite/gcc.target/powerpc/pr93122.c +++ gcc/testsuite/gcc.target/powerpc/pr93122.c @@ -1,6 +1,6 @@ /* PR target/93122 */ /* { dg-do compile { target lp64 } } */ -/* { dg-options "-fstack-clash-protection -mprefixed-addr -mfuture" } */ +/* { dg-options "-fstack-clash-protection -mprefixed -mfuture" } */ void bar (char *);