On 13 November 2015 at 12:24, Kyrill Tkachov <kyrylo.tkac...@arm.com> wrote: > > On 13/11/15 11:18, Ramana Radhakrishnan wrote: >>> >>> Hmm. I hadn't noticed that the crypto intrinsics tests where generated by >>> neon-testgen.ml, I thought they were hand-written. >>> The tests I added do not cover the crypto intrinsics, so I'm going >>> to revert r230274 and restore all the tests generated by neon-testgen.ml >>> until we have better coverage in advsimd-intrinsics. >> >> From what I remember from a few days back I thought it was generally >> ok to get rid of the lot as we had test coverage for everything else >> in gcc.target/arm . >> >> Thus don't bother reverting. > > > +1. I'll also add that you can now remove neon.ml from config/arm.
Sorry, I felt so guilty that I reverted the removal already. > And also, I think we can move the remaining hand-written tests from > gcc.target/arm/neon/ > into gcc.target/arm/ and remove the neon/ directory altogether. > I'll give a deeper look at all that, and will try not to rush to meet e/o stage1 deadline :-) > Kyrill > > >> >> Ramana >> >>> Sorry for the oversight. >>> >>> Christophe. >>> >>> >>>> Christophe. >>>> >>>>>> regards >>>>>> Ramana >>>>>> >>>>>> >>>>>> >>>>>>> Christophe. >>>>>>> >>>>>>>> regards >>>>>>>> Ramana > >