> -----Original Message----- > From: Andrew Bennett [mailto:andrew.benn...@imgtec.com] > Sent: Wednesday, January 20, 2016 9:42 AM > To: Matthew Fortune; gcc-patches@gcc.gnu.org > Cc: Moore, Catherine > Subject: RE: [PATCH] MIPS: Prevent the p5600-bonding.c test from being run > for the n32 and 64 ABIs > > Ping. >
This is OK now. > > > -----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))));
