Re: [PATCH] [ARM] neon-testgen.ml typo

2015-11-24 Thread Christophe Lyon
On 13 November 2015 at 14:36, Christophe Lyon wrote: > On 13 November 2015 at 12:24, Kyrill Tkachov wrote: >> >> On 13/11/15 11:18, Ramana Radhakrishnan wrote: Hmm. I hadn't noticed that the crypto intrinsics tests where generated by

Re: [PATCH] [ARM] neon-testgen.ml typo

2015-11-13 Thread Kyrill Tkachov
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

Re: [PATCH] [ARM] neon-testgen.ml typo

2015-11-13 Thread Christophe Lyon
On 12 November 2015 at 23:18, Christophe Lyon wrote: > On 6 November 2015 at 21:29, Christophe Lyon > wrote: >> On 4 November 2015 at 13:16, Ramana Radhakrishnan >> wrote: >>> On Fri, Oct 30, 2015 at 2:42 PM,

Re: [PATCH] [ARM] neon-testgen.ml typo

2015-11-13 Thread Ramana Radhakrishnan
> 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

Re: [PATCH] [ARM] neon-testgen.ml typo

2015-11-13 Thread Kyrill Tkachov
Hi Christophe, On 13/11/15 10:24, Christophe Lyon wrote: On 12 November 2015 at 23:18, Christophe Lyon wrote: On 6 November 2015 at 21:29, Christophe Lyon wrote: On 4 November 2015 at 13:16, Ramana Radhakrishnan

Re: [PATCH] [ARM] neon-testgen.ml typo

2015-11-13 Thread Christophe Lyon
On 13 November 2015 at 12:24, Kyrill Tkachov 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

Re: [PATCH] [ARM] neon-testgen.ml typo

2015-11-12 Thread Christophe Lyon
On 6 November 2015 at 21:29, Christophe Lyon wrote: > On 4 November 2015 at 13:16, Ramana Radhakrishnan > wrote: >> On Fri, Oct 30, 2015 at 2:42 PM, Christophe Lyon >> wrote: >>> On 30 October 2015 at 15:33,

Re: [PATCH] [ARM] neon-testgen.ml typo

2015-11-06 Thread Christophe Lyon
On 4 November 2015 at 13:16, Ramana Radhakrishnan wrote: > On Fri, Oct 30, 2015 at 2:42 PM, Christophe Lyon > wrote: >> On 30 October 2015 at 15:33, Ramana Radhakrishnan >> wrote: >>> >>> >>> On 29/10/15

Re: [PATCH] [ARM] neon-testgen.ml typo

2015-11-04 Thread Ramana Radhakrishnan
On Fri, Oct 30, 2015 at 2:42 PM, Christophe Lyon wrote: > On 30 October 2015 at 15:33, Ramana Radhakrishnan > wrote: >> >> >> On 29/10/15 17:23, Jim Wilson wrote: >>> I noticed a comment typo in this file while using grep to look for

Re: [PATCH] [ARM] neon-testgen.ml typo

2015-11-02 Thread Kyrill Tkachov
On 02/11/15 12:06, Julian Brown wrote: Hi, On Thu, 29 Oct 2015 10:23:58 -0700 Jim Wilson wrote: I noticed a comment typo in this file while using grep to look for other stuff. The typo is easy to fix. I tried running neon-testgen.ml to verify, but it is apparently

Re: [PATCH] [ARM] neon-testgen.ml typo

2015-11-02 Thread Julian Brown
Hi, On Thu, 29 Oct 2015 10:23:58 -0700 Jim Wilson wrote: > I noticed a comment typo in this file while using grep to look for > other stuff. The typo is easy to fix. > > I tried running neon-testgen.ml to verify, but it is apparently no > longer valid ocaml, as it

Re: [PATCH] [ARM] neon-testgen.ml typo

2015-10-30 Thread Ramana Radhakrishnan
On 29/10/15 17:23, Jim Wilson wrote: > I noticed a comment typo in this file while using grep to look for > other stuff. The typo is easy to fix. > > I tried running neon-testgen.ml to verify, but it is apparently no > longer valid ocaml, as it doesn't work with the ocamlc 4.01.0 I have > on

Re: [PATCH] [ARM] neon-testgen.ml typo

2015-10-30 Thread Christophe Lyon
On 30 October 2015 at 15:33, Ramana Radhakrishnan wrote: > > > On 29/10/15 17:23, Jim Wilson wrote: >> I noticed a comment typo in this file while using grep to look for >> other stuff. The typo is easy to fix. >> >> I tried running neon-testgen.ml to verify,

[PATCH] [ARM] neon-testgen.ml typo

2015-10-29 Thread Jim Wilson
I noticed a comment typo in this file while using grep to look for other stuff. The typo is easy to fix. I tried running neon-testgen.ml to verify, but it is apparently no longer valid ocaml, as it doesn't work with the ocamlc 4.01.0 I have on Ubuntu 14.04. I get a syntax error. Someone who