diff options
author | Sunil K Pandey <skpgkp2@gmail.com> | 2023-07-26 08:34:05 -0700 |
---|---|---|
committer | Sunil K Pandey <skpgkp2@gmail.com> | 2024-01-13 12:20:08 -0800 |
commit | 9d94997b5f9445afd4f2bccc5fa60ff7c4361ec1 (patch) | |
tree | 4a2b64a9eaac5cac1147c8ae8bb25504b4f7407c /sysdeps/x86_64 | |
parent | fe6c8bab3a8dad3927a12eecd7ec7d67602b5862 (diff) | |
download | glibc-9d94997b5f9445afd4f2bccc5fa60ff7c4361ec1.zip glibc-9d94997b5f9445afd4f2bccc5fa60ff7c4361ec1.tar.gz glibc-9d94997b5f9445afd4f2bccc5fa60ff7c4361ec1.tar.bz2 |
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>
Diffstat (limited to 'sysdeps/x86_64')
-rw-r--r-- | sysdeps/x86_64/ffsll.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/sysdeps/x86_64/ffsll.c b/sysdeps/x86_64/ffsll.c index 73cfc6d..2844428 100644 --- a/sysdeps/x86_64/ffsll.c +++ b/sysdeps/x86_64/ffsll.c @@ -26,13 +26,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__ |