From a7fbedff76494b5b694f647e9c681cf8988056ce Mon Sep 17 00:00:00 2001 From: "H.J. Lu" Date: Thu, 18 May 2017 14:02:02 -0700 Subject: Correct comments in x86_64/multiarch/memcmp.S * sysdeps/x86_64/multiarch/memcmp.S (__GI_memcmp): Correct comments. --- ChangeLog | 5 +++++ sysdeps/x86_64/multiarch/memcmp.S | 6 +++--- 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/ChangeLog b/ChangeLog index 06ef5789b0..5ebd3a04ab 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2017-05-18 H.J. Lu + + * sysdeps/x86_64/multiarch/memcmp.S (__GI_memcmp): Correct + comments. + 2017-05-18 Siddhesh Poyarekar * elf/Makefile (tst-env-setuid-ENV): Reduce value of diff --git a/sysdeps/x86_64/multiarch/memcmp.S b/sysdeps/x86_64/multiarch/memcmp.S index 82623173d6..6129820093 100644 --- a/sysdeps/x86_64/multiarch/memcmp.S +++ b/sysdeps/x86_64/multiarch/memcmp.S @@ -56,9 +56,9 @@ END(memcmp) # ifdef SHARED # undef libc_hidden_builtin_def -/* IFUNC doesn't work with the hidden functions in shared library since - they will be called without setting up EBX needed for PLT which is - used by IFUNC. */ +/* It doesn't make sense to send libc-internal memcmp calls through a PLT. + The speedup we get from using SSE4 instructions is likely eaten away + by the indirect call in the PLT. */ # define libc_hidden_builtin_def(name) \ .globl __GI_memcmp; __GI_memcmp = __memcmp_sse2 # endif -- cgit 1.4.1