Re: [PATCH, ARM] Restrict pr65647 testcase to ARMv6-M effective target

[Richard Earnshaw <Richard.Earnshaw@foss.arm.com>]

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.

R.

> Best regards,
> 
> Thomas
> 
>