[PATCH v2] Fix strrchr regression.
Liubov Dmitrieva
liubov.dmitrieva@gmail.com
Fri Aug 16 11:12:00 GMT 2013
Can you please attach the patch, not only Change Log. I don't see the
patch itself.
I will retest on Atom and Silvermont today.
--
Liubov Dmitrieva
On Fri, Aug 16, 2013 at 3:06 PM, Ondřej Bílka <neleai@seznam.cz> wrote:
> On Tue, Aug 06, 2013 at 10:34:30AM +0200, Ondřej Bílka wrote:
> Hi, according to profiling feedback I tuned strrchr implementation to
> much faster one.
>
> See results at:
> http://kam.mff.cuni.cz/~ondra/benchmark_string/strrchr_profile.html
> are updated and benchmark program is here:
> http://kam.mff.cuni.cz/~ondra/benchmark_string/strrchr_profile160813.tar.bz2
>
> I do not cover adding avx2 implementation which will be posted
> separately. A considerable speedup is possible there.
> http://kam.mff.cuni.cz/~ondra/benchmark_string/haswell/strrchr_profile/results_rand/result.html
>
> I need to retest it on atom and silvermont to verify how much I
> improved.
>
> Passes tests, OK to commit?
>
> Ondra
>
> * sysdeps/x86_64/multiarch/Makefile (sysdep_routines): Update.
> * sysdeps/x86_64/multiarch/ifunc-impl-list.c: Remove strrchr ifunc.
> * sysdeps/x86_64/multiarch/strend-sse4.S Remove.
> * sysdeps/x86_64/multiarch/strrchr-sse2-no-bsf.S Likewise.
> * sysdeps/x86_64/multiarch/strrchr.S: Likewise.
> * sysdeps/x86_64/strrchr.S (strrchr): Use optimized implementation.
>
> ---
More information about the Libc-alpha
mailing list