Re: [PATCH] aarch64: optimize the unaligned case of memcmp

[Corinna Vinschen <vinschen@redhat.com>]

[-- Attachment #1: Type: text/plain, Size: 1183 bytes --]

On Jun 23 15:23, Sebastian Pop wrote:
> This brings to newlib a performance improvement that we developed in Bionic
> libc.  That change has been submitted for review to Bionic libc:
> https://android-review.googlesource.com/418279
> 
> A similar patch has been submitted for review in glibc:
> https://sourceware.org/ml/libc-alpha/2017-06/msg01143.html
> 
> Patch written by Vikas Sinha and Sebastian Pop.
> 
> The performance was measured on the bionic-benchmarks on a hikey (aarch64 8xA53)
> board. There was no performance change to the existing benchmark
> and a performance improvement on the new benchmark for memcmp
> on the unaligned side. The new benchmark has been submitted for
> review at https://android-review.googlesource.com/414860
> 
> The overall performance improves by 18% for the small data set 8
> and the performance improves by 450% for the large data set 64k.
> 
> The base is with the libc from /system/lib64. The bionic libc
> with this patch is in /data.
> [...]
>  newlib/libc/machine/aarch64/memcmp.S | 57 ++++++++++++++++++++++++++++++++++--

Pushed.


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]