Re: [RFC PATCH v1 0/3] Introducing ARM SIMD Support

On 1 October 2014 12:35, Ralph Giles <[email protected]> wrote:

> On 2014-10-01 10:00 AM, Viswanath Puttagunta wrote:
>
> > I would like to get feedback on "1" above from this list. I first want
> > to make sure the modifications to autoconf files is in line with the
> > project.
>
> Not having looked at the patches, you should compare your work with the
> instruction set detection in the opus implementation. That has worked
> well so far, if you can say you did the same thing, it will be easier
> for the authors to review it.
>
>   https://git.xiph.org/?p=opus.git;a=summary

  >> Thanks, will do.

>
>
> > For "2", which is ARM NEON specific reviews, if necessary, I can ping a
> > few of ARM's NEON gurus and I think they will be happy to join this list
> > and review the code.
>
> Do you have output verification and benchmark results you can share?
>
>> As mentioned in cover letter for patches, available here
<https://docs.google.com/a/vishndivi.com/document/d/1LuMORT2jIroiuDKAQaj7YpizCRQBpgruC8DKDlzc8ik/edit>
.

>
>  -r
>
>

_______________________________________________
Vorbis-dev mailing list
[email protected]
http://lists.xiph.org/mailman/listinfo/vorbis-dev