<div dir="ltr">Hi Tim,<div><br></div><div>Thanks for the speed testing and patch.</div><div><br></div><div>Yes, I agree that it makes sense to reuse <span style="font-size:12.8px">xcorr_kernel_neon() in </span><span style="font-size:12.8px">SMALL_FOOTPRINT to save code size.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">For non-</span><span style="font-size:12.8px">SMALL_FOOTPRINT, (1710-1695)/1710 = 0.88%. Maybe it's worth to have </span><span style="font-size:12.8px">celt_fir_neon() there.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">However, considering that now </span><span style="font-size:12.8px">celt_fir() is disabled by default, </span><span style="font-size:12.8px">it's up to you if this patch (</span><span style="font-size:12.8px">and the previous one) should be skipped</span><span style="font-size:12.8px">. Let me know and I'll rebase all following patches which are based on them.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Thanks,</span></div><div><span style="font-size:12.8px">Linfeng</span></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 27, 2016 at 6:45 PM, Timothy B. Terriberry <span dir="ltr"><<a href="mailto:tterribe@xiph.org" target="_blank">tterribe@xiph.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Timothy B. Terriberry wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
and use that. I wrote a simple patch demonstrating this (attached... it<br>
</blockquote>
<br></span>
Err, actually attached this time.<br>
<br>
<br>______________________________<wbr>_________________<br>
opus mailing list<br>
<a href="mailto:opus@xiph.org">opus@xiph.org</a><br>
<a href="http://lists.xiph.org/mailman/listinfo/opus" rel="noreferrer" target="_blank">http://lists.xiph.org/mailman/<wbr>listinfo/opus</a><br>
<br></blockquote></div><br></div>