[PATCH] AArch64 Neon Scalar Integer Compare Instructions

James Molloy james at jamesmolloy.co.uk
Fri Oct 11 08:49:05 PDT 2013


Hi Chad,

I haven't done a massively in-depth review - I compared with Tim's previous
comments and it looks like everything he mentioned is fixed, so LGTM.

Cheers,

James


On 11 October 2013 14:54, <mcrosier at codeaurora.org> wrote:

> > Hi Chad,
> >
> >> Please have a look and give the LGTM if you're satisfied.
> >
> > It looks like you've attached the arithmetic LLVM patch rather than
> > the compare one. I'm afraid someone else will have to take over the
> > review next week.
>
> Epic fail! :(
>
> Revised patches attached.  If someone else could take a look (Joey, Jim,
> James), I would greatly appreciate a review!
>
>  Chad
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131011/c0f8d97d/attachment.html>


More information about the llvm-commits mailing list