This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] PowerPC: optimized strpbrk for POWER7
- From: Steven Munroe <munroesj at linux dot vnet dot ibm dot com>
- To: Adhemerval Zanella <azanella at linux dot vnet dot ibm dot com>
- Cc: libc-alpha at sourceware dot org
- Date: Thu, 20 Mar 2014 17:25:28 -0500
- Subject: Re: [PATCH] PowerPC: optimized strpbrk for POWER7
- Authentication-results: sourceware.org; auth=none
- References: <531B889F dot 2020505 at linux dot vnet dot ibm dot com>
- Reply-to: munroesj at us dot ibm dot com
On Sat, 2014-03-08 at 18:16 -0300, Adhemerval Zanella wrote:
> I am resending this patch to keep track using the patchwork and with
> some corrections (string/strpbrk.c cleanup and the no need to align
> stack, since stack is already aligned to 16 bytes in PPC64 abi).
>
> This patch add an optimized strpbrk for POWER7 by using a different
> algorithm than default implementation: it constructs a table based on
> the 'accept' argument and use this table to check for any occurance on
> the input string. The idea is similar as x86_64 uses.
> For PowerPC some tunings were added, such as unroll loops and memory
> clear using VSX instructions.
>
> --
>
> 2014-03-08 Adhemerval Zanella <azanella@linux.vnet.ibm.com>
> Vidya Ranganathan <vidya@linux.vnet.ibm.com>
>
> * string/strpbrk.c (strpbrk): Using macro to redefine symbol name.
> * sysdeps/powerpc/powerpc64/multiarch/Makefile: Add strpbrk-power7
> and strpbrk-ppc64 objects.
> * sysdeps/powerpc/powerpc64/multiarch/ifunc-impl-list.c
> (__libc_ifunc_impl_list): Add new strpbrk optimized symbols.
> * sysdeps/powerpc/powerpc64/multiarch/strpbrk-power7.S: New file:
> multiarch strpbrk for POWER7.
> * sysdeps/powerpc/powerpc64/multiarch/strpbrk-ppc64.c: New file:
> multiarch strpbrk for PPC64.
> * sysdeps/powerpc/powerpc64/multiarch/strpbrk.c: New file: strpbrk
> ifunc selector.
> * sysdeps/powerpc/powerpc64/power7/strpbrk.S: New file: optimited
> strpbrk for POWER7.
>
> ---
looks ok.