This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, ARM] Restrict pr65647 testcase to ARMv6-M effective target
- From: James Greenhalgh <james dot greenhalgh at arm dot com>
- To: Richard Earnshaw <Richard dot Earnshaw at foss dot arm dot com>
- Cc: "Thomas Preud'homme" <Thomas dot Preudhomme at arm dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Ramana Radhakrishnan <Ramana dot Radhakrishnan at arm dot com>, Kyrylo Tkachov <Kyrylo dot Tkachov at arm dot com>
- Date: Fri, 26 Jun 2015 11:15:06 +0100
- Subject: Re: [PATCH, ARM] Restrict pr65647 testcase to ARMv6-M effective target
- Authentication-results: sourceware.org; auth=none
- References: <003001d0afec$6de65a60$49b30f20$ at arm dot com> <558D1259 dot 8090803 at foss dot arm dot com>
On Fri, Jun 26, 2015 at 09:50:33AM +0100, Richard Earnshaw wrote:
> On 26/06/15 09:45, Thomas Preud'homme wrote:
> > Hi,
> >
> > Testcase for PR65647 assumes that the compiler can compile for ARMv6-M
> > which might not be the case if passing some extra options via
> > RUNTESTFLAGS (eg. -marm/-mcpu=cortex-a9). This patch restricts the
> > testcase to ARMv6-M effective targets.
> >
> >
> > Testsuite ChangeLog entry is as follows:
> >
> > 2015-06-25 Thomas Preud'homme <thomas.preudhomme@arm.com>
> >
> > * gcc.target/arm/pr65647.c: Restrict to ARMv6-M effective targets.
> >
> >
> > diff --git a/gcc/testsuite/gcc.target/arm/pr65647.c b/gcc/testsuite/gcc.target/arm/pr65647.c
> > index d3b44b2..d828d23 100644
> > --- a/gcc/testsuite/gcc.target/arm/pr65647.c
> > +++ b/gcc/testsuite/gcc.target/arm/pr65647.c
> > @@ -1,4 +1,5 @@
> > /* { dg-do compile } */
> > +/* { dg-require-effective-target arm_arch_v6m_ok } */
> > /* { dg-options "-march=armv6-m -mthumb -O3 -w -mfloat-abi=soft" } */
> >
> > a, b, c, e, g = &e, h, i = 7, l = 1, m, n, o, q = &m, r, s = &r, u, w = 9, x,
> >
> >
> > Patch was tested by running the testcase once with -mcpu=cortex-a9
> > (skipped as expected) and once with -mcpu=cortex-m0 (passes).
> >
> > Is this ok for trunk?
> >
>
> OK.
This should already have been covered by:
https://gcc.gnu.org/ml/gcc-patches/2015-06/msg01105.html
2015-06-16 James Greenhalgh <james.greenhalgh@arm.com>
* gcc.target/arm/pr65647.c: Do not override -mfloat-abi directives
passed by the testsuite driver.
Thanks,
James
---
diff --git a/gcc/testsuite/gcc.target/arm/pr65647.c b/gcc/testsuite/gcc.target/arm/pr65647.c
index d3b44b2..26b4e39 100644
--- a/gcc/testsuite/gcc.target/arm/pr65647.c
+++ b/gcc/testsuite/gcc.target/arm/pr65647.c
@@ -1,4 +1,6 @@
/* { dg-do compile } */
+/* { dg-require-effective-target arm_arch_v6m_ok } */
+/* { dg-skip-if "do not override -mfloat-abi" { *-*-* } { "-mfloat-abi=*" } {"-mfloat-abi=soft" } } */
/* { dg-options "-march=armv6-m -mthumb -O3 -w -mfloat-abi=soft" } */
a, b, c, e, g = &e, h, i = 7, l = 1, m, n, o, q = &m, r, s = &r, u, w = 9, x,
> > Best regards,
> >
> > Thomas
> >
> >
>