<div dir="ltr">Hi Jonathan,<div><br></div><div>Thanks for pointing me to the related submissions!</div><div class="gmail_extra"><br></div><div class="gmail_extra">I noticed the xcorr_kernel_neon() assembly in current code base, but don't know why it's not activated for celt_fir() etc. So I decided to inline it inside celt_fir() to save the sum[] buffer initializing/saving/loading, and handle variable filter order "ord" differently.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Thanks,</div><div class="gmail_extra">Linfeng</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 17, 2016 at 2:37 PM, Jonathan Lennox <span dir="ltr"><<a href="mailto:jonathan@vidyo.com" target="_blank">jonathan@vidyo.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Hi, Linfeng —<br>
<br>
Please note the aarch64 optimization patches I submitted in November and December (which Tim still hasn’t gotten around to reviewing).  As they used Neon intrinsics, several of these actually applied to both armv7 and aarch64 Neon.<br>
<br>
In particular, note <a href="http://lists.xiph.org/pipermail/opus/2015-December/003339.html" rel="noreferrer" target="_blank">http://lists.xiph.org/pipermail/opus/2015-December/003339.html</a> , which added a Neon-optimized version of xcorr_kernel.  xcorr_kernel is used in celt_fir, celt_iir, and celt_pitch_xcorr.<br>
<div><div><br>
> On Jun 17, 2016, at 5:09 PM, Linfeng Zhang <<a href="mailto:linfengz@google.com" target="_blank">linfengz@google.com</a>> wrote:<br>
><br>
> Hi all,<br>
><br>
> This is Linfeng Zhang from Google. I'll work on ARM NEON optimization in the<br>
> next few months.<br>
><br>
> I'm submitting 2 patches in the following couple of emails, which have the new<br>
> created celt_fir_neon().<br>
><br>
> I revised celt_fir_c() to not pass in argument "mem" in Patch 1. If there are<br>
> concerns to this change, please let me know.<br>
><br>
> Many thanks to your comments.<br>
><br>
> Linfeng Zhang<br>
><br>
</div></div>> _______________________________________________<br>
> opus mailing list<br>
> <a href="mailto:opus@xiph.org" target="_blank">opus@xiph.org</a><br>
> <a href="http://lists.xiph.org/mailman/listinfo/opus" rel="noreferrer" target="_blank">http://lists.xiph.org/mailman/listinfo/opus</a><br>
<br>
</blockquote></div><br></div></div>