[opus] [PATCH] Optimize silk_warped_autocorrelation_FIX() for ARM NEON

James Zern jzern at google.com
Wed Feb 1 02:15:47 UTC 2017


On Tue, Jan 31, 2017 at 12:55 PM, Jean-Marc Valin <jmvalin at jmvalin.ca> wrote:
> Hi Felicia,
>
> Thanks for the patch. Can you give more details on what checks/tests
> you've done so far on this patch?
>

I did builds for x86_64 and arm (USE_CELT_FIR on and off, -O0, -O2,
gcc & clang) that checked for new warnings / c90 compatibility. The
optimized arm gcc builds (with --enable-check-asm) were used to check
bit exactness for complexity 0-10 at 8, 16, 48kHz using a few sample
rtc clips. ASan/UBSan builds pass as well.
I also made a pass through the NEON before this was forwarded, it
looks in line with the C.

> Thanks,
>
>         Jean-Marc
>
> On 31/01/17 12:30 PM, Felicia Lim wrote:
>> Hi,
>>
>> Attached is a patch with arm neon optimizations for
>> silk_warped_autocorrelation_FIX(). Please review.
>>
>> Thanks,
>> Felicia
>>
>>
>> _______________________________________________
>> opus mailing list
>> opus at xiph.org
>> http://lists.xiph.org/mailman/listinfo/opus
>>
> _______________________________________________
> opus mailing list
> opus at xiph.org
> http://lists.xiph.org/mailman/listinfo/opus


More information about the opus mailing list