Ping. Andrew
> -----Original Message----- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On > Behalf Of Andrew Bennett > Sent: 02 September 2015 14:55 > To: Matthew Fortune; gcc-patches@gcc.gnu.org > Cc: Moore, Catherine (catherine_mo...@mentor.com) > Subject: RE: [PATCH] MIPS: Prevent the p5600-bonding.c test from being run for > the n32 and 64 ABIs > > > > diff --git a/gcc/testsuite/gcc.target/mips/p5600-bonding.c > > > b/gcc/testsuite/gcc.target/mips/p5600-bonding.c > > > index 0890ffa..20c26ca 100644 > > > --- a/gcc/testsuite/gcc.target/mips/p5600-bonding.c > > > +++ b/gcc/testsuite/gcc.target/mips/p5600-bonding.c > > > @@ -1,6 +1,7 @@ > > > /* { dg-do compile } */ > > > /* { dg-options "-dp -mtune=p5600 -mno-micromips -mno-mips16" } */ > > > /* { dg-skip-if "Bonding needs peephole optimization." { *-*-* } { "-O0" > "- > > O1" } { "" } } > > > */ > > > +/* { dg-skip-if "There is no DI mode support for load/store bonding" { *- > *- > > * } { "- > > > mabi=n32" "-mabi=64" } { "" } } */ > > > typedef int VINT32 __attribute__ ((vector_size((16)))); > > > > If the best fix we can do for this test is to limit what it tests then we > > should still not just skip it. There is some precedence for tests that > > require a specific arch with the isa=loongson special case. I'd rather > > just lock the test down to p5600 as per the filename. > > I have changed the testcase's dg-options so that it is only built for p5600. > The updated patch and ChangeLog are below. > > Ok to commit? > > Many thanks, > > > > Andrew > > > testsuite/ > * gcc.target/mips/p5600-bonding.c (dg-options): Force the test to be > always > built for p5600. > * gcc.target/mips/mips.exp (mips-dg-options): Add support for the > isa=p5600 > dg-option. > > > diff --git a/gcc/testsuite/gcc.target/mips/mips.exp > b/gcc/testsuite/gcc.target/mips/mips.exp > index 42e7fff..e8d1895 100644 > --- a/gcc/testsuite/gcc.target/mips/mips.exp > +++ b/gcc/testsuite/gcc.target/mips/mips.exp > @@ -142,6 +142,9 @@ > # isa=loongson > # select a Loongson processor > # > +# isa=p5600 > +# select a P5600 processor > +# > # addressing=absolute > # force absolute addresses to be used > # > @@ -1009,6 +1012,10 @@ proc mips-dg-options { args } { > if { ![regexp {^-march=loongson} $arch] } { > set arch "-march=loongson2f" > } > + } elseif { [string equal $spec "isa=p5600"] } { > + if { ![regexp {^-march=p5600} $arch] } { > + set arch "-march=p5600" > + } > } else { > if { ![regexp {^(isa(?:|_rev))(=|<=|>=)([0-9]*)$} \ > $spec dummy prop relation value nocpus] } { > diff --git a/gcc/testsuite/gcc.target/mips/p5600-bonding.c > b/gcc/testsuite/gcc.target/mips/p5600-bonding.c > index 0890ffa..0bc6d91 100644 > --- a/gcc/testsuite/gcc.target/mips/p5600-bonding.c > +++ b/gcc/testsuite/gcc.target/mips/p5600-bonding.c > @@ -1,5 +1,5 @@ > /* { dg-do compile } */ > -/* { dg-options "-dp -mtune=p5600 -mno-micromips -mno-mips16" } */ > +/* { dg-options "-dp isa=p5600 -mtune=p5600 -mno-micromips -mno-mips16" } */ > /* { dg-skip-if "Bonding needs peephole optimization." { *-*-* } { "-O0" "- > O1" } { "" } } */ > typedef int VINT32 __attribute__ ((vector_size((16))));