It's indeed fixed now. Thanks.

Best regards,

Thomas

> -----Original Message-----
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
> ow...@gcc.gnu.org] On Behalf Of Christian Bruel
> Sent: Wednesday, December 09, 2015 6:13 PM
> To: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH, testsuite] Fix PR68632: gcc.target/arm/lto/pr65837
> failure on M profile ARM targets
> 
> Hi Thomas,
> 
> 
> On 12/09/2015 10:57 AM, Thomas Preud'homme wrote:
> > gcc.target/arm/lto/pr65837 fails on M profile ARM targets because of
> lack of neon instructions. This patch adds the necessary arm_neon_ok
> effective target requirement to avoid running this test for such targets.
> >
> 
> This case also fails for all configs that don't have neon by default.
> This is being fixed with
> 
> https://gcc.gnu.org/ml/gcc-patches/2015-12/msg00865.html
> 
> 
> > ChangeLog entry is as follows:
> >
> >
> > * gcc/testsuite/ChangeLog ***
> >
> > 2015-12-08  Thomas Preud'homme  <thomas.preudho...@arm.com>
> >
> >          PR testsuite/68632
> >          * gcc.target/arm/lto/pr65837_0.c: Require arm_neon_ok effective
> >          target.
> >
> >
> > diff --git a/gcc/testsuite/gcc.target/arm/lto/pr65837_0.c
> b/gcc/testsuite/gcc.target/arm/lto/pr65837_0.c
> > index 000fc2a..fcc26a1 100644
> > --- a/gcc/testsuite/gcc.target/arm/lto/pr65837_0.c
> > +++ b/gcc/testsuite/gcc.target/arm/lto/pr65837_0.c
> > @@ -1,4 +1,5 @@
> >   /* { dg-lto-do run } */
> > +/* { dg-require-effective-target arm_neon_ok } */
> >   /* { dg-lto-options {{-flto -mfpu=neon}} } */
> >   /* { dg-suppress-ld-options {-mfpu=neon} } */
> >
> >
> >
> > Testcase fails without the patch and succeeds with.
> >
> > Is this ok for trunk?
> >
> > Best regards,
> >
> > Thomas
> >
> >


Reply via email to