public inbox for glibc-cvs@sourceware.org
help / color / mirror / Atom feed
* [glibc/release/2.34/master] x86_64: Optimize ffsll function code size.
@ 2024-02-01 17:47 Sunil Pandey
  0 siblings, 0 replies; only message in thread
From: Sunil Pandey @ 2024-02-01 17:47 UTC (permalink / raw)
  To: glibc-cvs

https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=a08677d389924c1bccf640f650f5f121abf90cd5

commit a08677d389924c1bccf640f650f5f121abf90cd5
Author: Sunil K Pandey <skpgkp2@gmail.com>
Date:   Wed Jul 26 08:34:05 2023 -0700

    x86_64: Optimize ffsll function code size.
    
    Ffsll function randomly regress by ~20%, depending on how code gets
    aligned in memory.  Ffsll function code size is 17 bytes.  Since default
    function alignment is 16 bytes, it can load on 16, 32, 48 or 64 bytes
    aligned memory.  When ffsll function load at 16, 32 or 64 bytes aligned
    memory, entire code fits in single 64 bytes cache line.  When ffsll
    function load at 48 bytes aligned memory, it splits in two cache line,
    hence random regression.
    
    Ffsll function size reduction from 17 bytes to 12 bytes ensures that it
    will always fit in single 64 bytes cache line.
    
    This patch fixes ffsll function random performance regression.
    
    Reviewed-by: Carlos O'Donell <carlos@redhat.com>
    (cherry picked from commit 9d94997b5f9445afd4f2bccc5fa60ff7c4361ec1)

Diff:
---
 sysdeps/x86_64/ffsll.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/sysdeps/x86_64/ffsll.c b/sysdeps/x86_64/ffsll.c
index 2201a7aacd..17551fcff8 100644
--- a/sysdeps/x86_64/ffsll.c
+++ b/sysdeps/x86_64/ffsll.c
@@ -27,13 +27,13 @@ int
 ffsll (long long int x)
 {
   long long int cnt;
-  long long int tmp;
 
-  asm ("bsfq %2,%0\n"		/* Count low bits in X and store in %1.  */
-       "cmoveq %1,%0\n"		/* If number was zero, use -1 as result.  */
-       : "=&r" (cnt), "=r" (tmp) : "rm" (x), "1" (-1));
+  asm ("mov $-1,%k0\n"	/* Initialize cnt to -1.  */
+       "bsf %1,%0\n"	/* Count low bits in x and store in cnt.  */
+       "inc %k0\n"	/* Increment cnt by 1.  */
+       : "=&r" (cnt) : "r" (x));
 
-  return cnt + 1;
+  return cnt;
 }
 
 #ifndef __ILP32__

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2024-02-01 17:47 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-02-01 17:47 [glibc/release/2.34/master] x86_64: Optimize ffsll function code size Sunil Pandey

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).