This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[PING] [PATCH, ARM] Fix PR45094

Ramana Radhakrishnan wrote:
>>> No - this test is still not ok. It is not necessary that all
>>> arm-linux-gnueabi targets have Neon. Think about the Tegra2 where
>>> there is no Neon unit where this test will fail. Thus if you really
>>> need the neon options (out of curiosity why?) please add the following
>>> to the test.
>>>
>>> /* { dg-require-effective-target arm_neon_hw } */
>> Add  /* { dg-require-effective-target arm_neon_hw } */ into the test
>> as you suggested here.  Re-create patch against mainline trunk, and
>> tested.  OK to apply?
> 
> Yes this looks much better - thanks. This should go in under the
> "obvious" rule but please wait for a backend maintainer to comment.
>  

Here is the latest patch to fix GCC PR45094, after Ramana
Radhakrishnan's review.
http://gcc.gnu.org/ml/gcc-patches/2010-08/msg00249.html

-- 
Yao Qi
CodeSourcery
yao@codesourcery.com
(650) 331-3385 x739
Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]