This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, testsuite/ARM] Skip pr70496.c for cortex-m devices
- From: Thomas Preudhomme <thomas dot preudhomme at foss dot arm dot com>
- To: Kyrill Tkachov <kyrylo dot tkachov at foss dot arm dot com>
- Cc: gcc-patches at gcc dot gnu dot org, Ramana Radhakrishnan <Ramana dot Radhakrishnan at arm dot com>
- Date: Thu, 07 Apr 2016 17:13:27 +0100
- Subject: Re: [PATCH, testsuite/ARM] Skip pr70496.c for cortex-m devices
- Authentication-results: sourceware.org; auth=none
- References: <6491652 dot xk5pjihgLD at e108577-lin> <5704EE65 dot 6070705 at foss dot arm dot com>
On Wednesday 06 April 2016 12:09:25 Kyrill Tkachov wrote:
> Hi Thomas,
>
> On 06/04/16 12:03, Thomas Preudhomme wrote:
> > Hi,
> >
> > Testcase in gcc.target/arm/pr70496.c uses an .arm directive so assumes the
> > target has an ARM execution state. This patch adds a dg-skip-if directive
> > to skip that test on Cortex-M targets since they don't have such an
> > execution state.
> >
> > ChangeLog entry is as follows:
> >
> >
> > *** gcc/testsuite/ChangeLog ***
> >
> > 2016-04-06 Thomas Preud'homme <thomas.preudhomme@arm.com>
> >
> > PR testsuite/70553
> > * gcc.target/arm/pr70496.c: Skip for ARM Cortex-M targets.
> >
> > diff --git a/gcc/testsuite/gcc.target/arm/pr70496.c
> > b/gcc/testsuite/gcc.target/arm/pr70496.c
> > index
> > 89957e2c7a75cb89153b3e3fc34d8051b6a997d1..548a8243059ddaec63ed897dc67f4751
> > d806a065 100644
> > --- a/gcc/testsuite/gcc.target/arm/pr70496.c
> > +++ b/gcc/testsuite/gcc.target/arm/pr70496.c
> > @@ -1,6 +1,7 @@
> >
> > /* { dg-do assemble } */
> > /* { dg-options "-mthumb -O2" } */
> > /* { dg-require-effective-target arm_thumb2_ok } */
> >
> > +/* { dg-skip-if "does not have ARM state" { arm_cortex_m } } */
>
> Would it be better to just require the arm_arm_ok effective target?
> That should try to compile a test with -marm added to the command,
> which should fail for Cortex-M targets.
Fair point. What about the following patch then?
*** gcc/testsuite/ChangeLog ***
2016-04-06 Thomas Preud'homme <thomas.preudhomme@arm.com>
PR testsuite/70553
* gcc.target/arm/pr70496.c: Also require arm_arm_ok effective target.
diff --git a/gcc/testsuite/gcc.target/arm/pr70496.c
b/gcc/testsuite/gcc.target/arm/pr70496.c
index
89957e2c7a75cb89153b3e3fc34d8051b6a997d1..d3ee0b505a842268dec0fa4da09da2355a9c3715
100644
--- a/gcc/testsuite/gcc.target/arm/pr70496.c
+++ b/gcc/testsuite/gcc.target/arm/pr70496.c
@@ -1,5 +1,6 @@
/* { dg-do assemble } */
/* { dg-options "-mthumb -O2" } */
+/* { dg-require-effective-target arm_arm_ok } */
/* { dg-require-effective-target arm_thumb2_ok } */
int i;
Is this ok for trunk?
Best regards,
Thomas