* Re: [PATCH v4 22/23] x86: Add EVEX optimized str{n}casecmp [not found] ` <CAFUsyf+FsB7x-O-FB87vsxNF5XaXJ8+CxOr-sgaO+5PbXcH6qQ@mail.gmail.com> @ 2022-05-12 19:47 ` Sunil Pandey 2022-05-12 19:52 ` Sunil Pandey 0 siblings, 1 reply; 3+ messages in thread From: Sunil Pandey @ 2022-05-12 19:47 UTC (permalink / raw) To: Noah Goldstein, Libc-stable Mailing List; +Cc: H.J. Lu, GNU C Library [-- Attachment #1: Type: text/plain, Size: 34190 bytes --] On Fri, Mar 25, 2022 at 11:20 AM Noah Goldstein via Libc-alpha <libc-alpha@sourceware.org> wrote: > > On Fri, Mar 25, 2022 at 1:15 PM H.J. Lu <hjl.tools@gmail.com> wrote: > > > > On Thu, Mar 24, 2022 at 4:56 PM Noah Goldstein <goldstein.w.n@gmail.com> wrote: > > > > > > geometric_mean(N=40) of all benchmarks EVEX / SSE42: .621 > > > > > > All string/memory tests pass. > > > Reviewed-by: H.J. Lu <hjl.tools@gmail.com> > > > --- > > > sysdeps/x86_64/multiarch/Makefile | 2 + > > > sysdeps/x86_64/multiarch/ifunc-impl-list.c | 16 + > > > sysdeps/x86_64/multiarch/ifunc-strcasecmp.h | 5 + > > > sysdeps/x86_64/multiarch/strcasecmp_l-evex.S | 23 ++ > > > sysdeps/x86_64/multiarch/strcmp-evex.S | 290 ++++++++++++++++--- > > > sysdeps/x86_64/multiarch/strncase_l-evex.S | 25 ++ > > > 6 files changed, 321 insertions(+), 40 deletions(-) > > > create mode 100644 sysdeps/x86_64/multiarch/strcasecmp_l-evex.S > > > create mode 100644 sysdeps/x86_64/multiarch/strncase_l-evex.S > > > > > > diff --git a/sysdeps/x86_64/multiarch/Makefile b/sysdeps/x86_64/multiarch/Makefile > > > index 06e1848823..35d80dc2ff 100644 > > > --- a/sysdeps/x86_64/multiarch/Makefile > > > +++ b/sysdeps/x86_64/multiarch/Makefile > > > @@ -57,6 +57,7 @@ sysdep_routines += \ > > > strcasecmp_l-avx \ > > > strcasecmp_l-avx2 \ > > > strcasecmp_l-avx2-rtm \ > > > + strcasecmp_l-evex \ > > > strcasecmp_l-sse2 \ > > > strcasecmp_l-sse4_2 \ > > > strcasecmp_l-ssse3 \ > > > @@ -97,6 +98,7 @@ sysdep_routines += \ > > > strncase_l-avx \ > > > strncase_l-avx2 \ > > > strncase_l-avx2-rtm \ > > > + strncase_l-evex \ > > > strncase_l-sse2 \ > > > strncase_l-sse4_2 \ > > > strncase_l-ssse3 \ > > > diff --git a/sysdeps/x86_64/multiarch/ifunc-impl-list.c b/sysdeps/x86_64/multiarch/ifunc-impl-list.c > > > index 3c556d07ac..f1a4d3dac2 100644 > > > --- a/sysdeps/x86_64/multiarch/ifunc-impl-list.c > > > +++ b/sysdeps/x86_64/multiarch/ifunc-impl-list.c > > > @@ -436,6 +436,10 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, > > > > > > /* Support sysdeps/x86_64/multiarch/strcasecmp_l.c. */ > > > IFUNC_IMPL (i, name, strcasecmp, > > > + IFUNC_IMPL_ADD (array, i, strcasecmp, > > > + (CPU_FEATURE_USABLE (AVX512VL) > > > + && CPU_FEATURE_USABLE (AVX512BW)), > > > + __strcasecmp_evex) > > > IFUNC_IMPL_ADD (array, i, strcasecmp, > > > CPU_FEATURE_USABLE (AVX2), > > > __strcasecmp_avx2) > > > @@ -456,6 +460,10 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, > > > > > > /* Support sysdeps/x86_64/multiarch/strcasecmp_l.c. */ > > > IFUNC_IMPL (i, name, strcasecmp_l, > > > + IFUNC_IMPL_ADD (array, i, strcasecmp, > > > + (CPU_FEATURE_USABLE (AVX512VL) > > > + && CPU_FEATURE_USABLE (AVX512BW)), > > > + __strcasecmp_l_evex) > > > IFUNC_IMPL_ADD (array, i, strcasecmp, > > > CPU_FEATURE_USABLE (AVX2), > > > __strcasecmp_l_avx2) > > > @@ -590,6 +598,10 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, > > > > > > /* Support sysdeps/x86_64/multiarch/strncase_l.c. */ > > > IFUNC_IMPL (i, name, strncasecmp, > > > + IFUNC_IMPL_ADD (array, i, strncasecmp, > > > + (CPU_FEATURE_USABLE (AVX512VL) > > > + && CPU_FEATURE_USABLE (AVX512BW)), > > > + __strncasecmp_evex) > > > IFUNC_IMPL_ADD (array, i, strncasecmp, > > > CPU_FEATURE_USABLE (AVX2), > > > __strncasecmp_avx2) > > > @@ -611,6 +623,10 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, > > > > > > /* Support sysdeps/x86_64/multiarch/strncase_l.c. */ > > > IFUNC_IMPL (i, name, strncasecmp_l, > > > + IFUNC_IMPL_ADD (array, i, strncasecmp, > > > + (CPU_FEATURE_USABLE (AVX512VL) > > > + && CPU_FEATURE_USABLE (AVX512BW)), > > > + __strncasecmp_l_evex) > > > IFUNC_IMPL_ADD (array, i, strncasecmp, > > > CPU_FEATURE_USABLE (AVX2), > > > __strncasecmp_l_avx2) > > > diff --git a/sysdeps/x86_64/multiarch/ifunc-strcasecmp.h b/sysdeps/x86_64/multiarch/ifunc-strcasecmp.h > > > index c4de111fd0..bf0d146e7f 100644 > > > --- a/sysdeps/x86_64/multiarch/ifunc-strcasecmp.h > > > +++ b/sysdeps/x86_64/multiarch/ifunc-strcasecmp.h > > > @@ -25,6 +25,7 @@ extern __typeof (REDIRECT_NAME) OPTIMIZE (sse42) attribute_hidden; > > > extern __typeof (REDIRECT_NAME) OPTIMIZE (avx) attribute_hidden; > > > extern __typeof (REDIRECT_NAME) OPTIMIZE (avx2) attribute_hidden; > > > extern __typeof (REDIRECT_NAME) OPTIMIZE (avx2_rtm) attribute_hidden; > > > +extern __typeof (REDIRECT_NAME) OPTIMIZE (evex) attribute_hidden; > > > > > > static inline void * > > > IFUNC_SELECTOR (void) > > > @@ -34,6 +35,10 @@ IFUNC_SELECTOR (void) > > > if (CPU_FEATURE_USABLE_P (cpu_features, AVX2) > > > && CPU_FEATURES_ARCH_P (cpu_features, AVX_Fast_Unaligned_Load)) > > > { > > > + if (CPU_FEATURE_USABLE_P (cpu_features, AVX512VL) > > > + && CPU_FEATURE_USABLE_P (cpu_features, AVX512BW)) > > > + return OPTIMIZE (evex); > > > + > > > if (CPU_FEATURE_USABLE_P (cpu_features, RTM)) > > > return OPTIMIZE (avx2_rtm); > > > > > > diff --git a/sysdeps/x86_64/multiarch/strcasecmp_l-evex.S b/sysdeps/x86_64/multiarch/strcasecmp_l-evex.S > > > new file mode 100644 > > > index 0000000000..58642db748 > > > --- /dev/null > > > +++ b/sysdeps/x86_64/multiarch/strcasecmp_l-evex.S > > > @@ -0,0 +1,23 @@ > > > +/* strcasecmp_l optimized with EVEX. > > > + Copyright (C) 2017-2022 Free Software Foundation, Inc. > > > + This file is part of the GNU C Library. > > > + > > > + The GNU C Library is free software; you can redistribute it and/or > > > + modify it under the terms of the GNU Lesser General Public > > > + License as published by the Free Software Foundation; either > > > + version 2.1 of the License, or (at your option) any later version. > > > + > > > + The GNU C Library is distributed in the hope that it will be useful, > > > + but WITHOUT ANY WARRANTY; without even the implied warranty of > > > + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > > > + Lesser General Public License for more details. > > > + > > > + You should have received a copy of the GNU Lesser General Public > > > + License along with the GNU C Library; if not, see > > > + <https://www.gnu.org/licenses/>. */ > > > + > > > +#ifndef STRCMP > > > +# define STRCMP __strcasecmp_l_evex > > > +#endif > > > +#define USE_AS_STRCASECMP_L > > > +#include "strcmp-evex.S" > > > diff --git a/sysdeps/x86_64/multiarch/strcmp-evex.S b/sysdeps/x86_64/multiarch/strcmp-evex.S > > > index 56d8c118e4..2a5b3ce037 100644 > > > --- a/sysdeps/x86_64/multiarch/strcmp-evex.S > > > +++ b/sysdeps/x86_64/multiarch/strcmp-evex.S > > > @@ -19,6 +19,9 @@ > > > #if IS_IN (libc) > > > > > > # include <sysdep.h> > > > +# if defined USE_AS_STRCASECMP_L > > > +# include "locale-defines.h" > > > +# endif > > > > > > # ifndef STRCMP > > > # define STRCMP __strcmp_evex > > > @@ -34,19 +37,29 @@ > > > # define VMOVA vmovdqa64 > > > > > > # ifdef USE_AS_WCSCMP > > > -# define TESTEQ subl $0xff, > > > +# ifndef OVERFLOW_STRCMP > > > +# define OVERFLOW_STRCMP __wcscmp_evex > > > +# endif > > > + > > > +# define TESTEQ subl $0xff, > > > /* Compare packed dwords. */ > > > # define VPCMP vpcmpd > > > # define VPMINU vpminud > > > # define VPTESTM vptestmd > > > +# define VPTESTNM vptestnmd > > > /* 1 dword char == 4 bytes. */ > > > # define SIZE_OF_CHAR 4 > > > # else > > > +# ifndef OVERFLOW_STRCMP > > > +# define OVERFLOW_STRCMP __strcmp_evex > > > +# endif > > > + > > > # define TESTEQ incl > > > /* Compare packed bytes. */ > > > # define VPCMP vpcmpb > > > # define VPMINU vpminub > > > # define VPTESTM vptestmb > > > +# define VPTESTNM vptestnmb > > > /* 1 byte char == 1 byte. */ > > > # define SIZE_OF_CHAR 1 > > > # endif > > > @@ -73,11 +86,16 @@ > > > # define VEC_OFFSET (-VEC_SIZE) > > > # endif > > > > > > -# define XMMZERO xmm16 > > > # define XMM0 xmm17 > > > # define XMM1 xmm18 > > > > > > -# define YMMZERO ymm16 > > > +# define XMM10 xmm27 > > > +# define XMM11 xmm28 > > > +# define XMM12 xmm29 > > > +# define XMM13 xmm30 > > > +# define XMM14 xmm31 > > > + > > > + > > > # define YMM0 ymm17 > > > # define YMM1 ymm18 > > > # define YMM2 ymm19 > > > @@ -89,6 +107,87 @@ > > > # define YMM8 ymm25 > > > # define YMM9 ymm26 > > > # define YMM10 ymm27 > > > +# define YMM11 ymm28 > > > +# define YMM12 ymm29 > > > +# define YMM13 ymm30 > > > +# define YMM14 ymm31 > > > + > > > +# ifdef USE_AS_STRCASECMP_L > > > +# define BYTE_LOOP_REG OFFSET_REG > > > +# else > > > +# define BYTE_LOOP_REG ecx > > > +# endif > > > + > > > +# ifdef USE_AS_STRCASECMP_L > > > +# ifdef USE_AS_STRNCMP > > > +# define STRCASECMP __strncasecmp_evex > > > +# define LOCALE_REG rcx > > > +# define LOCALE_REG_LP RCX_LP > > > +# define STRCASECMP_NONASCII __strncasecmp_l_nonascii > > > +# else > > > +# define STRCASECMP __strcasecmp_evex > > > +# define LOCALE_REG rdx > > > +# define LOCALE_REG_LP RDX_LP > > > +# define STRCASECMP_NONASCII __strcasecmp_l_nonascii > > > +# endif > > > +# endif > > > + > > > +# define LCASE_MIN_YMM %YMM12 > > > +# define LCASE_MAX_YMM %YMM13 > > > +# define CASE_ADD_YMM %YMM14 > > > + > > > +# define LCASE_MIN_XMM %XMM12 > > > +# define LCASE_MAX_XMM %XMM13 > > > +# define CASE_ADD_XMM %XMM14 > > > + > > > + /* NB: wcsncmp uses r11 but strcasecmp is never used in > > > + conjunction with wcscmp. */ > > > +# define TOLOWER_BASE %r11 > > > + > > > +# ifdef USE_AS_STRCASECMP_L > > > +# define _REG(x, y) x ## y > > > +# define REG(x, y) _REG(x, y) > > > +# define TOLOWER(reg1, reg2, ext) \ > > > + vpsubb REG(LCASE_MIN_, ext), reg1, REG(%ext, 10); \ > > > + vpsubb REG(LCASE_MIN_, ext), reg2, REG(%ext, 11); \ > > > + vpcmpub $1, REG(LCASE_MAX_, ext), REG(%ext, 10), %k5; \ > > > + vpcmpub $1, REG(LCASE_MAX_, ext), REG(%ext, 11), %k6; \ > > > + vpaddb reg1, REG(CASE_ADD_, ext), reg1{%k5}; \ > > > + vpaddb reg2, REG(CASE_ADD_, ext), reg2{%k6} > > > + > > > +# define TOLOWER_gpr(src, dst) movl (TOLOWER_BASE, src, 4), dst > > > +# define TOLOWER_YMM(...) TOLOWER(__VA_ARGS__, YMM) > > > +# define TOLOWER_XMM(...) TOLOWER(__VA_ARGS__, XMM) > > > + > > > +# define CMP_R1_R2(s1_reg, s2_reg, reg_out, ext) \ > > > + TOLOWER (s1_reg, s2_reg, ext); \ > > > + VPCMP $0, s1_reg, s2_reg, reg_out > > > + > > > +# define CMP_R1_S2(s1_reg, s2_mem, s2_reg, reg_out, ext) \ > > > + VMOVU s2_mem, s2_reg; \ > > > + CMP_R1_R2(s1_reg, s2_reg, reg_out, ext) > > > + > > > +# define CMP_R1_R2_YMM(...) CMP_R1_R2(__VA_ARGS__, YMM) > > > +# define CMP_R1_R2_XMM(...) CMP_R1_R2(__VA_ARGS__, XMM) > > > + > > > +# define CMP_R1_S2_YMM(...) CMP_R1_S2(__VA_ARGS__, YMM) > > > +# define CMP_R1_S2_XMM(...) CMP_R1_S2(__VA_ARGS__, XMM) > > > + > > > +# else > > > +# define TOLOWER_gpr(...) > > > +# define TOLOWER_YMM(...) > > > +# define TOLOWER_XMM(...) > > > + > > > +# define CMP_R1_R2_YMM(s1_reg, s2_reg, reg_out) \ > > > + VPCMP $0, s2_reg, s1_reg, reg_out > > > + > > > +# define CMP_R1_R2_XMM(...) CMP_R1_R2_YMM(__VA_ARGS__) > > > + > > > +# define CMP_R1_S2_YMM(s1_reg, s2_mem, unused, reg_out) \ > > > + VPCMP $0, s2_mem, s1_reg, reg_out > > > + > > > +# define CMP_R1_S2_XMM(...) CMP_R1_S2_YMM(__VA_ARGS__) > > > +# endif > > > > > > /* Warning! > > > wcscmp/wcsncmp have to use SIGNED comparison for elements. > > > @@ -112,8 +211,45 @@ > > > returned. */ > > > > > > .section .text.evex, "ax", @progbits > > > -ENTRY(STRCMP) > > > + .align 16 > > > + .type STRCMP, @function > > > + .globl STRCMP > > > + .hidden STRCMP > > > + > > > +# ifdef USE_AS_STRCASECMP_L > > > +ENTRY (STRCASECMP) > > > + movq __libc_tsd_LOCALE@gottpoff(%rip), %rax > > > + mov %fs:(%rax), %LOCALE_REG_LP > > > + > > > + /* Either 1 or 5 bytes (dependeing if CET is enabled). */ > > > + .p2align 4 > > > +END (STRCASECMP) > > > + /* FALLTHROUGH to strcasecmp/strncasecmp_l. */ > > > +# endif > > > + > > > + .p2align 4 > > > +STRCMP: > > > + cfi_startproc > > > + _CET_ENDBR > > > + CALL_MCOUNT > > > + > > > +# if defined USE_AS_STRCASECMP_L > > > + /* We have to fall back on the C implementation for locales with > > > + encodings not matching ASCII for single bytes. */ > > > +# if LOCALE_T___LOCALES != 0 || LC_CTYPE != 0 > > > + mov LOCALE_T___LOCALES + LC_CTYPE * LP_SIZE(%LOCALE_REG), %RAX_LP > > > +# else > > > + mov (%LOCALE_REG), %RAX_LP > > > +# endif > > > + testl $1, LOCALE_DATA_VALUES + _NL_CTYPE_NONASCII_CASE * SIZEOF_VALUES(%rax) > > > + jne STRCASECMP_NONASCII > > > + leaq _nl_C_LC_CTYPE_tolower + 128 * 4(%rip), TOLOWER_BASE > > > +# endif > > > + > > > # ifdef USE_AS_STRNCMP > > > + /* Don't overwrite LOCALE_REG (rcx) until we have pass > > > + L(one_or_less). Otherwise we might use the wrong locale in > > > + the OVERFLOW_STRCMP (strcasecmp_l). */ > > > # ifdef __ILP32__ > > > /* Clear the upper 32 bits. */ > > > movl %edx, %edx > > > @@ -125,6 +261,32 @@ ENTRY(STRCMP) > > > actually bound the buffer. */ > > > jle L(one_or_less) > > > # endif > > > + > > > +# if defined USE_AS_STRCASECMP_L > > > + .section .rodata.cst32, "aM", @progbits, 32 > > > + .align 32 > > > +L(lcase_min): > > > + .quad 0x4141414141414141 > > > + .quad 0x4141414141414141 > > > + .quad 0x4141414141414141 > > > + .quad 0x4141414141414141 > > > +L(lcase_max): > > > + .quad 0x1a1a1a1a1a1a1a1a > > > + .quad 0x1a1a1a1a1a1a1a1a > > > + .quad 0x1a1a1a1a1a1a1a1a > > > + .quad 0x1a1a1a1a1a1a1a1a > > > +L(case_add): > > > + .quad 0x2020202020202020 > > > + .quad 0x2020202020202020 > > > + .quad 0x2020202020202020 > > > + .quad 0x2020202020202020 > > > + .previous > > > + > > > + vmovdqa64 L(lcase_min)(%rip), LCASE_MIN_YMM > > > + vmovdqa64 L(lcase_max)(%rip), LCASE_MAX_YMM > > > + vmovdqa64 L(case_add)(%rip), CASE_ADD_YMM > > > +# endif > > > + > > > movl %edi, %eax > > > orl %esi, %eax > > > /* Shift out the bits irrelivant to page boundary ([63:12]). */ > > > @@ -139,7 +301,7 @@ L(no_page_cross): > > > VPTESTM %YMM0, %YMM0, %k2 > > > /* Each bit cleared in K1 represents a mismatch or a null CHAR > > > in YMM0 and 32 bytes at (%rsi). */ > > > - VPCMP $0, (%rsi), %YMM0, %k1{%k2} > > > + CMP_R1_S2_YMM (%YMM0, (%rsi), %YMM1, %k1){%k2} > > > kmovd %k1, %ecx > > > # ifdef USE_AS_STRNCMP > > > cmpq $CHAR_PER_VEC, %rdx > > > @@ -169,6 +331,8 @@ L(return_vec_0): > > > # else > > > movzbl (%rdi, %rcx), %eax > > > movzbl (%rsi, %rcx), %ecx > > > + TOLOWER_gpr (%rax, %eax) > > > + TOLOWER_gpr (%rcx, %ecx) > > > subl %ecx, %eax > > > # endif > > > L(ret0): > > > @@ -188,11 +352,15 @@ L(ret_zero): > > > > > > .p2align 4,, 5 > > > L(one_or_less): > > > +# ifdef USE_AS_STRCASECMP_L > > > + /* Set locale argument for strcasecmp. */ > > > + movq %LOCALE_REG, %rdx > > > +# endif > > > jb L(ret_zero) > > > -# ifdef USE_AS_WCSCMP > > > /* 'nbe' covers the case where length is negative (large > > > unsigned). */ > > > - jnbe __wcscmp_evex > > > + jnbe OVERFLOW_STRCMP > > > +# ifdef USE_AS_WCSCMP > > > movl (%rdi), %edx > > > xorl %eax, %eax > > > cmpl (%rsi), %edx > > > @@ -201,11 +369,10 @@ L(one_or_less): > > > negl %eax > > > orl $1, %eax > > > # else > > > - /* 'nbe' covers the case where length is negative (large > > > - unsigned). */ > > > - jnbe __strcmp_evex > > > movzbl (%rdi), %eax > > > movzbl (%rsi), %ecx > > > + TOLOWER_gpr (%rax, %eax) > > > + TOLOWER_gpr (%rcx, %ecx) > > > subl %ecx, %eax > > > # endif > > > L(ret1): > > > @@ -233,6 +400,8 @@ L(return_vec_1): > > > # else > > > movzbl VEC_SIZE(%rdi, %rcx), %eax > > > movzbl VEC_SIZE(%rsi, %rcx), %ecx > > > + TOLOWER_gpr (%rax, %eax) > > > + TOLOWER_gpr (%rcx, %ecx) > > > subl %ecx, %eax > > > # endif > > > L(ret2): > > > @@ -270,6 +439,8 @@ L(return_vec_2): > > > # else > > > movzbl (VEC_SIZE * 2)(%rdi, %rcx), %eax > > > movzbl (VEC_SIZE * 2)(%rsi, %rcx), %ecx > > > + TOLOWER_gpr (%rax, %eax) > > > + TOLOWER_gpr (%rcx, %ecx) > > > subl %ecx, %eax > > > # endif > > > L(ret3): > > > @@ -290,6 +461,8 @@ L(return_vec_3): > > > # else > > > movzbl (VEC_SIZE * 3)(%rdi, %rcx), %eax > > > movzbl (VEC_SIZE * 3)(%rsi, %rcx), %ecx > > > + TOLOWER_gpr (%rax, %eax) > > > + TOLOWER_gpr (%rcx, %ecx) > > > subl %ecx, %eax > > > # endif > > > L(ret4): > > > @@ -303,7 +476,7 @@ L(more_3x_vec): > > > /* Safe to compare 4x vectors. */ > > > VMOVU (VEC_SIZE)(%rdi), %YMM0 > > > VPTESTM %YMM0, %YMM0, %k2 > > > - VPCMP $0, (VEC_SIZE)(%rsi), %YMM0, %k1{%k2} > > > + CMP_R1_S2_YMM (%YMM0, VEC_SIZE(%rsi), %YMM1, %k1){%k2} > > > kmovd %k1, %ecx > > > TESTEQ %ecx > > > jnz L(return_vec_1) > > > @@ -315,14 +488,14 @@ L(more_3x_vec): > > > > > > VMOVU (VEC_SIZE * 2)(%rdi), %YMM0 > > > VPTESTM %YMM0, %YMM0, %k2 > > > - VPCMP $0, (VEC_SIZE * 2)(%rsi), %YMM0, %k1{%k2} > > > + CMP_R1_S2_YMM (%YMM0, (VEC_SIZE * 2)(%rsi), %YMM1, %k1){%k2} > > > kmovd %k1, %ecx > > > TESTEQ %ecx > > > jnz L(return_vec_2) > > > > > > VMOVU (VEC_SIZE * 3)(%rdi), %YMM0 > > > VPTESTM %YMM0, %YMM0, %k2 > > > - VPCMP $0, (VEC_SIZE * 3)(%rsi), %YMM0, %k1{%k2} > > > + CMP_R1_S2_YMM (%YMM0, (VEC_SIZE * 3)(%rsi), %YMM1, %k1){%k2} > > > kmovd %k1, %ecx > > > TESTEQ %ecx > > > jnz L(return_vec_3) > > > @@ -381,7 +554,6 @@ L(prepare_loop_aligned): > > > subl %esi, %eax > > > andl $(PAGE_SIZE - 1), %eax > > > > > > - vpxorq %YMMZERO, %YMMZERO, %YMMZERO > > > > > > /* Loop 4x comparisons at a time. */ > > > .p2align 4 > > > @@ -413,22 +585,35 @@ L(loop_skip_page_cross_check): > > > /* A zero CHAR in YMM9 means that there is a null CHAR. */ > > > VPMINU %YMM8, %YMM9, %YMM9 > > > > > > - /* Each bit set in K1 represents a non-null CHAR in YMM8. */ > > > + /* Each bit set in K1 represents a non-null CHAR in YMM9. */ > > > VPTESTM %YMM9, %YMM9, %k1 > > > - > > > +# ifndef USE_AS_STRCASECMP_L > > > vpxorq (VEC_SIZE * 0)(%rsi), %YMM0, %YMM1 > > > vpxorq (VEC_SIZE * 1)(%rsi), %YMM2, %YMM3 > > > vpxorq (VEC_SIZE * 2)(%rsi), %YMM4, %YMM5 > > > /* Ternary logic to xor (VEC_SIZE * 3)(%rsi) with YMM6 while > > > oring with YMM1. Result is stored in YMM6. */ > > > vpternlogd $0xde, (VEC_SIZE * 3)(%rsi), %YMM1, %YMM6 > > > - > > > +# else > > > + VMOVU (VEC_SIZE * 0)(%rsi), %YMM1 > > > + TOLOWER_YMM (%YMM0, %YMM1) > > > + VMOVU (VEC_SIZE * 1)(%rsi), %YMM3 > > > + TOLOWER_YMM (%YMM2, %YMM3) > > > + VMOVU (VEC_SIZE * 2)(%rsi), %YMM5 > > > + TOLOWER_YMM (%YMM4, %YMM5) > > > + VMOVU (VEC_SIZE * 3)(%rsi), %YMM7 > > > + TOLOWER_YMM (%YMM6, %YMM7) > > > + vpxorq %YMM0, %YMM1, %YMM1 > > > + vpxorq %YMM2, %YMM3, %YMM3 > > > + vpxorq %YMM4, %YMM5, %YMM5 > > > + vpternlogd $0xde, %YMM7, %YMM1, %YMM6 > > > +# endif > > > /* Or together YMM3, YMM5, and YMM6. */ > > > vpternlogd $0xfe, %YMM3, %YMM5, %YMM6 > > > > > > > > > /* A non-zero CHAR in YMM6 represents a mismatch. */ > > > - VPCMP $0, %YMMZERO, %YMM6, %k0{%k1} > > > + VPTESTNM %YMM6, %YMM6, %k0{%k1} > > > kmovd %k0, %LOOP_REG > > > > > > TESTEQ %LOOP_REG > > > @@ -437,13 +622,13 @@ L(loop_skip_page_cross_check): > > > > > > /* Find which VEC has the mismatch of end of string. */ > > > VPTESTM %YMM0, %YMM0, %k1 > > > - VPCMP $0, %YMMZERO, %YMM1, %k0{%k1} > > > + VPTESTNM %YMM1, %YMM1, %k0{%k1} > > > kmovd %k0, %ecx > > > TESTEQ %ecx > > > jnz L(return_vec_0_end) > > > > > > VPTESTM %YMM2, %YMM2, %k1 > > > - VPCMP $0, %YMMZERO, %YMM3, %k0{%k1} > > > + VPTESTNM %YMM3, %YMM3, %k0{%k1} > > > kmovd %k0, %ecx > > > TESTEQ %ecx > > > jnz L(return_vec_1_end) > > > @@ -457,7 +642,7 @@ L(return_vec_2_3_end): > > > # endif > > > > > > VPTESTM %YMM4, %YMM4, %k1 > > > - VPCMP $0, %YMMZERO, %YMM5, %k0{%k1} > > > + VPTESTNM %YMM5, %YMM5, %k0{%k1} > > > kmovd %k0, %ecx > > > TESTEQ %ecx > > > # if CHAR_PER_VEC <= 16 > > > @@ -493,6 +678,8 @@ L(return_vec_3_end): > > > # else > > > movzbl (VEC_SIZE * 2)(%rdi, %LOOP_REG64), %eax > > > movzbl (VEC_SIZE * 2)(%rsi, %LOOP_REG64), %ecx > > > + TOLOWER_gpr (%rax, %eax) > > > + TOLOWER_gpr (%rcx, %ecx) > > > subl %ecx, %eax > > > xorl %r8d, %eax > > > subl %r8d, %eax > > > @@ -545,6 +732,8 @@ L(return_vec_0_end): > > > # else > > > movzbl (%rdi, %rcx), %eax > > > movzbl (%rsi, %rcx), %ecx > > > + TOLOWER_gpr (%rax, %eax) > > > + TOLOWER_gpr (%rcx, %ecx) > > > subl %ecx, %eax > > > /* Flip `eax` if `rdi` and `rsi` where swapped in page cross > > > logic. Subtract `r8d` after xor for zero case. */ > > > @@ -569,6 +758,8 @@ L(return_vec_1_end): > > > # else > > > movzbl VEC_SIZE(%rdi, %rcx), %eax > > > movzbl VEC_SIZE(%rsi, %rcx), %ecx > > > + TOLOWER_gpr (%rax, %eax) > > > + TOLOWER_gpr (%rcx, %ecx) > > > subl %ecx, %eax > > > xorl %r8d, %eax > > > subl %r8d, %eax > > > @@ -598,7 +789,7 @@ L(page_cross_during_loop): > > > > > > VMOVA (%rdi), %YMM0 > > > VPTESTM %YMM0, %YMM0, %k2 > > > - VPCMP $0, (%rsi), %YMM0, %k1{%k2} > > > + CMP_R1_S2_YMM (%YMM0, (%rsi), %YMM1, %k1){%k2} > > > kmovd %k1, %ecx > > > TESTEQ %ecx > > > jnz L(return_vec_0_end) > > > @@ -619,8 +810,7 @@ L(less_1x_vec_till_page_cross): > > > been loaded earlier so must be valid. */ > > > VMOVU -VEC_SIZE(%rdi, %rax), %YMM0 > > > VPTESTM %YMM0, %YMM0, %k2 > > > - VPCMP $0, -VEC_SIZE(%rsi, %rax), %YMM0, %k1{%k2} > > > - > > > + CMP_R1_S2_YMM (%YMM0, -VEC_SIZE(%rsi, %rax), %YMM1, %k1){%k2} > > > /* Mask of potentially valid bits. The lower bits can be out of > > > range comparisons (but safe regarding page crosses). */ > > > > > > @@ -642,6 +832,8 @@ L(less_1x_vec_till_page_cross): > > > > > > # ifdef USE_AS_STRNCMP > > > # ifdef USE_AS_WCSCMP > > > + /* NB: strcasecmp not used with WCSCMP so this access to r11 is > > > + safe. */ > > > movl %eax, %r11d > > > shrl $2, %r11d > > > cmpq %r11, %rdx > > > @@ -679,6 +871,8 @@ L(return_page_cross_cmp_mem): > > > # else > > > movzbl VEC_OFFSET(%rdi, %rcx), %eax > > > movzbl VEC_OFFSET(%rsi, %rcx), %ecx > > > + TOLOWER_gpr (%rax, %eax) > > > + TOLOWER_gpr (%rcx, %ecx) > > > subl %ecx, %eax > > > xorl %r8d, %eax > > > subl %r8d, %eax > > > @@ -709,7 +903,7 @@ L(more_2x_vec_till_page_cross): > > > > > > VMOVA VEC_SIZE(%rdi), %YMM0 > > > VPTESTM %YMM0, %YMM0, %k2 > > > - VPCMP $0, VEC_SIZE(%rsi), %YMM0, %k1{%k2} > > > + CMP_R1_S2_YMM (%YMM0, VEC_SIZE(%rsi), %YMM1, %k1){%k2} > > > kmovd %k1, %ecx > > > TESTEQ %ecx > > > jnz L(return_vec_1_end) > > > @@ -724,14 +918,14 @@ L(more_2x_vec_till_page_cross): > > > /* Safe to include comparisons from lower bytes. */ > > > VMOVU -(VEC_SIZE * 2)(%rdi, %rax), %YMM0 > > > VPTESTM %YMM0, %YMM0, %k2 > > > - VPCMP $0, -(VEC_SIZE * 2)(%rsi, %rax), %YMM0, %k1{%k2} > > > + CMP_R1_S2_YMM (%YMM0, -(VEC_SIZE * 2)(%rsi, %rax), %YMM1, %k1){%k2} > > > kmovd %k1, %ecx > > > TESTEQ %ecx > > > jnz L(return_vec_page_cross_0) > > > > > > VMOVU -(VEC_SIZE * 1)(%rdi, %rax), %YMM0 > > > VPTESTM %YMM0, %YMM0, %k2 > > > - VPCMP $0, -(VEC_SIZE * 1)(%rsi, %rax), %YMM0, %k1{%k2} > > > + CMP_R1_S2_YMM (%YMM0, -(VEC_SIZE * 1)(%rsi, %rax), %YMM1, %k1){%k2} > > > kmovd %k1, %ecx > > > TESTEQ %ecx > > > jnz L(return_vec_page_cross_1) > > > @@ -740,6 +934,8 @@ L(more_2x_vec_till_page_cross): > > > /* Must check length here as length might proclude reading next > > > page. */ > > > # ifdef USE_AS_WCSCMP > > > + /* NB: strcasecmp not used with WCSCMP so this access to r11 is > > > + safe. */ > > > movl %eax, %r11d > > > shrl $2, %r11d > > > cmpq %r11, %rdx > > > @@ -754,12 +950,19 @@ L(more_2x_vec_till_page_cross): > > > VMOVA (VEC_SIZE * 3)(%rdi), %YMM6 > > > VPMINU %YMM4, %YMM6, %YMM9 > > > VPTESTM %YMM9, %YMM9, %k1 > > > - > > > +# ifndef USE_AS_STRCASECMP_L > > > vpxorq (VEC_SIZE * 2)(%rsi), %YMM4, %YMM5 > > > /* YMM6 = YMM5 | ((VEC_SIZE * 3)(%rsi) ^ YMM6). */ > > > vpternlogd $0xde, (VEC_SIZE * 3)(%rsi), %YMM5, %YMM6 > > > - > > > - VPCMP $0, %YMMZERO, %YMM6, %k0{%k1} > > > +# else > > > + VMOVU (VEC_SIZE * 2)(%rsi), %YMM5 > > > + TOLOWER_YMM (%YMM4, %YMM5) > > > + VMOVU (VEC_SIZE * 3)(%rsi), %YMM7 > > > + TOLOWER_YMM (%YMM6, %YMM7) > > > + vpxorq %YMM4, %YMM5, %YMM5 > > > + vpternlogd $0xde, %YMM7, %YMM5, %YMM6 > > > +# endif > > > + VPTESTNM %YMM6, %YMM6, %k0{%k1} > > > kmovd %k0, %LOOP_REG > > > TESTEQ %LOOP_REG > > > jnz L(return_vec_2_3_end) > > > @@ -815,6 +1018,8 @@ L(return_vec_page_cross_1): > > > # else > > > movzbl VEC_OFFSET(%rdi, %rcx), %eax > > > movzbl VEC_OFFSET(%rsi, %rcx), %ecx > > > + TOLOWER_gpr (%rax, %eax) > > > + TOLOWER_gpr (%rcx, %ecx) > > > subl %ecx, %eax > > > xorl %r8d, %eax > > > subl %r8d, %eax > > > @@ -871,7 +1076,7 @@ L(page_cross): > > > L(page_cross_loop): > > > VMOVU (%rdi, %OFFSET_REG64, SIZE_OF_CHAR), %YMM0 > > > VPTESTM %YMM0, %YMM0, %k2 > > > - VPCMP $0, (%rsi, %OFFSET_REG64, SIZE_OF_CHAR), %YMM0, %k1{%k2} > > > + CMP_R1_S2_YMM (%YMM0, (%rsi, %OFFSET_REG64, SIZE_OF_CHAR), %YMM1, %k1){%k2} > > > kmovd %k1, %ecx > > > TESTEQ %ecx > > > jnz L(check_ret_vec_page_cross) > > > @@ -895,7 +1100,7 @@ L(page_cross_loop): > > > */ > > > VMOVU (%rdi, %OFFSET_REG64, SIZE_OF_CHAR), %YMM0 > > > VPTESTM %YMM0, %YMM0, %k2 > > > - VPCMP $0, (%rsi, %OFFSET_REG64, SIZE_OF_CHAR), %YMM0, %k1{%k2} > > > + CMP_R1_S2_YMM (%YMM0, (%rsi, %OFFSET_REG64, SIZE_OF_CHAR), %YMM1, %k1){%k2} > > > > > > kmovd %k1, %ecx > > > # ifdef USE_AS_STRNCMP > > > @@ -930,6 +1135,8 @@ L(ret_vec_page_cross_cont): > > > # else > > > movzbl (%rdi, %rcx, SIZE_OF_CHAR), %eax > > > movzbl (%rsi, %rcx, SIZE_OF_CHAR), %ecx > > > + TOLOWER_gpr (%rax, %eax) > > > + TOLOWER_gpr (%rcx, %ecx) > > > subl %ecx, %eax > > > xorl %r8d, %eax > > > subl %r8d, %eax > > > @@ -989,7 +1196,7 @@ L(less_1x_vec_till_page): > > > /* Use 16 byte comparison. */ > > > vmovdqu (%rdi), %xmm0 > > > VPTESTM %xmm0, %xmm0, %k2 > > > - VPCMP $0, (%rsi), %xmm0, %k1{%k2} > > > + CMP_R1_S2_XMM (%xmm0, (%rsi), %xmm1, %k1){%k2} > > > kmovd %k1, %ecx > > > # ifdef USE_AS_WCSCMP > > > subl $0xf, %ecx > > > @@ -1009,7 +1216,7 @@ L(less_1x_vec_till_page): > > > # endif > > > vmovdqu (%rdi, %OFFSET_REG64, SIZE_OF_CHAR), %xmm0 > > > VPTESTM %xmm0, %xmm0, %k2 > > > - VPCMP $0, (%rsi, %OFFSET_REG64, SIZE_OF_CHAR), %xmm0, %k1{%k2} > > > + CMP_R1_S2_XMM (%xmm0, (%rsi, %OFFSET_REG64, SIZE_OF_CHAR), %xmm1, %k1){%k2} > > > kmovd %k1, %ecx > > > # ifdef USE_AS_WCSCMP > > > subl $0xf, %ecx > > > @@ -1048,7 +1255,7 @@ L(less_16_till_page): > > > vmovq (%rdi), %xmm0 > > > vmovq (%rsi), %xmm1 > > > VPTESTM %xmm0, %xmm0, %k2 > > > - VPCMP $0, %xmm1, %xmm0, %k1{%k2} > > > + CMP_R1_R2_XMM (%xmm0, %xmm1, %k1){%k2} > > > kmovd %k1, %ecx > > > # ifdef USE_AS_WCSCMP > > > subl $0x3, %ecx > > > @@ -1068,7 +1275,7 @@ L(less_16_till_page): > > > vmovq (%rdi, %OFFSET_REG64, SIZE_OF_CHAR), %xmm0 > > > vmovq (%rsi, %OFFSET_REG64, SIZE_OF_CHAR), %xmm1 > > > VPTESTM %xmm0, %xmm0, %k2 > > > - VPCMP $0, %xmm1, %xmm0, %k1{%k2} > > > + CMP_R1_R2_XMM (%xmm0, %xmm1, %k1){%k2} > > > kmovd %k1, %ecx > > > # ifdef USE_AS_WCSCMP > > > subl $0x3, %ecx > > > @@ -1128,7 +1335,7 @@ L(ret_less_8_wcs): > > > vmovd (%rdi), %xmm0 > > > vmovd (%rsi), %xmm1 > > > VPTESTM %xmm0, %xmm0, %k2 > > > - VPCMP $0, %xmm1, %xmm0, %k1{%k2} > > > + CMP_R1_R2_XMM (%xmm0, %xmm1, %k1){%k2} > > > kmovd %k1, %ecx > > > subl $0xf, %ecx > > > jnz L(check_ret_vec_page_cross) > > > @@ -1143,7 +1350,7 @@ L(ret_less_8_wcs): > > > vmovd (%rdi, %OFFSET_REG64, SIZE_OF_CHAR), %xmm0 > > > vmovd (%rsi, %OFFSET_REG64, SIZE_OF_CHAR), %xmm1 > > > VPTESTM %xmm0, %xmm0, %k2 > > > - VPCMP $0, %xmm1, %xmm0, %k1{%k2} > > > + CMP_R1_R2_XMM (%xmm0, %xmm1, %k1){%k2} > > > kmovd %k1, %ecx > > > subl $0xf, %ecx > > > jnz L(check_ret_vec_page_cross) > > > @@ -1176,7 +1383,9 @@ L(less_4_till_page): > > > L(less_4_loop): > > > movzbl (%rdi), %eax > > > movzbl (%rsi, %rdi), %ecx > > > - subl %ecx, %eax > > > + TOLOWER_gpr (%rax, %eax) > > > + TOLOWER_gpr (%rcx, %BYTE_LOOP_REG) > > > + subl %BYTE_LOOP_REG, %eax > > > jnz L(ret_less_4_loop) > > > testl %ecx, %ecx > > > jz L(ret_zero_4_loop) > > > @@ -1203,5 +1412,6 @@ L(ret_less_4_loop): > > > subl %r8d, %eax > > > ret > > > # endif > > > -END(STRCMP) > > > + cfi_endproc > > > + .size STRCMP, .-STRCMP > > > #endif > > > diff --git a/sysdeps/x86_64/multiarch/strncase_l-evex.S b/sysdeps/x86_64/multiarch/strncase_l-evex.S > > > new file mode 100644 > > > index 0000000000..8a5af3695c > > > --- /dev/null > > > +++ b/sysdeps/x86_64/multiarch/strncase_l-evex.S > > > @@ -0,0 +1,25 @@ > > > +/* strncasecmp_l optimized with EVEX. > > > + Copyright (C) 2017-2022 Free Software Foundation, Inc. > > > + This file is part of the GNU C Library. > > > + > > > + The GNU C Library is free software; you can redistribute it and/or > > > + modify it under the terms of the GNU Lesser General Public > > > + License as published by the Free Software Foundation; either > > > + version 2.1 of the License, or (at your option) any later version. > > > + > > > + The GNU C Library is distributed in the hope that it will be useful, > > > + but WITHOUT ANY WARRANTY; without even the implied warranty of > > > + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > > > + Lesser General Public License for more details. > > > + > > > + You should have received a copy of the GNU Lesser General Public > > > + License along with the GNU C Library; if not, see > > > + <https://www.gnu.org/licenses/>. */ > > > + > > > +#ifndef STRCMP > > > +# define STRCMP __strncasecmp_l_evex > > > +#endif > > > +#define OVERFLOW_STRCMP __strcasecmp_l_evex > > > +#define USE_AS_STRCASECMP_L > > > +#define USE_AS_STRNCMP > > > +#include "strcmp-evex.S" > > > -- > > > 2.25.1 > > > > > > > LGTM. > > > > Reviewed-by: H.J. Lu <hjl.tools@gmail.com> > > Thanks, pushed the patchset. > > > > Thanks. > > > > -- > > H.J. I would like to backport this patch to release branches. Any comments or objections? Conflict resolution patch attached. --Sunil [-- Attachment #2: 0015-x86-Add-AVX2-optimized-str-n-casecmp.patch --] [-- Type: application/octet-stream, Size: 24673 bytes --] From b382e4caf50dfee62e170f9b6617b470b1289dcb Mon Sep 17 00:00:00 2001 From: Noah Goldstein <goldstein.w.n@gmail.com> Date: Thu, 24 Mar 2022 18:56:12 -0500 Subject: [PATCH 15/26] x86: Add AVX2 optimized str{n}casecmp geometric_mean(N=40) of all benchmarks AVX2 / SSE42: .702 All string/memory tests pass. Reviewed-by: H.J. Lu <hjl.tools@gmail.com> (cherry picked from commit bbf81222343fed5cd704001a2ae0d86c71544151) --- sysdeps/x86_64/multiarch/Makefile | 4 + sysdeps/x86_64/multiarch/ifunc-impl-list.c | 28 +++ sysdeps/x86_64/multiarch/ifunc-strcasecmp.h | 12 + .../x86_64/multiarch/strcasecmp_l-avx2-rtm.S | 15 ++ sysdeps/x86_64/multiarch/strcasecmp_l-avx2.S | 23 ++ sysdeps/x86_64/multiarch/strcmp-avx2.S | 237 +++++++++++++++--- .../x86_64/multiarch/strncase_l-avx2-rtm.S | 16 ++ sysdeps/x86_64/multiarch/strncase_l-avx2.S | 27 ++ 8 files changed, 331 insertions(+), 31 deletions(-) create mode 100644 sysdeps/x86_64/multiarch/strcasecmp_l-avx2-rtm.S create mode 100644 sysdeps/x86_64/multiarch/strcasecmp_l-avx2.S create mode 100644 sysdeps/x86_64/multiarch/strncase_l-avx2-rtm.S create mode 100644 sysdeps/x86_64/multiarch/strncase_l-avx2.S diff --git a/sysdeps/x86_64/multiarch/Makefile b/sysdeps/x86_64/multiarch/Makefile index e7b413edad..06e1848823 100644 --- a/sysdeps/x86_64/multiarch/Makefile +++ b/sysdeps/x86_64/multiarch/Makefile @@ -55,6 +55,8 @@ sysdep_routines += \ stpncpy-sse2-unaligned \ stpncpy-ssse3 \ strcasecmp_l-avx \ + strcasecmp_l-avx2 \ + strcasecmp_l-avx2-rtm \ strcasecmp_l-sse2 \ strcasecmp_l-sse4_2 \ strcasecmp_l-ssse3 \ @@ -93,6 +95,8 @@ sysdep_routines += \ strlen-evex \ strlen-sse2 \ strncase_l-avx \ + strncase_l-avx2 \ + strncase_l-avx2-rtm \ strncase_l-sse2 \ strncase_l-sse4_2 \ strncase_l-ssse3 \ diff --git a/sysdeps/x86_64/multiarch/ifunc-impl-list.c b/sysdeps/x86_64/multiarch/ifunc-impl-list.c index a594f4176e..3c556d07ac 100644 --- a/sysdeps/x86_64/multiarch/ifunc-impl-list.c +++ b/sysdeps/x86_64/multiarch/ifunc-impl-list.c @@ -436,6 +436,13 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, /* Support sysdeps/x86_64/multiarch/strcasecmp_l.c. */ IFUNC_IMPL (i, name, strcasecmp, + IFUNC_IMPL_ADD (array, i, strcasecmp, + CPU_FEATURE_USABLE (AVX2), + __strcasecmp_avx2) + IFUNC_IMPL_ADD (array, i, strcasecmp, + (CPU_FEATURE_USABLE (AVX2) + && CPU_FEATURE_USABLE (RTM)), + __strcasecmp_avx2_rtm) IFUNC_IMPL_ADD (array, i, strcasecmp, CPU_FEATURE_USABLE (AVX), __strcasecmp_avx) @@ -449,6 +456,13 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, /* Support sysdeps/x86_64/multiarch/strcasecmp_l.c. */ IFUNC_IMPL (i, name, strcasecmp_l, + IFUNC_IMPL_ADD (array, i, strcasecmp, + CPU_FEATURE_USABLE (AVX2), + __strcasecmp_l_avx2) + IFUNC_IMPL_ADD (array, i, strcasecmp, + (CPU_FEATURE_USABLE (AVX2) + && CPU_FEATURE_USABLE (RTM)), + __strcasecmp_l_avx2_rtm) IFUNC_IMPL_ADD (array, i, strcasecmp_l, CPU_FEATURE_USABLE (AVX), __strcasecmp_l_avx) @@ -576,6 +590,13 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, /* Support sysdeps/x86_64/multiarch/strncase_l.c. */ IFUNC_IMPL (i, name, strncasecmp, + IFUNC_IMPL_ADD (array, i, strncasecmp, + CPU_FEATURE_USABLE (AVX2), + __strncasecmp_avx2) + IFUNC_IMPL_ADD (array, i, strncasecmp, + (CPU_FEATURE_USABLE (AVX2) + && CPU_FEATURE_USABLE (RTM)), + __strncasecmp_avx2_rtm) IFUNC_IMPL_ADD (array, i, strncasecmp, CPU_FEATURE_USABLE (AVX), __strncasecmp_avx) @@ -590,6 +611,13 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, /* Support sysdeps/x86_64/multiarch/strncase_l.c. */ IFUNC_IMPL (i, name, strncasecmp_l, + IFUNC_IMPL_ADD (array, i, strncasecmp, + CPU_FEATURE_USABLE (AVX2), + __strncasecmp_l_avx2) + IFUNC_IMPL_ADD (array, i, strncasecmp, + (CPU_FEATURE_USABLE (AVX2) + && CPU_FEATURE_USABLE (RTM)), + __strncasecmp_l_avx2_rtm) IFUNC_IMPL_ADD (array, i, strncasecmp_l, CPU_FEATURE_USABLE (AVX), __strncasecmp_l_avx) diff --git a/sysdeps/x86_64/multiarch/ifunc-strcasecmp.h b/sysdeps/x86_64/multiarch/ifunc-strcasecmp.h index 9e3cc61ac0..c4de111fd0 100644 --- a/sysdeps/x86_64/multiarch/ifunc-strcasecmp.h +++ b/sysdeps/x86_64/multiarch/ifunc-strcasecmp.h @@ -23,12 +23,24 @@ extern __typeof (REDIRECT_NAME) OPTIMIZE (sse2) attribute_hidden; extern __typeof (REDIRECT_NAME) OPTIMIZE (ssse3) attribute_hidden; extern __typeof (REDIRECT_NAME) OPTIMIZE (sse42) attribute_hidden; extern __typeof (REDIRECT_NAME) OPTIMIZE (avx) attribute_hidden; +extern __typeof (REDIRECT_NAME) OPTIMIZE (avx2) attribute_hidden; +extern __typeof (REDIRECT_NAME) OPTIMIZE (avx2_rtm) attribute_hidden; static inline void * IFUNC_SELECTOR (void) { const struct cpu_features* cpu_features = __get_cpu_features (); + if (CPU_FEATURE_USABLE_P (cpu_features, AVX2) + && CPU_FEATURES_ARCH_P (cpu_features, AVX_Fast_Unaligned_Load)) + { + if (CPU_FEATURE_USABLE_P (cpu_features, RTM)) + return OPTIMIZE (avx2_rtm); + + if (!CPU_FEATURES_ARCH_P (cpu_features, Prefer_No_VZEROUPPER)) + return OPTIMIZE (avx2); + } + if (CPU_FEATURE_USABLE_P (cpu_features, AVX)) return OPTIMIZE (avx); diff --git a/sysdeps/x86_64/multiarch/strcasecmp_l-avx2-rtm.S b/sysdeps/x86_64/multiarch/strcasecmp_l-avx2-rtm.S new file mode 100644 index 0000000000..09957fc3c5 --- /dev/null +++ b/sysdeps/x86_64/multiarch/strcasecmp_l-avx2-rtm.S @@ -0,0 +1,15 @@ +#ifndef STRCMP +# define STRCMP __strcasecmp_l_avx2_rtm +#endif + +#define _GLABEL(x) x ## _rtm +#define GLABEL(x) _GLABEL(x) + +#define ZERO_UPPER_VEC_REGISTERS_RETURN \ + ZERO_UPPER_VEC_REGISTERS_RETURN_XTEST + +#define VZEROUPPER_RETURN jmp L(return_vzeroupper) + +#define SECTION(p) p##.avx.rtm + +#include "strcasecmp_l-avx2.S" diff --git a/sysdeps/x86_64/multiarch/strcasecmp_l-avx2.S b/sysdeps/x86_64/multiarch/strcasecmp_l-avx2.S new file mode 100644 index 0000000000..e2762f2a22 --- /dev/null +++ b/sysdeps/x86_64/multiarch/strcasecmp_l-avx2.S @@ -0,0 +1,23 @@ +/* strcasecmp_l optimized with AVX2. + Copyright (C) 2017-2022 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + <https://www.gnu.org/licenses/>. */ + +#ifndef STRCMP +# define STRCMP __strcasecmp_l_avx2 +#endif +#define USE_AS_STRCASECMP_L +#include "strcmp-avx2.S" diff --git a/sysdeps/x86_64/multiarch/strcmp-avx2.S b/sysdeps/x86_64/multiarch/strcmp-avx2.S index b94fc5c39a..3366d0b083 100644 --- a/sysdeps/x86_64/multiarch/strcmp-avx2.S +++ b/sysdeps/x86_64/multiarch/strcmp-avx2.S @@ -20,6 +20,10 @@ # include <sysdep.h> +# if defined USE_AS_STRCASECMP_L +# include "locale-defines.h" +# endif + # ifndef STRCMP # define STRCMP __strcmp_avx2 # endif @@ -74,13 +78,88 @@ # define VEC_OFFSET (-VEC_SIZE) # endif +# ifdef USE_AS_STRCASECMP_L +# define BYTE_LOOP_REG OFFSET_REG +# else +# define BYTE_LOOP_REG ecx +# endif + +# ifdef USE_AS_STRCASECMP_L +# ifdef USE_AS_STRNCMP +# define STRCASECMP __strncasecmp_avx2 +# define LOCALE_REG rcx +# define LOCALE_REG_LP RCX_LP +# define STRCASECMP_NONASCII __strncasecmp_l_nonascii +# else +# define STRCASECMP __strcasecmp_avx2 +# define LOCALE_REG rdx +# define LOCALE_REG_LP RDX_LP +# define STRCASECMP_NONASCII __strcasecmp_l_nonascii +# endif +# endif + # define xmmZERO xmm15 # define ymmZERO ymm15 +# define LCASE_MIN_ymm %ymm10 +# define LCASE_MAX_ymm %ymm11 +# define CASE_ADD_ymm %ymm12 + +# define LCASE_MIN_xmm %xmm10 +# define LCASE_MAX_xmm %xmm11 +# define CASE_ADD_xmm %xmm12 + + /* r11 is never use elsewhere so this is safe to maintain. */ +# define TOLOWER_BASE %r11 + # ifndef SECTION # define SECTION(p) p##.avx # endif +# ifdef USE_AS_STRCASECMP_L +# define REG(x, y) x ## y +# define TOLOWER(reg1_in, reg1_out, reg2_in, reg2_out, ext) \ + vpaddb REG(LCASE_MIN_, ext), reg1_in, REG(%ext, 8); \ + vpaddb REG(LCASE_MIN_, ext), reg2_in, REG(%ext, 9); \ + vpcmpgtb REG(LCASE_MAX_, ext), REG(%ext, 8), REG(%ext, 8); \ + vpcmpgtb REG(LCASE_MAX_, ext), REG(%ext, 9), REG(%ext, 9); \ + vpandn REG(CASE_ADD_, ext), REG(%ext, 8), REG(%ext, 8); \ + vpandn REG(CASE_ADD_, ext), REG(%ext, 9), REG(%ext, 9); \ + vpaddb REG(%ext, 8), reg1_in, reg1_out; \ + vpaddb REG(%ext, 9), reg2_in, reg2_out + +# define TOLOWER_gpr(src, dst) movl (TOLOWER_BASE, src, 4), dst +# define TOLOWER_ymm(...) TOLOWER(__VA_ARGS__, ymm) +# define TOLOWER_xmm(...) TOLOWER(__VA_ARGS__, xmm) + +# define CMP_R1_R2(s1_reg, s2_reg, scratch_reg, reg_out, ext) \ + TOLOWER (s1_reg, scratch_reg, s2_reg, s2_reg, ext); \ + VPCMPEQ scratch_reg, s2_reg, reg_out + +# define CMP_R1_S2(s1_reg, s2_mem, scratch_reg, reg_out, ext) \ + VMOVU s2_mem, reg_out; \ + CMP_R1_R2(s1_reg, reg_out, scratch_reg, reg_out, ext) + +# define CMP_R1_R2_ymm(...) CMP_R1_R2(__VA_ARGS__, ymm) +# define CMP_R1_R2_xmm(...) CMP_R1_R2(__VA_ARGS__, xmm) + +# define CMP_R1_S2_ymm(...) CMP_R1_S2(__VA_ARGS__, ymm) +# define CMP_R1_S2_xmm(...) CMP_R1_S2(__VA_ARGS__, xmm) + +# else +# define TOLOWER_gpr(...) +# define TOLOWER_ymm(...) +# define TOLOWER_xmm(...) + +# define CMP_R1_R2_ymm(s1_reg, s2_reg, scratch_reg, reg_out) \ + VPCMPEQ s2_reg, s1_reg, reg_out + +# define CMP_R1_R2_xmm(...) CMP_R1_R2_ymm(__VA_ARGS__) + +# define CMP_R1_S2_ymm(...) CMP_R1_R2_ymm(__VA_ARGS__) +# define CMP_R1_S2_xmm(...) CMP_R1_R2_xmm(__VA_ARGS__) +# endif + /* Warning! wcscmp/wcsncmp have to use SIGNED comparison for elements. strcmp/strncmp have to use UNSIGNED comparison for elements. @@ -102,8 +181,49 @@ returned. */ .section SECTION(.text), "ax", @progbits -ENTRY(STRCMP) + .align 16 + .type STRCMP, @function + .globl STRCMP + .hidden STRCMP + +# ifndef GLABEL +# define GLABEL(...) __VA_ARGS__ +# endif + +# ifdef USE_AS_STRCASECMP_L +ENTRY (GLABEL(STRCASECMP)) + movq __libc_tsd_LOCALE@gottpoff(%rip), %rax + mov %fs:(%rax), %LOCALE_REG_LP + + /* Either 1 or 5 bytes (dependeing if CET is enabled). */ + .p2align 4 +END (GLABEL(STRCASECMP)) + /* FALLTHROUGH to strcasecmp/strncasecmp_l. */ +# endif + + .p2align 4 +STRCMP: + cfi_startproc + _CET_ENDBR + CALL_MCOUNT + +# if defined USE_AS_STRCASECMP_L + /* We have to fall back on the C implementation for locales with + encodings not matching ASCII for single bytes. */ +# if LOCALE_T___LOCALES != 0 || LC_CTYPE != 0 + mov LOCALE_T___LOCALES + LC_CTYPE * LP_SIZE(%LOCALE_REG), %RAX_LP +# else + mov (%LOCALE_REG), %RAX_LP +# endif + testl $1, LOCALE_DATA_VALUES + _NL_CTYPE_NONASCII_CASE * SIZEOF_VALUES(%rax) + jne STRCASECMP_NONASCII + leaq _nl_C_LC_CTYPE_tolower + 128 * 4(%rip), TOLOWER_BASE +# endif + # ifdef USE_AS_STRNCMP + /* Don't overwrite LOCALE_REG (rcx) until we have pass + L(one_or_less). Otherwise we might use the wrong locale in + the OVERFLOW_STRCMP (strcasecmp_l). */ # ifdef __ILP32__ /* Clear the upper 32 bits. */ movl %edx, %edx @@ -128,6 +248,30 @@ ENTRY(STRCMP) # endif # endif vpxor %xmmZERO, %xmmZERO, %xmmZERO +# if defined USE_AS_STRCASECMP_L + .section .rodata.cst32, "aM", @progbits, 32 + .align 32 +L(lcase_min): + .quad 0x3f3f3f3f3f3f3f3f + .quad 0x3f3f3f3f3f3f3f3f + .quad 0x3f3f3f3f3f3f3f3f + .quad 0x3f3f3f3f3f3f3f3f +L(lcase_max): + .quad 0x9999999999999999 + .quad 0x9999999999999999 + .quad 0x9999999999999999 + .quad 0x9999999999999999 +L(case_add): + .quad 0x2020202020202020 + .quad 0x2020202020202020 + .quad 0x2020202020202020 + .quad 0x2020202020202020 + .previous + + vmovdqa L(lcase_min)(%rip), LCASE_MIN_ymm + vmovdqa L(lcase_max)(%rip), LCASE_MAX_ymm + vmovdqa L(case_add)(%rip), CASE_ADD_ymm +# endif movl %edi, %eax orl %esi, %eax sall $20, %eax @@ -138,8 +282,10 @@ ENTRY(STRCMP) L(no_page_cross): /* Safe to compare 4x vectors. */ VMOVU (%rdi), %ymm0 - /* 1s where s1 and s2 equal. */ - VPCMPEQ (%rsi), %ymm0, %ymm1 + /* 1s where s1 and s2 equal. Just VPCMPEQ if its not strcasecmp. + Otherwise converts ymm0 and load from rsi to lower. ymm2 is + scratch and ymm1 is the return. */ + CMP_R1_S2_ymm (%ymm0, (%rsi), %ymm2, %ymm1) /* 1s at null CHAR. */ VPCMPEQ %ymm0, %ymmZERO, %ymm2 /* 1s where s1 and s2 equal AND not null CHAR. */ @@ -172,6 +318,8 @@ L(return_vec_0): # else movzbl (%rdi, %rcx), %eax movzbl (%rsi, %rcx), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax # endif L(ret0): @@ -192,6 +340,10 @@ L(ret_zero): .p2align 4,, 5 L(one_or_less): +# ifdef USE_AS_STRCASECMP_L + /* Set locale argument for strcasecmp. */ + movq %LOCALE_REG, %rdx +# endif jb L(ret_zero) # ifdef USE_AS_WCSCMP /* 'nbe' covers the case where length is negative (large @@ -211,6 +363,8 @@ L(one_or_less): jnbe __strcmp_avx2 movzbl (%rdi), %eax movzbl (%rsi), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax # endif L(ret1): @@ -238,6 +392,8 @@ L(return_vec_1): # else movzbl VEC_SIZE(%rdi, %rcx), %eax movzbl VEC_SIZE(%rsi, %rcx), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax # endif L(ret2): @@ -269,6 +425,8 @@ L(return_vec_2): # else movzbl (VEC_SIZE * 2)(%rdi, %rcx), %eax movzbl (VEC_SIZE * 2)(%rsi, %rcx), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax # endif L(ret3): @@ -289,6 +447,8 @@ L(return_vec_3): # else movzbl (VEC_SIZE * 3)(%rdi, %rcx), %eax movzbl (VEC_SIZE * 3)(%rsi, %rcx), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax # endif L(ret4): @@ -299,7 +459,7 @@ L(ret4): L(more_3x_vec): /* Safe to compare 4x vectors. */ VMOVU VEC_SIZE(%rdi), %ymm0 - VPCMPEQ VEC_SIZE(%rsi), %ymm0, %ymm1 + CMP_R1_S2_ymm (%ymm0, VEC_SIZE(%rsi), %ymm2, %ymm1) VPCMPEQ %ymm0, %ymmZERO, %ymm2 vpandn %ymm1, %ymm2, %ymm1 vpmovmskb %ymm1, %ecx @@ -312,7 +472,7 @@ L(more_3x_vec): # endif VMOVU (VEC_SIZE * 2)(%rdi), %ymm0 - VPCMPEQ (VEC_SIZE * 2)(%rsi), %ymm0, %ymm1 + CMP_R1_S2_ymm (%ymm0, (VEC_SIZE * 2)(%rsi), %ymm2, %ymm1) VPCMPEQ %ymm0, %ymmZERO, %ymm2 vpandn %ymm1, %ymm2, %ymm1 vpmovmskb %ymm1, %ecx @@ -320,7 +480,7 @@ L(more_3x_vec): jnz L(return_vec_2) VMOVU (VEC_SIZE * 3)(%rdi), %ymm0 - VPCMPEQ (VEC_SIZE * 3)(%rsi), %ymm0, %ymm1 + CMP_R1_S2_ymm (%ymm0, (VEC_SIZE * 3)(%rsi), %ymm2, %ymm1) VPCMPEQ %ymm0, %ymmZERO, %ymm2 vpandn %ymm1, %ymm2, %ymm1 vpmovmskb %ymm1, %ecx @@ -395,12 +555,10 @@ L(loop_skip_page_cross_check): VMOVA (VEC_SIZE * 3)(%rdi), %ymm6 /* ymm1 all 1s where s1 and s2 equal. All 0s otherwise. */ - VPCMPEQ (VEC_SIZE * 0)(%rsi), %ymm0, %ymm1 - - VPCMPEQ (VEC_SIZE * 1)(%rsi), %ymm2, %ymm3 - VPCMPEQ (VEC_SIZE * 2)(%rsi), %ymm4, %ymm5 - VPCMPEQ (VEC_SIZE * 3)(%rsi), %ymm6, %ymm7 - + CMP_R1_S2_ymm (%ymm0, (VEC_SIZE * 0)(%rsi), %ymm3, %ymm1) + CMP_R1_S2_ymm (%ymm2, (VEC_SIZE * 1)(%rsi), %ymm5, %ymm3) + CMP_R1_S2_ymm (%ymm4, (VEC_SIZE * 2)(%rsi), %ymm7, %ymm5) + CMP_R1_S2_ymm (%ymm6, (VEC_SIZE * 3)(%rsi), %ymm13, %ymm7) /* If any mismatches or null CHAR then 0 CHAR, otherwise non- zero. */ @@ -469,6 +627,8 @@ L(return_vec_2_3_end): # else movzbl (VEC_SIZE * 2 - VEC_OFFSET)(%rdi, %LOOP_REG64), %eax movzbl (VEC_SIZE * 2 - VEC_OFFSET)(%rsi, %LOOP_REG64), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax xorl %r8d, %eax subl %r8d, %eax @@ -512,6 +672,8 @@ L(return_vec_0_end): # else movzbl (%rdi, %rcx), %eax movzbl (%rsi, %rcx), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax xorl %r8d, %eax subl %r8d, %eax @@ -534,6 +696,8 @@ L(return_vec_1_end): # else movzbl VEC_SIZE(%rdi, %rcx), %eax movzbl VEC_SIZE(%rsi, %rcx), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax xorl %r8d, %eax subl %r8d, %eax @@ -560,6 +724,8 @@ L(return_vec_2_end): # else movzbl (VEC_SIZE * 2)(%rdi, %rcx), %eax movzbl (VEC_SIZE * 2)(%rsi, %rcx), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax xorl %r8d, %eax subl %r8d, %eax @@ -587,7 +753,7 @@ L(page_cross_during_loop): jle L(less_1x_vec_till_page_cross) VMOVA (%rdi), %ymm0 - VPCMPEQ (%rsi), %ymm0, %ymm1 + CMP_R1_S2_ymm (%ymm0, (%rsi), %ymm2, %ymm1) VPCMPEQ %ymm0, %ymmZERO, %ymm2 vpandn %ymm1, %ymm2, %ymm1 vpmovmskb %ymm1, %ecx @@ -609,7 +775,7 @@ L(less_1x_vec_till_page_cross): here, it means the previous page (rdi - VEC_SIZE) has already been loaded earlier so must be valid. */ VMOVU -VEC_SIZE(%rdi, %rax), %ymm0 - VPCMPEQ -VEC_SIZE(%rsi, %rax), %ymm0, %ymm1 + CMP_R1_S2_ymm (%ymm0, -VEC_SIZE(%rsi, %rax), %ymm2, %ymm1) VPCMPEQ %ymm0, %ymmZERO, %ymm2 vpandn %ymm1, %ymm2, %ymm1 vpmovmskb %ymm1, %ecx @@ -651,6 +817,8 @@ L(return_page_cross_cmp_mem): # else movzbl VEC_OFFSET(%rdi, %rcx), %eax movzbl VEC_OFFSET(%rsi, %rcx), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax xorl %r8d, %eax subl %r8d, %eax @@ -677,7 +845,7 @@ L(more_2x_vec_till_page_cross): iteration here. */ VMOVU VEC_SIZE(%rdi), %ymm0 - VPCMPEQ VEC_SIZE(%rsi), %ymm0, %ymm1 + CMP_R1_S2_ymm (%ymm0, VEC_SIZE(%rsi), %ymm2, %ymm1) VPCMPEQ %ymm0, %ymmZERO, %ymm2 vpandn %ymm1, %ymm2, %ymm1 vpmovmskb %ymm1, %ecx @@ -693,7 +861,7 @@ L(more_2x_vec_till_page_cross): /* Safe to include comparisons from lower bytes. */ VMOVU -(VEC_SIZE * 2)(%rdi, %rax), %ymm0 - VPCMPEQ -(VEC_SIZE * 2)(%rsi, %rax), %ymm0, %ymm1 + CMP_R1_S2_ymm (%ymm0, -(VEC_SIZE * 2)(%rsi, %rax), %ymm2, %ymm1) VPCMPEQ %ymm0, %ymmZERO, %ymm2 vpandn %ymm1, %ymm2, %ymm1 vpmovmskb %ymm1, %ecx @@ -701,7 +869,7 @@ L(more_2x_vec_till_page_cross): jnz L(return_vec_page_cross_0) VMOVU -(VEC_SIZE * 1)(%rdi, %rax), %ymm0 - VPCMPEQ -(VEC_SIZE * 1)(%rsi, %rax), %ymm0, %ymm1 + CMP_R1_S2_ymm (%ymm0, -(VEC_SIZE * 1)(%rsi, %rax), %ymm2, %ymm1) VPCMPEQ %ymm0, %ymmZERO, %ymm2 vpandn %ymm1, %ymm2, %ymm1 vpmovmskb %ymm1, %ecx @@ -719,8 +887,8 @@ L(more_2x_vec_till_page_cross): VMOVA (VEC_SIZE * 2)(%rdi), %ymm4 VMOVA (VEC_SIZE * 3)(%rdi), %ymm6 - VPCMPEQ (VEC_SIZE * 2)(%rsi), %ymm4, %ymm5 - VPCMPEQ (VEC_SIZE * 3)(%rsi), %ymm6, %ymm7 + CMP_R1_S2_ymm (%ymm4, (VEC_SIZE * 2)(%rsi), %ymm7, %ymm5) + CMP_R1_S2_ymm (%ymm6, (VEC_SIZE * 3)(%rsi), %ymm13, %ymm7) vpand %ymm4, %ymm5, %ymm5 vpand %ymm6, %ymm7, %ymm7 VPMINU %ymm5, %ymm7, %ymm7 @@ -771,6 +939,8 @@ L(return_vec_page_cross_1): # else movzbl VEC_OFFSET(%rdi, %rcx), %eax movzbl VEC_OFFSET(%rsi, %rcx), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax xorl %r8d, %eax subl %r8d, %eax @@ -826,7 +996,7 @@ L(page_cross): L(page_cross_loop): VMOVU (%rdi, %OFFSET_REG64), %ymm0 - VPCMPEQ (%rsi, %OFFSET_REG64), %ymm0, %ymm1 + CMP_R1_S2_ymm (%ymm0, (%rsi, %OFFSET_REG64), %ymm2, %ymm1) VPCMPEQ %ymm0, %ymmZERO, %ymm2 vpandn %ymm1, %ymm2, %ymm1 vpmovmskb %ymm1, %ecx @@ -844,11 +1014,11 @@ L(page_cross_loop): subl %eax, %OFFSET_REG /* OFFSET_REG has distance to page cross - VEC_SIZE. Guranteed to not cross page so is safe to load. Since we have already - loaded at least 1 VEC from rsi it is also guranteed to be safe. - */ + loaded at least 1 VEC from rsi it is also guranteed to be + safe. */ VMOVU (%rdi, %OFFSET_REG64), %ymm0 - VPCMPEQ (%rsi, %OFFSET_REG64), %ymm0, %ymm1 + CMP_R1_S2_ymm (%ymm0, (%rsi, %OFFSET_REG64), %ymm2, %ymm1) VPCMPEQ %ymm0, %ymmZERO, %ymm2 vpandn %ymm1, %ymm2, %ymm1 vpmovmskb %ymm1, %ecx @@ -881,6 +1051,8 @@ L(ret_vec_page_cross_cont): # else movzbl (%rdi, %rcx), %eax movzbl (%rsi, %rcx), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax xorl %r8d, %eax subl %r8d, %eax @@ -934,7 +1106,7 @@ L(less_1x_vec_till_page): ja L(less_16_till_page) VMOVU (%rdi), %xmm0 - VPCMPEQ (%rsi), %xmm0, %xmm1 + CMP_R1_S2_xmm (%xmm0, (%rsi), %xmm2, %xmm1) VPCMPEQ %xmm0, %xmmZERO, %xmm2 vpandn %xmm1, %xmm2, %xmm1 vpmovmskb %ymm1, %ecx @@ -952,7 +1124,7 @@ L(less_1x_vec_till_page): # endif VMOVU (%rdi, %OFFSET_REG64), %xmm0 - VPCMPEQ (%rsi, %OFFSET_REG64), %xmm0, %xmm1 + CMP_R1_S2_xmm (%xmm0, (%rsi, %OFFSET_REG64), %xmm2, %xmm1) VPCMPEQ %xmm0, %xmmZERO, %xmm2 vpandn %xmm1, %xmm2, %xmm1 vpmovmskb %ymm1, %ecx @@ -990,7 +1162,7 @@ L(less_16_till_page): vmovq (%rdi), %xmm0 vmovq (%rsi), %xmm1 VPCMPEQ %xmm0, %xmmZERO, %xmm2 - VPCMPEQ %xmm1, %xmm0, %xmm1 + CMP_R1_R2_xmm (%xmm0, %xmm1, %xmm3, %xmm1) vpandn %xmm1, %xmm2, %xmm1 vpmovmskb %ymm1, %ecx incb %cl @@ -1010,7 +1182,7 @@ L(less_16_till_page): vmovq (%rdi, %OFFSET_REG64), %xmm0 vmovq (%rsi, %OFFSET_REG64), %xmm1 VPCMPEQ %xmm0, %xmmZERO, %xmm2 - VPCMPEQ %xmm1, %xmm0, %xmm1 + CMP_R1_R2_xmm (%xmm0, %xmm1, %xmm3, %xmm1) vpandn %xmm1, %xmm2, %xmm1 vpmovmskb %ymm1, %ecx incb %cl @@ -1066,7 +1238,7 @@ L(ret_less_8_wcs): vmovd (%rdi), %xmm0 vmovd (%rsi), %xmm1 VPCMPEQ %xmm0, %xmmZERO, %xmm2 - VPCMPEQ %xmm1, %xmm0, %xmm1 + CMP_R1_R2_xmm (%xmm0, %xmm1, %xmm3, %xmm1) vpandn %xmm1, %xmm2, %xmm1 vpmovmskb %ymm1, %ecx subl $0xf, %ecx @@ -1085,7 +1257,7 @@ L(ret_less_8_wcs): vmovd (%rdi, %OFFSET_REG64), %xmm0 vmovd (%rsi, %OFFSET_REG64), %xmm1 VPCMPEQ %xmm0, %xmmZERO, %xmm2 - VPCMPEQ %xmm1, %xmm0, %xmm1 + CMP_R1_R2_xmm (%xmm0, %xmm1, %xmm3, %xmm1) vpandn %xmm1, %xmm2, %xmm1 vpmovmskb %ymm1, %ecx subl $0xf, %ecx @@ -1119,7 +1291,9 @@ L(less_4_till_page): L(less_4_loop): movzbl (%rdi), %eax movzbl (%rsi, %rdi), %ecx - subl %ecx, %eax + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %BYTE_LOOP_REG) + subl %BYTE_LOOP_REG, %eax jnz L(ret_less_4_loop) testl %ecx, %ecx jz L(ret_zero_4_loop) @@ -1146,5 +1320,6 @@ L(ret_less_4_loop): subl %r8d, %eax ret # endif -END(STRCMP) + cfi_endproc + .size STRCMP, .-STRCMP #endif diff --git a/sysdeps/x86_64/multiarch/strncase_l-avx2-rtm.S b/sysdeps/x86_64/multiarch/strncase_l-avx2-rtm.S new file mode 100644 index 0000000000..58c05dcfb8 --- /dev/null +++ b/sysdeps/x86_64/multiarch/strncase_l-avx2-rtm.S @@ -0,0 +1,16 @@ +#ifndef STRCMP +# define STRCMP __strncasecmp_l_avx2_rtm +#endif + +#define _GLABEL(x) x ## _rtm +#define GLABEL(x) _GLABEL(x) + +#define ZERO_UPPER_VEC_REGISTERS_RETURN \ + ZERO_UPPER_VEC_REGISTERS_RETURN_XTEST + +#define VZEROUPPER_RETURN jmp L(return_vzeroupper) + +#define SECTION(p) p##.avx.rtm +#define OVERFLOW_STRCMP __strcasecmp_l_avx2_rtm + +#include "strncase_l-avx2.S" diff --git a/sysdeps/x86_64/multiarch/strncase_l-avx2.S b/sysdeps/x86_64/multiarch/strncase_l-avx2.S new file mode 100644 index 0000000000..48c0aa21f8 --- /dev/null +++ b/sysdeps/x86_64/multiarch/strncase_l-avx2.S @@ -0,0 +1,27 @@ +/* strncasecmp_l optimized with AVX2. + Copyright (C) 2017-2022 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + <https://www.gnu.org/licenses/>. */ + +#ifndef STRCMP +# define STRCMP __strncasecmp_l_avx2 +#endif +#define USE_AS_STRCASECMP_L +#define USE_AS_STRNCMP +#ifndef OVERFLOW_STRCMP +# define OVERFLOW_STRCMP __strcasecmp_l_avx2 +#endif +#include "strcmp-avx2.S" -- 2.35.1 ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v4 22/23] x86: Add EVEX optimized str{n}casecmp 2022-05-12 19:47 ` [PATCH v4 22/23] x86: Add EVEX optimized str{n}casecmp Sunil Pandey @ 2022-05-12 19:52 ` Sunil Pandey 0 siblings, 0 replies; 3+ messages in thread From: Sunil Pandey @ 2022-05-12 19:52 UTC (permalink / raw) To: Noah Goldstein, Libc-stable Mailing List; +Cc: H.J. Lu, GNU C Library On Thu, May 12, 2022 at 12:47 PM Sunil Pandey <skpgkp2@gmail.com> wrote: > > On Fri, Mar 25, 2022 at 11:20 AM Noah Goldstein via Libc-alpha > <libc-alpha@sourceware.org> wrote: > > > > On Fri, Mar 25, 2022 at 1:15 PM H.J. Lu <hjl.tools@gmail.com> wrote: > > > > > > On Thu, Mar 24, 2022 at 4:56 PM Noah Goldstein <goldstein.w.n@gmail.com> wrote: > > > > > > > > geometric_mean(N=40) of all benchmarks EVEX / SSE42: .621 > > > > > > > > All string/memory tests pass. > > > > Reviewed-by: H.J. Lu <hjl.tools@gmail.com> > > > > --- > > > > sysdeps/x86_64/multiarch/Makefile | 2 + > > > > sysdeps/x86_64/multiarch/ifunc-impl-list.c | 16 + > > > > sysdeps/x86_64/multiarch/ifunc-strcasecmp.h | 5 + > > > > sysdeps/x86_64/multiarch/strcasecmp_l-evex.S | 23 ++ > > > > sysdeps/x86_64/multiarch/strcmp-evex.S | 290 ++++++++++++++++--- > > > > sysdeps/x86_64/multiarch/strncase_l-evex.S | 25 ++ > > > > 6 files changed, 321 insertions(+), 40 deletions(-) > > > > create mode 100644 sysdeps/x86_64/multiarch/strcasecmp_l-evex.S > > > > create mode 100644 sysdeps/x86_64/multiarch/strncase_l-evex.S > > > > > > > > diff --git a/sysdeps/x86_64/multiarch/Makefile b/sysdeps/x86_64/multiarch/Makefile > > > > index 06e1848823..35d80dc2ff 100644 > > > > --- a/sysdeps/x86_64/multiarch/Makefile > > > > +++ b/sysdeps/x86_64/multiarch/Makefile > > > > @@ -57,6 +57,7 @@ sysdep_routines += \ > > > > strcasecmp_l-avx \ > > > > strcasecmp_l-avx2 \ > > > > strcasecmp_l-avx2-rtm \ > > > > + strcasecmp_l-evex \ > > > > strcasecmp_l-sse2 \ > > > > strcasecmp_l-sse4_2 \ > > > > strcasecmp_l-ssse3 \ > > > > @@ -97,6 +98,7 @@ sysdep_routines += \ > > > > strncase_l-avx \ > > > > strncase_l-avx2 \ > > > > strncase_l-avx2-rtm \ > > > > + strncase_l-evex \ > > > > strncase_l-sse2 \ > > > > strncase_l-sse4_2 \ > > > > strncase_l-ssse3 \ > > > > diff --git a/sysdeps/x86_64/multiarch/ifunc-impl-list.c b/sysdeps/x86_64/multiarch/ifunc-impl-list.c > > > > index 3c556d07ac..f1a4d3dac2 100644 > > > > --- a/sysdeps/x86_64/multiarch/ifunc-impl-list.c > > > > +++ b/sysdeps/x86_64/multiarch/ifunc-impl-list.c > > > > @@ -436,6 +436,10 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, > > > > > > > > /* Support sysdeps/x86_64/multiarch/strcasecmp_l.c. */ > > > > IFUNC_IMPL (i, name, strcasecmp, > > > > + IFUNC_IMPL_ADD (array, i, strcasecmp, > > > > + (CPU_FEATURE_USABLE (AVX512VL) > > > > + && CPU_FEATURE_USABLE (AVX512BW)), > > > > + __strcasecmp_evex) > > > > IFUNC_IMPL_ADD (array, i, strcasecmp, > > > > CPU_FEATURE_USABLE (AVX2), > > > > __strcasecmp_avx2) > > > > @@ -456,6 +460,10 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, > > > > > > > > /* Support sysdeps/x86_64/multiarch/strcasecmp_l.c. */ > > > > IFUNC_IMPL (i, name, strcasecmp_l, > > > > + IFUNC_IMPL_ADD (array, i, strcasecmp, > > > > + (CPU_FEATURE_USABLE (AVX512VL) > > > > + && CPU_FEATURE_USABLE (AVX512BW)), > > > > + __strcasecmp_l_evex) > > > > IFUNC_IMPL_ADD (array, i, strcasecmp, > > > > CPU_FEATURE_USABLE (AVX2), > > > > __strcasecmp_l_avx2) > > > > @@ -590,6 +598,10 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, > > > > > > > > /* Support sysdeps/x86_64/multiarch/strncase_l.c. */ > > > > IFUNC_IMPL (i, name, strncasecmp, > > > > + IFUNC_IMPL_ADD (array, i, strncasecmp, > > > > + (CPU_FEATURE_USABLE (AVX512VL) > > > > + && CPU_FEATURE_USABLE (AVX512BW)), > > > > + __strncasecmp_evex) > > > > IFUNC_IMPL_ADD (array, i, strncasecmp, > > > > CPU_FEATURE_USABLE (AVX2), > > > > __strncasecmp_avx2) > > > > @@ -611,6 +623,10 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, > > > > > > > > /* Support sysdeps/x86_64/multiarch/strncase_l.c. */ > > > > IFUNC_IMPL (i, name, strncasecmp_l, > > > > + IFUNC_IMPL_ADD (array, i, strncasecmp, > > > > + (CPU_FEATURE_USABLE (AVX512VL) > > > > + && CPU_FEATURE_USABLE (AVX512BW)), > > > > + __strncasecmp_l_evex) > > > > IFUNC_IMPL_ADD (array, i, strncasecmp, > > > > CPU_FEATURE_USABLE (AVX2), > > > > __strncasecmp_l_avx2) > > > > diff --git a/sysdeps/x86_64/multiarch/ifunc-strcasecmp.h b/sysdeps/x86_64/multiarch/ifunc-strcasecmp.h > > > > index c4de111fd0..bf0d146e7f 100644 > > > > --- a/sysdeps/x86_64/multiarch/ifunc-strcasecmp.h > > > > +++ b/sysdeps/x86_64/multiarch/ifunc-strcasecmp.h > > > > @@ -25,6 +25,7 @@ extern __typeof (REDIRECT_NAME) OPTIMIZE (sse42) attribute_hidden; > > > > extern __typeof (REDIRECT_NAME) OPTIMIZE (avx) attribute_hidden; > > > > extern __typeof (REDIRECT_NAME) OPTIMIZE (avx2) attribute_hidden; > > > > extern __typeof (REDIRECT_NAME) OPTIMIZE (avx2_rtm) attribute_hidden; > > > > +extern __typeof (REDIRECT_NAME) OPTIMIZE (evex) attribute_hidden; > > > > > > > > static inline void * > > > > IFUNC_SELECTOR (void) > > > > @@ -34,6 +35,10 @@ IFUNC_SELECTOR (void) > > > > if (CPU_FEATURE_USABLE_P (cpu_features, AVX2) > > > > && CPU_FEATURES_ARCH_P (cpu_features, AVX_Fast_Unaligned_Load)) > > > > { > > > > + if (CPU_FEATURE_USABLE_P (cpu_features, AVX512VL) > > > > + && CPU_FEATURE_USABLE_P (cpu_features, AVX512BW)) > > > > + return OPTIMIZE (evex); > > > > + > > > > if (CPU_FEATURE_USABLE_P (cpu_features, RTM)) > > > > return OPTIMIZE (avx2_rtm); > > > > > > > > diff --git a/sysdeps/x86_64/multiarch/strcasecmp_l-evex.S b/sysdeps/x86_64/multiarch/strcasecmp_l-evex.S > > > > new file mode 100644 > > > > index 0000000000..58642db748 > > > > --- /dev/null > > > > +++ b/sysdeps/x86_64/multiarch/strcasecmp_l-evex.S > > > > @@ -0,0 +1,23 @@ > > > > +/* strcasecmp_l optimized with EVEX. > > > > + Copyright (C) 2017-2022 Free Software Foundation, Inc. > > > > + This file is part of the GNU C Library. > > > > + > > > > + The GNU C Library is free software; you can redistribute it and/or > > > > + modify it under the terms of the GNU Lesser General Public > > > > + License as published by the Free Software Foundation; either > > > > + version 2.1 of the License, or (at your option) any later version. > > > > + > > > > + The GNU C Library is distributed in the hope that it will be useful, > > > > + but WITHOUT ANY WARRANTY; without even the implied warranty of > > > > + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > > > > + Lesser General Public License for more details. > > > > + > > > > + You should have received a copy of the GNU Lesser General Public > > > > + License along with the GNU C Library; if not, see > > > > + <https://www.gnu.org/licenses/>. */ > > > > + > > > > +#ifndef STRCMP > > > > +# define STRCMP __strcasecmp_l_evex > > > > +#endif > > > > +#define USE_AS_STRCASECMP_L > > > > +#include "strcmp-evex.S" > > > > diff --git a/sysdeps/x86_64/multiarch/strcmp-evex.S b/sysdeps/x86_64/multiarch/strcmp-evex.S > > > > index 56d8c118e4..2a5b3ce037 100644 > > > > --- a/sysdeps/x86_64/multiarch/strcmp-evex.S > > > > +++ b/sysdeps/x86_64/multiarch/strcmp-evex.S > > > > @@ -19,6 +19,9 @@ > > > > #if IS_IN (libc) > > > > > > > > # include <sysdep.h> > > > > +# if defined USE_AS_STRCASECMP_L > > > > +# include "locale-defines.h" > > > > +# endif > > > > > > > > # ifndef STRCMP > > > > # define STRCMP __strcmp_evex > > > > @@ -34,19 +37,29 @@ > > > > # define VMOVA vmovdqa64 > > > > > > > > # ifdef USE_AS_WCSCMP > > > > -# define TESTEQ subl $0xff, > > > > +# ifndef OVERFLOW_STRCMP > > > > +# define OVERFLOW_STRCMP __wcscmp_evex > > > > +# endif > > > > + > > > > +# define TESTEQ subl $0xff, > > > > /* Compare packed dwords. */ > > > > # define VPCMP vpcmpd > > > > # define VPMINU vpminud > > > > # define VPTESTM vptestmd > > > > +# define VPTESTNM vptestnmd > > > > /* 1 dword char == 4 bytes. */ > > > > # define SIZE_OF_CHAR 4 > > > > # else > > > > +# ifndef OVERFLOW_STRCMP > > > > +# define OVERFLOW_STRCMP __strcmp_evex > > > > +# endif > > > > + > > > > # define TESTEQ incl > > > > /* Compare packed bytes. */ > > > > # define VPCMP vpcmpb > > > > # define VPMINU vpminub > > > > # define VPTESTM vptestmb > > > > +# define VPTESTNM vptestnmb > > > > /* 1 byte char == 1 byte. */ > > > > # define SIZE_OF_CHAR 1 > > > > # endif > > > > @@ -73,11 +86,16 @@ > > > > # define VEC_OFFSET (-VEC_SIZE) > > > > # endif > > > > > > > > -# define XMMZERO xmm16 > > > > # define XMM0 xmm17 > > > > # define XMM1 xmm18 > > > > > > > > -# define YMMZERO ymm16 > > > > +# define XMM10 xmm27 > > > > +# define XMM11 xmm28 > > > > +# define XMM12 xmm29 > > > > +# define XMM13 xmm30 > > > > +# define XMM14 xmm31 > > > > + > > > > + > > > > # define YMM0 ymm17 > > > > # define YMM1 ymm18 > > > > # define YMM2 ymm19 > > > > @@ -89,6 +107,87 @@ > > > > # define YMM8 ymm25 > > > > # define YMM9 ymm26 > > > > # define YMM10 ymm27 > > > > +# define YMM11 ymm28 > > > > +# define YMM12 ymm29 > > > > +# define YMM13 ymm30 > > > > +# define YMM14 ymm31 > > > > + > > > > +# ifdef USE_AS_STRCASECMP_L > > > > +# define BYTE_LOOP_REG OFFSET_REG > > > > +# else > > > > +# define BYTE_LOOP_REG ecx > > > > +# endif > > > > + > > > > +# ifdef USE_AS_STRCASECMP_L > > > > +# ifdef USE_AS_STRNCMP > > > > +# define STRCASECMP __strncasecmp_evex > > > > +# define LOCALE_REG rcx > > > > +# define LOCALE_REG_LP RCX_LP > > > > +# define STRCASECMP_NONASCII __strncasecmp_l_nonascii > > > > +# else > > > > +# define STRCASECMP __strcasecmp_evex > > > > +# define LOCALE_REG rdx > > > > +# define LOCALE_REG_LP RDX_LP > > > > +# define STRCASECMP_NONASCII __strcasecmp_l_nonascii > > > > +# endif > > > > +# endif > > > > + > > > > +# define LCASE_MIN_YMM %YMM12 > > > > +# define LCASE_MAX_YMM %YMM13 > > > > +# define CASE_ADD_YMM %YMM14 > > > > + > > > > +# define LCASE_MIN_XMM %XMM12 > > > > +# define LCASE_MAX_XMM %XMM13 > > > > +# define CASE_ADD_XMM %XMM14 > > > > + > > > > + /* NB: wcsncmp uses r11 but strcasecmp is never used in > > > > + conjunction with wcscmp. */ > > > > +# define TOLOWER_BASE %r11 > > > > + > > > > +# ifdef USE_AS_STRCASECMP_L > > > > +# define _REG(x, y) x ## y > > > > +# define REG(x, y) _REG(x, y) > > > > +# define TOLOWER(reg1, reg2, ext) \ > > > > + vpsubb REG(LCASE_MIN_, ext), reg1, REG(%ext, 10); \ > > > > + vpsubb REG(LCASE_MIN_, ext), reg2, REG(%ext, 11); \ > > > > + vpcmpub $1, REG(LCASE_MAX_, ext), REG(%ext, 10), %k5; \ > > > > + vpcmpub $1, REG(LCASE_MAX_, ext), REG(%ext, 11), %k6; \ > > > > + vpaddb reg1, REG(CASE_ADD_, ext), reg1{%k5}; \ > > > > + vpaddb reg2, REG(CASE_ADD_, ext), reg2{%k6} > > > > + > > > > +# define TOLOWER_gpr(src, dst) movl (TOLOWER_BASE, src, 4), dst > > > > +# define TOLOWER_YMM(...) TOLOWER(__VA_ARGS__, YMM) > > > > +# define TOLOWER_XMM(...) TOLOWER(__VA_ARGS__, XMM) > > > > + > > > > +# define CMP_R1_R2(s1_reg, s2_reg, reg_out, ext) \ > > > > + TOLOWER (s1_reg, s2_reg, ext); \ > > > > + VPCMP $0, s1_reg, s2_reg, reg_out > > > > + > > > > +# define CMP_R1_S2(s1_reg, s2_mem, s2_reg, reg_out, ext) \ > > > > + VMOVU s2_mem, s2_reg; \ > > > > + CMP_R1_R2(s1_reg, s2_reg, reg_out, ext) > > > > + > > > > +# define CMP_R1_R2_YMM(...) CMP_R1_R2(__VA_ARGS__, YMM) > > > > +# define CMP_R1_R2_XMM(...) CMP_R1_R2(__VA_ARGS__, XMM) > > > > + > > > > +# define CMP_R1_S2_YMM(...) CMP_R1_S2(__VA_ARGS__, YMM) > > > > +# define CMP_R1_S2_XMM(...) CMP_R1_S2(__VA_ARGS__, XMM) > > > > + > > > > +# else > > > > +# define TOLOWER_gpr(...) > > > > +# define TOLOWER_YMM(...) > > > > +# define TOLOWER_XMM(...) > > > > + > > > > +# define CMP_R1_R2_YMM(s1_reg, s2_reg, reg_out) \ > > > > + VPCMP $0, s2_reg, s1_reg, reg_out > > > > + > > > > +# define CMP_R1_R2_XMM(...) CMP_R1_R2_YMM(__VA_ARGS__) > > > > + > > > > +# define CMP_R1_S2_YMM(s1_reg, s2_mem, unused, reg_out) \ > > > > + VPCMP $0, s2_mem, s1_reg, reg_out > > > > + > > > > +# define CMP_R1_S2_XMM(...) CMP_R1_S2_YMM(__VA_ARGS__) > > > > +# endif > > > > > > > > /* Warning! > > > > wcscmp/wcsncmp have to use SIGNED comparison for elements. > > > > @@ -112,8 +211,45 @@ > > > > returned. */ > > > > > > > > .section .text.evex, "ax", @progbits > > > > -ENTRY(STRCMP) > > > > + .align 16 > > > > + .type STRCMP, @function > > > > + .globl STRCMP > > > > + .hidden STRCMP > > > > + > > > > +# ifdef USE_AS_STRCASECMP_L > > > > +ENTRY (STRCASECMP) > > > > + movq __libc_tsd_LOCALE@gottpoff(%rip), %rax > > > > + mov %fs:(%rax), %LOCALE_REG_LP > > > > + > > > > + /* Either 1 or 5 bytes (dependeing if CET is enabled). */ > > > > + .p2align 4 > > > > +END (STRCASECMP) > > > > + /* FALLTHROUGH to strcasecmp/strncasecmp_l. */ > > > > +# endif > > > > + > > > > + .p2align 4 > > > > +STRCMP: > > > > + cfi_startproc > > > > + _CET_ENDBR > > > > + CALL_MCOUNT > > > > + > > > > +# if defined USE_AS_STRCASECMP_L > > > > + /* We have to fall back on the C implementation for locales with > > > > + encodings not matching ASCII for single bytes. */ > > > > +# if LOCALE_T___LOCALES != 0 || LC_CTYPE != 0 > > > > + mov LOCALE_T___LOCALES + LC_CTYPE * LP_SIZE(%LOCALE_REG), %RAX_LP > > > > +# else > > > > + mov (%LOCALE_REG), %RAX_LP > > > > +# endif > > > > + testl $1, LOCALE_DATA_VALUES + _NL_CTYPE_NONASCII_CASE * SIZEOF_VALUES(%rax) > > > > + jne STRCASECMP_NONASCII > > > > + leaq _nl_C_LC_CTYPE_tolower + 128 * 4(%rip), TOLOWER_BASE > > > > +# endif > > > > + > > > > # ifdef USE_AS_STRNCMP > > > > + /* Don't overwrite LOCALE_REG (rcx) until we have pass > > > > + L(one_or_less). Otherwise we might use the wrong locale in > > > > + the OVERFLOW_STRCMP (strcasecmp_l). */ > > > > # ifdef __ILP32__ > > > > /* Clear the upper 32 bits. */ > > > > movl %edx, %edx > > > > @@ -125,6 +261,32 @@ ENTRY(STRCMP) > > > > actually bound the buffer. */ > > > > jle L(one_or_less) > > > > # endif > > > > + > > > > +# if defined USE_AS_STRCASECMP_L > > > > + .section .rodata.cst32, "aM", @progbits, 32 > > > > + .align 32 > > > > +L(lcase_min): > > > > + .quad 0x4141414141414141 > > > > + .quad 0x4141414141414141 > > > > + .quad 0x4141414141414141 > > > > + .quad 0x4141414141414141 > > > > +L(lcase_max): > > > > + .quad 0x1a1a1a1a1a1a1a1a > > > > + .quad 0x1a1a1a1a1a1a1a1a > > > > + .quad 0x1a1a1a1a1a1a1a1a > > > > + .quad 0x1a1a1a1a1a1a1a1a > > > > +L(case_add): > > > > + .quad 0x2020202020202020 > > > > + .quad 0x2020202020202020 > > > > + .quad 0x2020202020202020 > > > > + .quad 0x2020202020202020 > > > > + .previous > > > > + > > > > + vmovdqa64 L(lcase_min)(%rip), LCASE_MIN_YMM > > > > + vmovdqa64 L(lcase_max)(%rip), LCASE_MAX_YMM > > > > + vmovdqa64 L(case_add)(%rip), CASE_ADD_YMM > > > > +# endif > > > > + > > > > movl %edi, %eax > > > > orl %esi, %eax > > > > /* Shift out the bits irrelivant to page boundary ([63:12]). */ > > > > @@ -139,7 +301,7 @@ L(no_page_cross): > > > > VPTESTM %YMM0, %YMM0, %k2 > > > > /* Each bit cleared in K1 represents a mismatch or a null CHAR > > > > in YMM0 and 32 bytes at (%rsi). */ > > > > - VPCMP $0, (%rsi), %YMM0, %k1{%k2} > > > > + CMP_R1_S2_YMM (%YMM0, (%rsi), %YMM1, %k1){%k2} > > > > kmovd %k1, %ecx > > > > # ifdef USE_AS_STRNCMP > > > > cmpq $CHAR_PER_VEC, %rdx > > > > @@ -169,6 +331,8 @@ L(return_vec_0): > > > > # else > > > > movzbl (%rdi, %rcx), %eax > > > > movzbl (%rsi, %rcx), %ecx > > > > + TOLOWER_gpr (%rax, %eax) > > > > + TOLOWER_gpr (%rcx, %ecx) > > > > subl %ecx, %eax > > > > # endif > > > > L(ret0): > > > > @@ -188,11 +352,15 @@ L(ret_zero): > > > > > > > > .p2align 4,, 5 > > > > L(one_or_less): > > > > +# ifdef USE_AS_STRCASECMP_L > > > > + /* Set locale argument for strcasecmp. */ > > > > + movq %LOCALE_REG, %rdx > > > > +# endif > > > > jb L(ret_zero) > > > > -# ifdef USE_AS_WCSCMP > > > > /* 'nbe' covers the case where length is negative (large > > > > unsigned). */ > > > > - jnbe __wcscmp_evex > > > > + jnbe OVERFLOW_STRCMP > > > > +# ifdef USE_AS_WCSCMP > > > > movl (%rdi), %edx > > > > xorl %eax, %eax > > > > cmpl (%rsi), %edx > > > > @@ -201,11 +369,10 @@ L(one_or_less): > > > > negl %eax > > > > orl $1, %eax > > > > # else > > > > - /* 'nbe' covers the case where length is negative (large > > > > - unsigned). */ > > > > - jnbe __strcmp_evex > > > > movzbl (%rdi), %eax > > > > movzbl (%rsi), %ecx > > > > + TOLOWER_gpr (%rax, %eax) > > > > + TOLOWER_gpr (%rcx, %ecx) > > > > subl %ecx, %eax > > > > # endif > > > > L(ret1): > > > > @@ -233,6 +400,8 @@ L(return_vec_1): > > > > # else > > > > movzbl VEC_SIZE(%rdi, %rcx), %eax > > > > movzbl VEC_SIZE(%rsi, %rcx), %ecx > > > > + TOLOWER_gpr (%rax, %eax) > > > > + TOLOWER_gpr (%rcx, %ecx) > > > > subl %ecx, %eax > > > > # endif > > > > L(ret2): > > > > @@ -270,6 +439,8 @@ L(return_vec_2): > > > > # else > > > > movzbl (VEC_SIZE * 2)(%rdi, %rcx), %eax > > > > movzbl (VEC_SIZE * 2)(%rsi, %rcx), %ecx > > > > + TOLOWER_gpr (%rax, %eax) > > > > + TOLOWER_gpr (%rcx, %ecx) > > > > subl %ecx, %eax > > > > # endif > > > > L(ret3): > > > > @@ -290,6 +461,8 @@ L(return_vec_3): > > > > # else > > > > movzbl (VEC_SIZE * 3)(%rdi, %rcx), %eax > > > > movzbl (VEC_SIZE * 3)(%rsi, %rcx), %ecx > > > > + TOLOWER_gpr (%rax, %eax) > > > > + TOLOWER_gpr (%rcx, %ecx) > > > > subl %ecx, %eax > > > > # endif > > > > L(ret4): > > > > @@ -303,7 +476,7 @@ L(more_3x_vec): > > > > /* Safe to compare 4x vectors. */ > > > > VMOVU (VEC_SIZE)(%rdi), %YMM0 > > > > VPTESTM %YMM0, %YMM0, %k2 > > > > - VPCMP $0, (VEC_SIZE)(%rsi), %YMM0, %k1{%k2} > > > > + CMP_R1_S2_YMM (%YMM0, VEC_SIZE(%rsi), %YMM1, %k1){%k2} > > > > kmovd %k1, %ecx > > > > TESTEQ %ecx > > > > jnz L(return_vec_1) > > > > @@ -315,14 +488,14 @@ L(more_3x_vec): > > > > > > > > VMOVU (VEC_SIZE * 2)(%rdi), %YMM0 > > > > VPTESTM %YMM0, %YMM0, %k2 > > > > - VPCMP $0, (VEC_SIZE * 2)(%rsi), %YMM0, %k1{%k2} > > > > + CMP_R1_S2_YMM (%YMM0, (VEC_SIZE * 2)(%rsi), %YMM1, %k1){%k2} > > > > kmovd %k1, %ecx > > > > TESTEQ %ecx > > > > jnz L(return_vec_2) > > > > > > > > VMOVU (VEC_SIZE * 3)(%rdi), %YMM0 > > > > VPTESTM %YMM0, %YMM0, %k2 > > > > - VPCMP $0, (VEC_SIZE * 3)(%rsi), %YMM0, %k1{%k2} > > > > + CMP_R1_S2_YMM (%YMM0, (VEC_SIZE * 3)(%rsi), %YMM1, %k1){%k2} > > > > kmovd %k1, %ecx > > > > TESTEQ %ecx > > > > jnz L(return_vec_3) > > > > @@ -381,7 +554,6 @@ L(prepare_loop_aligned): > > > > subl %esi, %eax > > > > andl $(PAGE_SIZE - 1), %eax > > > > > > > > - vpxorq %YMMZERO, %YMMZERO, %YMMZERO > > > > > > > > /* Loop 4x comparisons at a time. */ > > > > .p2align 4 > > > > @@ -413,22 +585,35 @@ L(loop_skip_page_cross_check): > > > > /* A zero CHAR in YMM9 means that there is a null CHAR. */ > > > > VPMINU %YMM8, %YMM9, %YMM9 > > > > > > > > - /* Each bit set in K1 represents a non-null CHAR in YMM8. */ > > > > + /* Each bit set in K1 represents a non-null CHAR in YMM9. */ > > > > VPTESTM %YMM9, %YMM9, %k1 > > > > - > > > > +# ifndef USE_AS_STRCASECMP_L > > > > vpxorq (VEC_SIZE * 0)(%rsi), %YMM0, %YMM1 > > > > vpxorq (VEC_SIZE * 1)(%rsi), %YMM2, %YMM3 > > > > vpxorq (VEC_SIZE * 2)(%rsi), %YMM4, %YMM5 > > > > /* Ternary logic to xor (VEC_SIZE * 3)(%rsi) with YMM6 while > > > > oring with YMM1. Result is stored in YMM6. */ > > > > vpternlogd $0xde, (VEC_SIZE * 3)(%rsi), %YMM1, %YMM6 > > > > - > > > > +# else > > > > + VMOVU (VEC_SIZE * 0)(%rsi), %YMM1 > > > > + TOLOWER_YMM (%YMM0, %YMM1) > > > > + VMOVU (VEC_SIZE * 1)(%rsi), %YMM3 > > > > + TOLOWER_YMM (%YMM2, %YMM3) > > > > + VMOVU (VEC_SIZE * 2)(%rsi), %YMM5 > > > > + TOLOWER_YMM (%YMM4, %YMM5) > > > > + VMOVU (VEC_SIZE * 3)(%rsi), %YMM7 > > > > + TOLOWER_YMM (%YMM6, %YMM7) > > > > + vpxorq %YMM0, %YMM1, %YMM1 > > > > + vpxorq %YMM2, %YMM3, %YMM3 > > > > + vpxorq %YMM4, %YMM5, %YMM5 > > > > + vpternlogd $0xde, %YMM7, %YMM1, %YMM6 > > > > +# endif > > > > /* Or together YMM3, YMM5, and YMM6. */ > > > > vpternlogd $0xfe, %YMM3, %YMM5, %YMM6 > > > > > > > > > > > > /* A non-zero CHAR in YMM6 represents a mismatch. */ > > > > - VPCMP $0, %YMMZERO, %YMM6, %k0{%k1} > > > > + VPTESTNM %YMM6, %YMM6, %k0{%k1} > > > > kmovd %k0, %LOOP_REG > > > > > > > > TESTEQ %LOOP_REG > > > > @@ -437,13 +622,13 @@ L(loop_skip_page_cross_check): > > > > > > > > /* Find which VEC has the mismatch of end of string. */ > > > > VPTESTM %YMM0, %YMM0, %k1 > > > > - VPCMP $0, %YMMZERO, %YMM1, %k0{%k1} > > > > + VPTESTNM %YMM1, %YMM1, %k0{%k1} > > > > kmovd %k0, %ecx > > > > TESTEQ %ecx > > > > jnz L(return_vec_0_end) > > > > > > > > VPTESTM %YMM2, %YMM2, %k1 > > > > - VPCMP $0, %YMMZERO, %YMM3, %k0{%k1} > > > > + VPTESTNM %YMM3, %YMM3, %k0{%k1} > > > > kmovd %k0, %ecx > > > > TESTEQ %ecx > > > > jnz L(return_vec_1_end) > > > > @@ -457,7 +642,7 @@ L(return_vec_2_3_end): > > > > # endif > > > > > > > > VPTESTM %YMM4, %YMM4, %k1 > > > > - VPCMP $0, %YMMZERO, %YMM5, %k0{%k1} > > > > + VPTESTNM %YMM5, %YMM5, %k0{%k1} > > > > kmovd %k0, %ecx > > > > TESTEQ %ecx > > > > # if CHAR_PER_VEC <= 16 > > > > @@ -493,6 +678,8 @@ L(return_vec_3_end): > > > > # else > > > > movzbl (VEC_SIZE * 2)(%rdi, %LOOP_REG64), %eax > > > > movzbl (VEC_SIZE * 2)(%rsi, %LOOP_REG64), %ecx > > > > + TOLOWER_gpr (%rax, %eax) > > > > + TOLOWER_gpr (%rcx, %ecx) > > > > subl %ecx, %eax > > > > xorl %r8d, %eax > > > > subl %r8d, %eax > > > > @@ -545,6 +732,8 @@ L(return_vec_0_end): > > > > # else > > > > movzbl (%rdi, %rcx), %eax > > > > movzbl (%rsi, %rcx), %ecx > > > > + TOLOWER_gpr (%rax, %eax) > > > > + TOLOWER_gpr (%rcx, %ecx) > > > > subl %ecx, %eax > > > > /* Flip `eax` if `rdi` and `rsi` where swapped in page cross > > > > logic. Subtract `r8d` after xor for zero case. */ > > > > @@ -569,6 +758,8 @@ L(return_vec_1_end): > > > > # else > > > > movzbl VEC_SIZE(%rdi, %rcx), %eax > > > > movzbl VEC_SIZE(%rsi, %rcx), %ecx > > > > + TOLOWER_gpr (%rax, %eax) > > > > + TOLOWER_gpr (%rcx, %ecx) > > > > subl %ecx, %eax > > > > xorl %r8d, %eax > > > > subl %r8d, %eax > > > > @@ -598,7 +789,7 @@ L(page_cross_during_loop): > > > > > > > > VMOVA (%rdi), %YMM0 > > > > VPTESTM %YMM0, %YMM0, %k2 > > > > - VPCMP $0, (%rsi), %YMM0, %k1{%k2} > > > > + CMP_R1_S2_YMM (%YMM0, (%rsi), %YMM1, %k1){%k2} > > > > kmovd %k1, %ecx > > > > TESTEQ %ecx > > > > jnz L(return_vec_0_end) > > > > @@ -619,8 +810,7 @@ L(less_1x_vec_till_page_cross): > > > > been loaded earlier so must be valid. */ > > > > VMOVU -VEC_SIZE(%rdi, %rax), %YMM0 > > > > VPTESTM %YMM0, %YMM0, %k2 > > > > - VPCMP $0, -VEC_SIZE(%rsi, %rax), %YMM0, %k1{%k2} > > > > - > > > > + CMP_R1_S2_YMM (%YMM0, -VEC_SIZE(%rsi, %rax), %YMM1, %k1){%k2} > > > > /* Mask of potentially valid bits. The lower bits can be out of > > > > range comparisons (but safe regarding page crosses). */ > > > > > > > > @@ -642,6 +832,8 @@ L(less_1x_vec_till_page_cross): > > > > > > > > # ifdef USE_AS_STRNCMP > > > > # ifdef USE_AS_WCSCMP > > > > + /* NB: strcasecmp not used with WCSCMP so this access to r11 is > > > > + safe. */ > > > > movl %eax, %r11d > > > > shrl $2, %r11d > > > > cmpq %r11, %rdx > > > > @@ -679,6 +871,8 @@ L(return_page_cross_cmp_mem): > > > > # else > > > > movzbl VEC_OFFSET(%rdi, %rcx), %eax > > > > movzbl VEC_OFFSET(%rsi, %rcx), %ecx > > > > + TOLOWER_gpr (%rax, %eax) > > > > + TOLOWER_gpr (%rcx, %ecx) > > > > subl %ecx, %eax > > > > xorl %r8d, %eax > > > > subl %r8d, %eax > > > > @@ -709,7 +903,7 @@ L(more_2x_vec_till_page_cross): > > > > > > > > VMOVA VEC_SIZE(%rdi), %YMM0 > > > > VPTESTM %YMM0, %YMM0, %k2 > > > > - VPCMP $0, VEC_SIZE(%rsi), %YMM0, %k1{%k2} > > > > + CMP_R1_S2_YMM (%YMM0, VEC_SIZE(%rsi), %YMM1, %k1){%k2} > > > > kmovd %k1, %ecx > > > > TESTEQ %ecx > > > > jnz L(return_vec_1_end) > > > > @@ -724,14 +918,14 @@ L(more_2x_vec_till_page_cross): > > > > /* Safe to include comparisons from lower bytes. */ > > > > VMOVU -(VEC_SIZE * 2)(%rdi, %rax), %YMM0 > > > > VPTESTM %YMM0, %YMM0, %k2 > > > > - VPCMP $0, -(VEC_SIZE * 2)(%rsi, %rax), %YMM0, %k1{%k2} > > > > + CMP_R1_S2_YMM (%YMM0, -(VEC_SIZE * 2)(%rsi, %rax), %YMM1, %k1){%k2} > > > > kmovd %k1, %ecx > > > > TESTEQ %ecx > > > > jnz L(return_vec_page_cross_0) > > > > > > > > VMOVU -(VEC_SIZE * 1)(%rdi, %rax), %YMM0 > > > > VPTESTM %YMM0, %YMM0, %k2 > > > > - VPCMP $0, -(VEC_SIZE * 1)(%rsi, %rax), %YMM0, %k1{%k2} > > > > + CMP_R1_S2_YMM (%YMM0, -(VEC_SIZE * 1)(%rsi, %rax), %YMM1, %k1){%k2} > > > > kmovd %k1, %ecx > > > > TESTEQ %ecx > > > > jnz L(return_vec_page_cross_1) > > > > @@ -740,6 +934,8 @@ L(more_2x_vec_till_page_cross): > > > > /* Must check length here as length might proclude reading next > > > > page. */ > > > > # ifdef USE_AS_WCSCMP > > > > + /* NB: strcasecmp not used with WCSCMP so this access to r11 is > > > > + safe. */ > > > > movl %eax, %r11d > > > > shrl $2, %r11d > > > > cmpq %r11, %rdx > > > > @@ -754,12 +950,19 @@ L(more_2x_vec_till_page_cross): > > > > VMOVA (VEC_SIZE * 3)(%rdi), %YMM6 > > > > VPMINU %YMM4, %YMM6, %YMM9 > > > > VPTESTM %YMM9, %YMM9, %k1 > > > > - > > > > +# ifndef USE_AS_STRCASECMP_L > > > > vpxorq (VEC_SIZE * 2)(%rsi), %YMM4, %YMM5 > > > > /* YMM6 = YMM5 | ((VEC_SIZE * 3)(%rsi) ^ YMM6). */ > > > > vpternlogd $0xde, (VEC_SIZE * 3)(%rsi), %YMM5, %YMM6 > > > > - > > > > - VPCMP $0, %YMMZERO, %YMM6, %k0{%k1} > > > > +# else > > > > + VMOVU (VEC_SIZE * 2)(%rsi), %YMM5 > > > > + TOLOWER_YMM (%YMM4, %YMM5) > > > > + VMOVU (VEC_SIZE * 3)(%rsi), %YMM7 > > > > + TOLOWER_YMM (%YMM6, %YMM7) > > > > + vpxorq %YMM4, %YMM5, %YMM5 > > > > + vpternlogd $0xde, %YMM7, %YMM5, %YMM6 > > > > +# endif > > > > + VPTESTNM %YMM6, %YMM6, %k0{%k1} > > > > kmovd %k0, %LOOP_REG > > > > TESTEQ %LOOP_REG > > > > jnz L(return_vec_2_3_end) > > > > @@ -815,6 +1018,8 @@ L(return_vec_page_cross_1): > > > > # else > > > > movzbl VEC_OFFSET(%rdi, %rcx), %eax > > > > movzbl VEC_OFFSET(%rsi, %rcx), %ecx > > > > + TOLOWER_gpr (%rax, %eax) > > > > + TOLOWER_gpr (%rcx, %ecx) > > > > subl %ecx, %eax > > > > xorl %r8d, %eax > > > > subl %r8d, %eax > > > > @@ -871,7 +1076,7 @@ L(page_cross): > > > > L(page_cross_loop): > > > > VMOVU (%rdi, %OFFSET_REG64, SIZE_OF_CHAR), %YMM0 > > > > VPTESTM %YMM0, %YMM0, %k2 > > > > - VPCMP $0, (%rsi, %OFFSET_REG64, SIZE_OF_CHAR), %YMM0, %k1{%k2} > > > > + CMP_R1_S2_YMM (%YMM0, (%rsi, %OFFSET_REG64, SIZE_OF_CHAR), %YMM1, %k1){%k2} > > > > kmovd %k1, %ecx > > > > TESTEQ %ecx > > > > jnz L(check_ret_vec_page_cross) > > > > @@ -895,7 +1100,7 @@ L(page_cross_loop): > > > > */ > > > > VMOVU (%rdi, %OFFSET_REG64, SIZE_OF_CHAR), %YMM0 > > > > VPTESTM %YMM0, %YMM0, %k2 > > > > - VPCMP $0, (%rsi, %OFFSET_REG64, SIZE_OF_CHAR), %YMM0, %k1{%k2} > > > > + CMP_R1_S2_YMM (%YMM0, (%rsi, %OFFSET_REG64, SIZE_OF_CHAR), %YMM1, %k1){%k2} > > > > > > > > kmovd %k1, %ecx > > > > # ifdef USE_AS_STRNCMP > > > > @@ -930,6 +1135,8 @@ L(ret_vec_page_cross_cont): > > > > # else > > > > movzbl (%rdi, %rcx, SIZE_OF_CHAR), %eax > > > > movzbl (%rsi, %rcx, SIZE_OF_CHAR), %ecx > > > > + TOLOWER_gpr (%rax, %eax) > > > > + TOLOWER_gpr (%rcx, %ecx) > > > > subl %ecx, %eax > > > > xorl %r8d, %eax > > > > subl %r8d, %eax > > > > @@ -989,7 +1196,7 @@ L(less_1x_vec_till_page): > > > > /* Use 16 byte comparison. */ > > > > vmovdqu (%rdi), %xmm0 > > > > VPTESTM %xmm0, %xmm0, %k2 > > > > - VPCMP $0, (%rsi), %xmm0, %k1{%k2} > > > > + CMP_R1_S2_XMM (%xmm0, (%rsi), %xmm1, %k1){%k2} > > > > kmovd %k1, %ecx > > > > # ifdef USE_AS_WCSCMP > > > > subl $0xf, %ecx > > > > @@ -1009,7 +1216,7 @@ L(less_1x_vec_till_page): > > > > # endif > > > > vmovdqu (%rdi, %OFFSET_REG64, SIZE_OF_CHAR), %xmm0 > > > > VPTESTM %xmm0, %xmm0, %k2 > > > > - VPCMP $0, (%rsi, %OFFSET_REG64, SIZE_OF_CHAR), %xmm0, %k1{%k2} > > > > + CMP_R1_S2_XMM (%xmm0, (%rsi, %OFFSET_REG64, SIZE_OF_CHAR), %xmm1, %k1){%k2} > > > > kmovd %k1, %ecx > > > > # ifdef USE_AS_WCSCMP > > > > subl $0xf, %ecx > > > > @@ -1048,7 +1255,7 @@ L(less_16_till_page): > > > > vmovq (%rdi), %xmm0 > > > > vmovq (%rsi), %xmm1 > > > > VPTESTM %xmm0, %xmm0, %k2 > > > > - VPCMP $0, %xmm1, %xmm0, %k1{%k2} > > > > + CMP_R1_R2_XMM (%xmm0, %xmm1, %k1){%k2} > > > > kmovd %k1, %ecx > > > > # ifdef USE_AS_WCSCMP > > > > subl $0x3, %ecx > > > > @@ -1068,7 +1275,7 @@ L(less_16_till_page): > > > > vmovq (%rdi, %OFFSET_REG64, SIZE_OF_CHAR), %xmm0 > > > > vmovq (%rsi, %OFFSET_REG64, SIZE_OF_CHAR), %xmm1 > > > > VPTESTM %xmm0, %xmm0, %k2 > > > > - VPCMP $0, %xmm1, %xmm0, %k1{%k2} > > > > + CMP_R1_R2_XMM (%xmm0, %xmm1, %k1){%k2} > > > > kmovd %k1, %ecx > > > > # ifdef USE_AS_WCSCMP > > > > subl $0x3, %ecx > > > > @@ -1128,7 +1335,7 @@ L(ret_less_8_wcs): > > > > vmovd (%rdi), %xmm0 > > > > vmovd (%rsi), %xmm1 > > > > VPTESTM %xmm0, %xmm0, %k2 > > > > - VPCMP $0, %xmm1, %xmm0, %k1{%k2} > > > > + CMP_R1_R2_XMM (%xmm0, %xmm1, %k1){%k2} > > > > kmovd %k1, %ecx > > > > subl $0xf, %ecx > > > > jnz L(check_ret_vec_page_cross) > > > > @@ -1143,7 +1350,7 @@ L(ret_less_8_wcs): > > > > vmovd (%rdi, %OFFSET_REG64, SIZE_OF_CHAR), %xmm0 > > > > vmovd (%rsi, %OFFSET_REG64, SIZE_OF_CHAR), %xmm1 > > > > VPTESTM %xmm0, %xmm0, %k2 > > > > - VPCMP $0, %xmm1, %xmm0, %k1{%k2} > > > > + CMP_R1_R2_XMM (%xmm0, %xmm1, %k1){%k2} > > > > kmovd %k1, %ecx > > > > subl $0xf, %ecx > > > > jnz L(check_ret_vec_page_cross) > > > > @@ -1176,7 +1383,9 @@ L(less_4_till_page): > > > > L(less_4_loop): > > > > movzbl (%rdi), %eax > > > > movzbl (%rsi, %rdi), %ecx > > > > - subl %ecx, %eax > > > > + TOLOWER_gpr (%rax, %eax) > > > > + TOLOWER_gpr (%rcx, %BYTE_LOOP_REG) > > > > + subl %BYTE_LOOP_REG, %eax > > > > jnz L(ret_less_4_loop) > > > > testl %ecx, %ecx > > > > jz L(ret_zero_4_loop) > > > > @@ -1203,5 +1412,6 @@ L(ret_less_4_loop): > > > > subl %r8d, %eax > > > > ret > > > > # endif > > > > -END(STRCMP) > > > > + cfi_endproc > > > > + .size STRCMP, .-STRCMP > > > > #endif > > > > diff --git a/sysdeps/x86_64/multiarch/strncase_l-evex.S b/sysdeps/x86_64/multiarch/strncase_l-evex.S > > > > new file mode 100644 > > > > index 0000000000..8a5af3695c > > > > --- /dev/null > > > > +++ b/sysdeps/x86_64/multiarch/strncase_l-evex.S > > > > @@ -0,0 +1,25 @@ > > > > +/* strncasecmp_l optimized with EVEX. > > > > + Copyright (C) 2017-2022 Free Software Foundation, Inc. > > > > + This file is part of the GNU C Library. > > > > + > > > > + The GNU C Library is free software; you can redistribute it and/or > > > > + modify it under the terms of the GNU Lesser General Public > > > > + License as published by the Free Software Foundation; either > > > > + version 2.1 of the License, or (at your option) any later version. > > > > + > > > > + The GNU C Library is distributed in the hope that it will be useful, > > > > + but WITHOUT ANY WARRANTY; without even the implied warranty of > > > > + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > > > > + Lesser General Public License for more details. > > > > + > > > > + You should have received a copy of the GNU Lesser General Public > > > > + License along with the GNU C Library; if not, see > > > > + <https://www.gnu.org/licenses/>. */ > > > > + > > > > +#ifndef STRCMP > > > > +# define STRCMP __strncasecmp_l_evex > > > > +#endif > > > > +#define OVERFLOW_STRCMP __strcasecmp_l_evex > > > > +#define USE_AS_STRCASECMP_L > > > > +#define USE_AS_STRNCMP > > > > +#include "strcmp-evex.S" > > > > -- > > > > 2.25.1 > > > > > > > > > > LGTM. > > > > > > Reviewed-by: H.J. Lu <hjl.tools@gmail.com> > > > > Thanks, pushed the patchset. > > > > > > Thanks. > > > > > > -- > > > H.J. > > I would like to backport this patch to release branches. > Any comments or objections? > > Conflict resolution patch attached. > > --Sunil Please ignore, this patch doesn't have any conflict. --Sunil --Sunil ^ permalink raw reply [flat|nested] 3+ messages in thread
[parent not found: <CAMe9rOqZ8h1dOUctD6jF3OmUv9TXjyA7_ser3p_6157myHq1Og@mail.gmail.com>]
* Re: [PATCH v4 21/23] x86: Add AVX2 optimized str{n}casecmp [not found] ` <CAMe9rOqZ8h1dOUctD6jF3OmUv9TXjyA7_ser3p_6157myHq1Og@mail.gmail.com> @ 2022-05-12 19:52 ` Sunil Pandey 0 siblings, 0 replies; 3+ messages in thread From: Sunil Pandey @ 2022-05-12 19:52 UTC (permalink / raw) To: H.J. Lu, Libc-stable Mailing List; +Cc: Noah Goldstein, GNU C Library [-- Attachment #1: Type: text/plain, Size: 31021 bytes --] On Fri, Mar 25, 2022 at 11:15 AM H.J. Lu via Libc-alpha <libc-alpha@sourceware.org> wrote: > > On Thu, Mar 24, 2022 at 4:56 PM Noah Goldstein <goldstein.w.n@gmail.com> wrote: > > > > geometric_mean(N=40) of all benchmarks AVX2 / SSE42: .702 > > > > All string/memory tests pass. > > Reviewed-by: H.J. Lu <hjl.tools@gmail.com> > > --- > > sysdeps/x86_64/multiarch/Makefile | 4 + > > sysdeps/x86_64/multiarch/ifunc-impl-list.c | 28 +++ > > sysdeps/x86_64/multiarch/ifunc-strcasecmp.h | 12 + > > .../x86_64/multiarch/strcasecmp_l-avx2-rtm.S | 15 ++ > > sysdeps/x86_64/multiarch/strcasecmp_l-avx2.S | 23 ++ > > sysdeps/x86_64/multiarch/strcmp-avx2.S | 237 +++++++++++++++--- > > .../x86_64/multiarch/strncase_l-avx2-rtm.S | 16 ++ > > sysdeps/x86_64/multiarch/strncase_l-avx2.S | 27 ++ > > 8 files changed, 331 insertions(+), 31 deletions(-) > > create mode 100644 sysdeps/x86_64/multiarch/strcasecmp_l-avx2-rtm.S > > create mode 100644 sysdeps/x86_64/multiarch/strcasecmp_l-avx2.S > > create mode 100644 sysdeps/x86_64/multiarch/strncase_l-avx2-rtm.S > > create mode 100644 sysdeps/x86_64/multiarch/strncase_l-avx2.S > > > > diff --git a/sysdeps/x86_64/multiarch/Makefile b/sysdeps/x86_64/multiarch/Makefile > > index e7b413edad..06e1848823 100644 > > --- a/sysdeps/x86_64/multiarch/Makefile > > +++ b/sysdeps/x86_64/multiarch/Makefile > > @@ -55,6 +55,8 @@ sysdep_routines += \ > > stpncpy-sse2-unaligned \ > > stpncpy-ssse3 \ > > strcasecmp_l-avx \ > > + strcasecmp_l-avx2 \ > > + strcasecmp_l-avx2-rtm \ > > strcasecmp_l-sse2 \ > > strcasecmp_l-sse4_2 \ > > strcasecmp_l-ssse3 \ > > @@ -93,6 +95,8 @@ sysdep_routines += \ > > strlen-evex \ > > strlen-sse2 \ > > strncase_l-avx \ > > + strncase_l-avx2 \ > > + strncase_l-avx2-rtm \ > > strncase_l-sse2 \ > > strncase_l-sse4_2 \ > > strncase_l-ssse3 \ > > diff --git a/sysdeps/x86_64/multiarch/ifunc-impl-list.c b/sysdeps/x86_64/multiarch/ifunc-impl-list.c > > index a594f4176e..3c556d07ac 100644 > > --- a/sysdeps/x86_64/multiarch/ifunc-impl-list.c > > +++ b/sysdeps/x86_64/multiarch/ifunc-impl-list.c > > @@ -436,6 +436,13 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, > > > > /* Support sysdeps/x86_64/multiarch/strcasecmp_l.c. */ > > IFUNC_IMPL (i, name, strcasecmp, > > + IFUNC_IMPL_ADD (array, i, strcasecmp, > > + CPU_FEATURE_USABLE (AVX2), > > + __strcasecmp_avx2) > > + IFUNC_IMPL_ADD (array, i, strcasecmp, > > + (CPU_FEATURE_USABLE (AVX2) > > + && CPU_FEATURE_USABLE (RTM)), > > + __strcasecmp_avx2_rtm) > > IFUNC_IMPL_ADD (array, i, strcasecmp, > > CPU_FEATURE_USABLE (AVX), > > __strcasecmp_avx) > > @@ -449,6 +456,13 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, > > > > /* Support sysdeps/x86_64/multiarch/strcasecmp_l.c. */ > > IFUNC_IMPL (i, name, strcasecmp_l, > > + IFUNC_IMPL_ADD (array, i, strcasecmp, > > + CPU_FEATURE_USABLE (AVX2), > > + __strcasecmp_l_avx2) > > + IFUNC_IMPL_ADD (array, i, strcasecmp, > > + (CPU_FEATURE_USABLE (AVX2) > > + && CPU_FEATURE_USABLE (RTM)), > > + __strcasecmp_l_avx2_rtm) > > IFUNC_IMPL_ADD (array, i, strcasecmp_l, > > CPU_FEATURE_USABLE (AVX), > > __strcasecmp_l_avx) > > @@ -576,6 +590,13 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, > > > > /* Support sysdeps/x86_64/multiarch/strncase_l.c. */ > > IFUNC_IMPL (i, name, strncasecmp, > > + IFUNC_IMPL_ADD (array, i, strncasecmp, > > + CPU_FEATURE_USABLE (AVX2), > > + __strncasecmp_avx2) > > + IFUNC_IMPL_ADD (array, i, strncasecmp, > > + (CPU_FEATURE_USABLE (AVX2) > > + && CPU_FEATURE_USABLE (RTM)), > > + __strncasecmp_avx2_rtm) > > IFUNC_IMPL_ADD (array, i, strncasecmp, > > CPU_FEATURE_USABLE (AVX), > > __strncasecmp_avx) > > @@ -590,6 +611,13 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, > > > > /* Support sysdeps/x86_64/multiarch/strncase_l.c. */ > > IFUNC_IMPL (i, name, strncasecmp_l, > > + IFUNC_IMPL_ADD (array, i, strncasecmp, > > + CPU_FEATURE_USABLE (AVX2), > > + __strncasecmp_l_avx2) > > + IFUNC_IMPL_ADD (array, i, strncasecmp, > > + (CPU_FEATURE_USABLE (AVX2) > > + && CPU_FEATURE_USABLE (RTM)), > > + __strncasecmp_l_avx2_rtm) > > IFUNC_IMPL_ADD (array, i, strncasecmp_l, > > CPU_FEATURE_USABLE (AVX), > > __strncasecmp_l_avx) > > diff --git a/sysdeps/x86_64/multiarch/ifunc-strcasecmp.h b/sysdeps/x86_64/multiarch/ifunc-strcasecmp.h > > index 9e3cc61ac0..c4de111fd0 100644 > > --- a/sysdeps/x86_64/multiarch/ifunc-strcasecmp.h > > +++ b/sysdeps/x86_64/multiarch/ifunc-strcasecmp.h > > @@ -23,12 +23,24 @@ extern __typeof (REDIRECT_NAME) OPTIMIZE (sse2) attribute_hidden; > > extern __typeof (REDIRECT_NAME) OPTIMIZE (ssse3) attribute_hidden; > > extern __typeof (REDIRECT_NAME) OPTIMIZE (sse42) attribute_hidden; > > extern __typeof (REDIRECT_NAME) OPTIMIZE (avx) attribute_hidden; > > +extern __typeof (REDIRECT_NAME) OPTIMIZE (avx2) attribute_hidden; > > +extern __typeof (REDIRECT_NAME) OPTIMIZE (avx2_rtm) attribute_hidden; > > > > static inline void * > > IFUNC_SELECTOR (void) > > { > > const struct cpu_features* cpu_features = __get_cpu_features (); > > > > + if (CPU_FEATURE_USABLE_P (cpu_features, AVX2) > > + && CPU_FEATURES_ARCH_P (cpu_features, AVX_Fast_Unaligned_Load)) > > + { > > + if (CPU_FEATURE_USABLE_P (cpu_features, RTM)) > > + return OPTIMIZE (avx2_rtm); > > + > > + if (!CPU_FEATURES_ARCH_P (cpu_features, Prefer_No_VZEROUPPER)) > > + return OPTIMIZE (avx2); > > + } > > + > > if (CPU_FEATURE_USABLE_P (cpu_features, AVX)) > > return OPTIMIZE (avx); > > > > diff --git a/sysdeps/x86_64/multiarch/strcasecmp_l-avx2-rtm.S b/sysdeps/x86_64/multiarch/strcasecmp_l-avx2-rtm.S > > new file mode 100644 > > index 0000000000..09957fc3c5 > > --- /dev/null > > +++ b/sysdeps/x86_64/multiarch/strcasecmp_l-avx2-rtm.S > > @@ -0,0 +1,15 @@ > > +#ifndef STRCMP > > +# define STRCMP __strcasecmp_l_avx2_rtm > > +#endif > > + > > +#define _GLABEL(x) x ## _rtm > > +#define GLABEL(x) _GLABEL(x) > > + > > +#define ZERO_UPPER_VEC_REGISTERS_RETURN \ > > + ZERO_UPPER_VEC_REGISTERS_RETURN_XTEST > > + > > +#define VZEROUPPER_RETURN jmp L(return_vzeroupper) > > + > > +#define SECTION(p) p##.avx.rtm > > + > > +#include "strcasecmp_l-avx2.S" > > diff --git a/sysdeps/x86_64/multiarch/strcasecmp_l-avx2.S b/sysdeps/x86_64/multiarch/strcasecmp_l-avx2.S > > new file mode 100644 > > index 0000000000..e2762f2a22 > > --- /dev/null > > +++ b/sysdeps/x86_64/multiarch/strcasecmp_l-avx2.S > > @@ -0,0 +1,23 @@ > > +/* strcasecmp_l optimized with AVX2. > > + Copyright (C) 2017-2022 Free Software Foundation, Inc. > > + This file is part of the GNU C Library. > > + > > + The GNU C Library is free software; you can redistribute it and/or > > + modify it under the terms of the GNU Lesser General Public > > + License as published by the Free Software Foundation; either > > + version 2.1 of the License, or (at your option) any later version. > > + > > + The GNU C Library is distributed in the hope that it will be useful, > > + but WITHOUT ANY WARRANTY; without even the implied warranty of > > + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > > + Lesser General Public License for more details. > > + > > + You should have received a copy of the GNU Lesser General Public > > + License along with the GNU C Library; if not, see > > + <https://www.gnu.org/licenses/>. */ > > + > > +#ifndef STRCMP > > +# define STRCMP __strcasecmp_l_avx2 > > +#endif > > +#define USE_AS_STRCASECMP_L > > +#include "strcmp-avx2.S" > > diff --git a/sysdeps/x86_64/multiarch/strcmp-avx2.S b/sysdeps/x86_64/multiarch/strcmp-avx2.S > > index 86a86b68e3..8da09bd86d 100644 > > --- a/sysdeps/x86_64/multiarch/strcmp-avx2.S > > +++ b/sysdeps/x86_64/multiarch/strcmp-avx2.S > > @@ -20,6 +20,10 @@ > > > > # include <sysdep.h> > > > > +# if defined USE_AS_STRCASECMP_L > > +# include "locale-defines.h" > > +# endif > > + > > # ifndef STRCMP > > # define STRCMP __strcmp_avx2 > > # endif > > @@ -74,13 +78,88 @@ > > # define VEC_OFFSET (-VEC_SIZE) > > # endif > > > > +# ifdef USE_AS_STRCASECMP_L > > +# define BYTE_LOOP_REG OFFSET_REG > > +# else > > +# define BYTE_LOOP_REG ecx > > +# endif > > + > > +# ifdef USE_AS_STRCASECMP_L > > +# ifdef USE_AS_STRNCMP > > +# define STRCASECMP __strncasecmp_avx2 > > +# define LOCALE_REG rcx > > +# define LOCALE_REG_LP RCX_LP > > +# define STRCASECMP_NONASCII __strncasecmp_l_nonascii > > +# else > > +# define STRCASECMP __strcasecmp_avx2 > > +# define LOCALE_REG rdx > > +# define LOCALE_REG_LP RDX_LP > > +# define STRCASECMP_NONASCII __strcasecmp_l_nonascii > > +# endif > > +# endif > > + > > # define xmmZERO xmm15 > > # define ymmZERO ymm15 > > > > +# define LCASE_MIN_ymm %ymm10 > > +# define LCASE_MAX_ymm %ymm11 > > +# define CASE_ADD_ymm %ymm12 > > + > > +# define LCASE_MIN_xmm %xmm10 > > +# define LCASE_MAX_xmm %xmm11 > > +# define CASE_ADD_xmm %xmm12 > > + > > + /* r11 is never use elsewhere so this is safe to maintain. */ > > +# define TOLOWER_BASE %r11 > > + > > # ifndef SECTION > > # define SECTION(p) p##.avx > > # endif > > > > +# ifdef USE_AS_STRCASECMP_L > > +# define REG(x, y) x ## y > > +# define TOLOWER(reg1_in, reg1_out, reg2_in, reg2_out, ext) \ > > + vpaddb REG(LCASE_MIN_, ext), reg1_in, REG(%ext, 8); \ > > + vpaddb REG(LCASE_MIN_, ext), reg2_in, REG(%ext, 9); \ > > + vpcmpgtb REG(LCASE_MAX_, ext), REG(%ext, 8), REG(%ext, 8); \ > > + vpcmpgtb REG(LCASE_MAX_, ext), REG(%ext, 9), REG(%ext, 9); \ > > + vpandn REG(CASE_ADD_, ext), REG(%ext, 8), REG(%ext, 8); \ > > + vpandn REG(CASE_ADD_, ext), REG(%ext, 9), REG(%ext, 9); \ > > + vpaddb REG(%ext, 8), reg1_in, reg1_out; \ > > + vpaddb REG(%ext, 9), reg2_in, reg2_out > > + > > +# define TOLOWER_gpr(src, dst) movl (TOLOWER_BASE, src, 4), dst > > +# define TOLOWER_ymm(...) TOLOWER(__VA_ARGS__, ymm) > > +# define TOLOWER_xmm(...) TOLOWER(__VA_ARGS__, xmm) > > + > > +# define CMP_R1_R2(s1_reg, s2_reg, scratch_reg, reg_out, ext) \ > > + TOLOWER (s1_reg, scratch_reg, s2_reg, s2_reg, ext); \ > > + VPCMPEQ scratch_reg, s2_reg, reg_out > > + > > +# define CMP_R1_S2(s1_reg, s2_mem, scratch_reg, reg_out, ext) \ > > + VMOVU s2_mem, reg_out; \ > > + CMP_R1_R2(s1_reg, reg_out, scratch_reg, reg_out, ext) > > + > > +# define CMP_R1_R2_ymm(...) CMP_R1_R2(__VA_ARGS__, ymm) > > +# define CMP_R1_R2_xmm(...) CMP_R1_R2(__VA_ARGS__, xmm) > > + > > +# define CMP_R1_S2_ymm(...) CMP_R1_S2(__VA_ARGS__, ymm) > > +# define CMP_R1_S2_xmm(...) CMP_R1_S2(__VA_ARGS__, xmm) > > + > > +# else > > +# define TOLOWER_gpr(...) > > +# define TOLOWER_ymm(...) > > +# define TOLOWER_xmm(...) > > + > > +# define CMP_R1_R2_ymm(s1_reg, s2_reg, scratch_reg, reg_out) \ > > + VPCMPEQ s2_reg, s1_reg, reg_out > > + > > +# define CMP_R1_R2_xmm(...) CMP_R1_R2_ymm(__VA_ARGS__) > > + > > +# define CMP_R1_S2_ymm(...) CMP_R1_R2_ymm(__VA_ARGS__) > > +# define CMP_R1_S2_xmm(...) CMP_R1_R2_xmm(__VA_ARGS__) > > +# endif > > + > > /* Warning! > > wcscmp/wcsncmp have to use SIGNED comparison for elements. > > strcmp/strncmp have to use UNSIGNED comparison for elements. > > @@ -102,8 +181,49 @@ > > returned. */ > > > > .section SECTION(.text), "ax", @progbits > > -ENTRY(STRCMP) > > + .align 16 > > + .type STRCMP, @function > > + .globl STRCMP > > + .hidden STRCMP > > + > > +# ifndef GLABEL > > +# define GLABEL(...) __VA_ARGS__ > > +# endif > > + > > +# ifdef USE_AS_STRCASECMP_L > > +ENTRY (GLABEL(STRCASECMP)) > > + movq __libc_tsd_LOCALE@gottpoff(%rip), %rax > > + mov %fs:(%rax), %LOCALE_REG_LP > > + > > + /* Either 1 or 5 bytes (dependeing if CET is enabled). */ > > + .p2align 4 > > +END (GLABEL(STRCASECMP)) > > + /* FALLTHROUGH to strcasecmp/strncasecmp_l. */ > > +# endif > > + > > + .p2align 4 > > +STRCMP: > > + cfi_startproc > > + _CET_ENDBR > > + CALL_MCOUNT > > + > > +# if defined USE_AS_STRCASECMP_L > > + /* We have to fall back on the C implementation for locales with > > + encodings not matching ASCII for single bytes. */ > > +# if LOCALE_T___LOCALES != 0 || LC_CTYPE != 0 > > + mov LOCALE_T___LOCALES + LC_CTYPE * LP_SIZE(%LOCALE_REG), %RAX_LP > > +# else > > + mov (%LOCALE_REG), %RAX_LP > > +# endif > > + testl $1, LOCALE_DATA_VALUES + _NL_CTYPE_NONASCII_CASE * SIZEOF_VALUES(%rax) > > + jne STRCASECMP_NONASCII > > + leaq _nl_C_LC_CTYPE_tolower + 128 * 4(%rip), TOLOWER_BASE > > +# endif > > + > > # ifdef USE_AS_STRNCMP > > + /* Don't overwrite LOCALE_REG (rcx) until we have pass > > + L(one_or_less). Otherwise we might use the wrong locale in > > + the OVERFLOW_STRCMP (strcasecmp_l). */ > > # ifdef __ILP32__ > > /* Clear the upper 32 bits. */ > > movl %edx, %edx > > @@ -128,6 +248,30 @@ ENTRY(STRCMP) > > # endif > > # endif > > vpxor %xmmZERO, %xmmZERO, %xmmZERO > > +# if defined USE_AS_STRCASECMP_L > > + .section .rodata.cst32, "aM", @progbits, 32 > > + .align 32 > > +L(lcase_min): > > + .quad 0x3f3f3f3f3f3f3f3f > > + .quad 0x3f3f3f3f3f3f3f3f > > + .quad 0x3f3f3f3f3f3f3f3f > > + .quad 0x3f3f3f3f3f3f3f3f > > +L(lcase_max): > > + .quad 0x9999999999999999 > > + .quad 0x9999999999999999 > > + .quad 0x9999999999999999 > > + .quad 0x9999999999999999 > > +L(case_add): > > + .quad 0x2020202020202020 > > + .quad 0x2020202020202020 > > + .quad 0x2020202020202020 > > + .quad 0x2020202020202020 > > + .previous > > + > > + vmovdqa L(lcase_min)(%rip), LCASE_MIN_ymm > > + vmovdqa L(lcase_max)(%rip), LCASE_MAX_ymm > > + vmovdqa L(case_add)(%rip), CASE_ADD_ymm > > +# endif > > movl %edi, %eax > > orl %esi, %eax > > sall $20, %eax > > @@ -138,8 +282,10 @@ ENTRY(STRCMP) > > L(no_page_cross): > > /* Safe to compare 4x vectors. */ > > VMOVU (%rdi), %ymm0 > > - /* 1s where s1 and s2 equal. */ > > - VPCMPEQ (%rsi), %ymm0, %ymm1 > > + /* 1s where s1 and s2 equal. Just VPCMPEQ if its not strcasecmp. > > + Otherwise converts ymm0 and load from rsi to lower. ymm2 is > > + scratch and ymm1 is the return. */ > > + CMP_R1_S2_ymm (%ymm0, (%rsi), %ymm2, %ymm1) > > /* 1s at null CHAR. */ > > VPCMPEQ %ymm0, %ymmZERO, %ymm2 > > /* 1s where s1 and s2 equal AND not null CHAR. */ > > @@ -172,6 +318,8 @@ L(return_vec_0): > > # else > > movzbl (%rdi, %rcx), %eax > > movzbl (%rsi, %rcx), %ecx > > + TOLOWER_gpr (%rax, %eax) > > + TOLOWER_gpr (%rcx, %ecx) > > subl %ecx, %eax > > # endif > > L(ret0): > > @@ -192,6 +340,10 @@ L(ret_zero): > > > > .p2align 4,, 5 > > L(one_or_less): > > +# ifdef USE_AS_STRCASECMP_L > > + /* Set locale argument for strcasecmp. */ > > + movq %LOCALE_REG, %rdx > > +# endif > > jb L(ret_zero) > > /* 'nbe' covers the case where length is negative (large > > unsigned). */ > > @@ -207,6 +359,8 @@ L(one_or_less): > > # else > > movzbl (%rdi), %eax > > movzbl (%rsi), %ecx > > + TOLOWER_gpr (%rax, %eax) > > + TOLOWER_gpr (%rcx, %ecx) > > subl %ecx, %eax > > # endif > > L(ret1): > > @@ -234,6 +388,8 @@ L(return_vec_1): > > # else > > movzbl VEC_SIZE(%rdi, %rcx), %eax > > movzbl VEC_SIZE(%rsi, %rcx), %ecx > > + TOLOWER_gpr (%rax, %eax) > > + TOLOWER_gpr (%rcx, %ecx) > > subl %ecx, %eax > > # endif > > L(ret2): > > @@ -265,6 +421,8 @@ L(return_vec_2): > > # else > > movzbl (VEC_SIZE * 2)(%rdi, %rcx), %eax > > movzbl (VEC_SIZE * 2)(%rsi, %rcx), %ecx > > + TOLOWER_gpr (%rax, %eax) > > + TOLOWER_gpr (%rcx, %ecx) > > subl %ecx, %eax > > # endif > > L(ret3): > > @@ -285,6 +443,8 @@ L(return_vec_3): > > # else > > movzbl (VEC_SIZE * 3)(%rdi, %rcx), %eax > > movzbl (VEC_SIZE * 3)(%rsi, %rcx), %ecx > > + TOLOWER_gpr (%rax, %eax) > > + TOLOWER_gpr (%rcx, %ecx) > > subl %ecx, %eax > > # endif > > L(ret4): > > @@ -295,7 +455,7 @@ L(ret4): > > L(more_3x_vec): > > /* Safe to compare 4x vectors. */ > > VMOVU VEC_SIZE(%rdi), %ymm0 > > - VPCMPEQ VEC_SIZE(%rsi), %ymm0, %ymm1 > > + CMP_R1_S2_ymm (%ymm0, VEC_SIZE(%rsi), %ymm2, %ymm1) > > VPCMPEQ %ymm0, %ymmZERO, %ymm2 > > vpandn %ymm1, %ymm2, %ymm1 > > vpmovmskb %ymm1, %ecx > > @@ -308,7 +468,7 @@ L(more_3x_vec): > > # endif > > > > VMOVU (VEC_SIZE * 2)(%rdi), %ymm0 > > - VPCMPEQ (VEC_SIZE * 2)(%rsi), %ymm0, %ymm1 > > + CMP_R1_S2_ymm (%ymm0, (VEC_SIZE * 2)(%rsi), %ymm2, %ymm1) > > VPCMPEQ %ymm0, %ymmZERO, %ymm2 > > vpandn %ymm1, %ymm2, %ymm1 > > vpmovmskb %ymm1, %ecx > > @@ -316,7 +476,7 @@ L(more_3x_vec): > > jnz L(return_vec_2) > > > > VMOVU (VEC_SIZE * 3)(%rdi), %ymm0 > > - VPCMPEQ (VEC_SIZE * 3)(%rsi), %ymm0, %ymm1 > > + CMP_R1_S2_ymm (%ymm0, (VEC_SIZE * 3)(%rsi), %ymm2, %ymm1) > > VPCMPEQ %ymm0, %ymmZERO, %ymm2 > > vpandn %ymm1, %ymm2, %ymm1 > > vpmovmskb %ymm1, %ecx > > @@ -391,12 +551,10 @@ L(loop_skip_page_cross_check): > > VMOVA (VEC_SIZE * 3)(%rdi), %ymm6 > > > > /* ymm1 all 1s where s1 and s2 equal. All 0s otherwise. */ > > - VPCMPEQ (VEC_SIZE * 0)(%rsi), %ymm0, %ymm1 > > - > > - VPCMPEQ (VEC_SIZE * 1)(%rsi), %ymm2, %ymm3 > > - VPCMPEQ (VEC_SIZE * 2)(%rsi), %ymm4, %ymm5 > > - VPCMPEQ (VEC_SIZE * 3)(%rsi), %ymm6, %ymm7 > > - > > + CMP_R1_S2_ymm (%ymm0, (VEC_SIZE * 0)(%rsi), %ymm3, %ymm1) > > + CMP_R1_S2_ymm (%ymm2, (VEC_SIZE * 1)(%rsi), %ymm5, %ymm3) > > + CMP_R1_S2_ymm (%ymm4, (VEC_SIZE * 2)(%rsi), %ymm7, %ymm5) > > + CMP_R1_S2_ymm (%ymm6, (VEC_SIZE * 3)(%rsi), %ymm13, %ymm7) > > > > /* If any mismatches or null CHAR then 0 CHAR, otherwise non- > > zero. */ > > @@ -465,6 +623,8 @@ L(return_vec_2_3_end): > > # else > > movzbl (VEC_SIZE * 2 - VEC_OFFSET)(%rdi, %LOOP_REG64), %eax > > movzbl (VEC_SIZE * 2 - VEC_OFFSET)(%rsi, %LOOP_REG64), %ecx > > + TOLOWER_gpr (%rax, %eax) > > + TOLOWER_gpr (%rcx, %ecx) > > subl %ecx, %eax > > xorl %r8d, %eax > > subl %r8d, %eax > > @@ -508,6 +668,8 @@ L(return_vec_0_end): > > # else > > movzbl (%rdi, %rcx), %eax > > movzbl (%rsi, %rcx), %ecx > > + TOLOWER_gpr (%rax, %eax) > > + TOLOWER_gpr (%rcx, %ecx) > > subl %ecx, %eax > > xorl %r8d, %eax > > subl %r8d, %eax > > @@ -530,6 +692,8 @@ L(return_vec_1_end): > > # else > > movzbl VEC_SIZE(%rdi, %rcx), %eax > > movzbl VEC_SIZE(%rsi, %rcx), %ecx > > + TOLOWER_gpr (%rax, %eax) > > + TOLOWER_gpr (%rcx, %ecx) > > subl %ecx, %eax > > xorl %r8d, %eax > > subl %r8d, %eax > > @@ -556,6 +720,8 @@ L(return_vec_2_end): > > # else > > movzbl (VEC_SIZE * 2)(%rdi, %rcx), %eax > > movzbl (VEC_SIZE * 2)(%rsi, %rcx), %ecx > > + TOLOWER_gpr (%rax, %eax) > > + TOLOWER_gpr (%rcx, %ecx) > > subl %ecx, %eax > > xorl %r8d, %eax > > subl %r8d, %eax > > @@ -583,7 +749,7 @@ L(page_cross_during_loop): > > jle L(less_1x_vec_till_page_cross) > > > > VMOVA (%rdi), %ymm0 > > - VPCMPEQ (%rsi), %ymm0, %ymm1 > > + CMP_R1_S2_ymm (%ymm0, (%rsi), %ymm2, %ymm1) > > VPCMPEQ %ymm0, %ymmZERO, %ymm2 > > vpandn %ymm1, %ymm2, %ymm1 > > vpmovmskb %ymm1, %ecx > > @@ -605,7 +771,7 @@ L(less_1x_vec_till_page_cross): > > here, it means the previous page (rdi - VEC_SIZE) has already > > been loaded earlier so must be valid. */ > > VMOVU -VEC_SIZE(%rdi, %rax), %ymm0 > > - VPCMPEQ -VEC_SIZE(%rsi, %rax), %ymm0, %ymm1 > > + CMP_R1_S2_ymm (%ymm0, -VEC_SIZE(%rsi, %rax), %ymm2, %ymm1) > > VPCMPEQ %ymm0, %ymmZERO, %ymm2 > > vpandn %ymm1, %ymm2, %ymm1 > > vpmovmskb %ymm1, %ecx > > @@ -647,6 +813,8 @@ L(return_page_cross_cmp_mem): > > # else > > movzbl VEC_OFFSET(%rdi, %rcx), %eax > > movzbl VEC_OFFSET(%rsi, %rcx), %ecx > > + TOLOWER_gpr (%rax, %eax) > > + TOLOWER_gpr (%rcx, %ecx) > > subl %ecx, %eax > > xorl %r8d, %eax > > subl %r8d, %eax > > @@ -673,7 +841,7 @@ L(more_2x_vec_till_page_cross): > > iteration here. */ > > > > VMOVU VEC_SIZE(%rdi), %ymm0 > > - VPCMPEQ VEC_SIZE(%rsi), %ymm0, %ymm1 > > + CMP_R1_S2_ymm (%ymm0, VEC_SIZE(%rsi), %ymm2, %ymm1) > > VPCMPEQ %ymm0, %ymmZERO, %ymm2 > > vpandn %ymm1, %ymm2, %ymm1 > > vpmovmskb %ymm1, %ecx > > @@ -689,7 +857,7 @@ L(more_2x_vec_till_page_cross): > > > > /* Safe to include comparisons from lower bytes. */ > > VMOVU -(VEC_SIZE * 2)(%rdi, %rax), %ymm0 > > - VPCMPEQ -(VEC_SIZE * 2)(%rsi, %rax), %ymm0, %ymm1 > > + CMP_R1_S2_ymm (%ymm0, -(VEC_SIZE * 2)(%rsi, %rax), %ymm2, %ymm1) > > VPCMPEQ %ymm0, %ymmZERO, %ymm2 > > vpandn %ymm1, %ymm2, %ymm1 > > vpmovmskb %ymm1, %ecx > > @@ -697,7 +865,7 @@ L(more_2x_vec_till_page_cross): > > jnz L(return_vec_page_cross_0) > > > > VMOVU -(VEC_SIZE * 1)(%rdi, %rax), %ymm0 > > - VPCMPEQ -(VEC_SIZE * 1)(%rsi, %rax), %ymm0, %ymm1 > > + CMP_R1_S2_ymm (%ymm0, -(VEC_SIZE * 1)(%rsi, %rax), %ymm2, %ymm1) > > VPCMPEQ %ymm0, %ymmZERO, %ymm2 > > vpandn %ymm1, %ymm2, %ymm1 > > vpmovmskb %ymm1, %ecx > > @@ -715,8 +883,8 @@ L(more_2x_vec_till_page_cross): > > VMOVA (VEC_SIZE * 2)(%rdi), %ymm4 > > VMOVA (VEC_SIZE * 3)(%rdi), %ymm6 > > > > - VPCMPEQ (VEC_SIZE * 2)(%rsi), %ymm4, %ymm5 > > - VPCMPEQ (VEC_SIZE * 3)(%rsi), %ymm6, %ymm7 > > + CMP_R1_S2_ymm (%ymm4, (VEC_SIZE * 2)(%rsi), %ymm7, %ymm5) > > + CMP_R1_S2_ymm (%ymm6, (VEC_SIZE * 3)(%rsi), %ymm13, %ymm7) > > vpand %ymm4, %ymm5, %ymm5 > > vpand %ymm6, %ymm7, %ymm7 > > VPMINU %ymm5, %ymm7, %ymm7 > > @@ -767,6 +935,8 @@ L(return_vec_page_cross_1): > > # else > > movzbl VEC_OFFSET(%rdi, %rcx), %eax > > movzbl VEC_OFFSET(%rsi, %rcx), %ecx > > + TOLOWER_gpr (%rax, %eax) > > + TOLOWER_gpr (%rcx, %ecx) > > subl %ecx, %eax > > xorl %r8d, %eax > > subl %r8d, %eax > > @@ -822,7 +992,7 @@ L(page_cross): > > L(page_cross_loop): > > > > VMOVU (%rdi, %OFFSET_REG64), %ymm0 > > - VPCMPEQ (%rsi, %OFFSET_REG64), %ymm0, %ymm1 > > + CMP_R1_S2_ymm (%ymm0, (%rsi, %OFFSET_REG64), %ymm2, %ymm1) > > VPCMPEQ %ymm0, %ymmZERO, %ymm2 > > vpandn %ymm1, %ymm2, %ymm1 > > vpmovmskb %ymm1, %ecx > > @@ -840,11 +1010,11 @@ L(page_cross_loop): > > subl %eax, %OFFSET_REG > > /* OFFSET_REG has distance to page cross - VEC_SIZE. Guranteed > > to not cross page so is safe to load. Since we have already > > - loaded at least 1 VEC from rsi it is also guranteed to be safe. > > - */ > > + loaded at least 1 VEC from rsi it is also guranteed to be > > + safe. */ > > > > VMOVU (%rdi, %OFFSET_REG64), %ymm0 > > - VPCMPEQ (%rsi, %OFFSET_REG64), %ymm0, %ymm1 > > + CMP_R1_S2_ymm (%ymm0, (%rsi, %OFFSET_REG64), %ymm2, %ymm1) > > VPCMPEQ %ymm0, %ymmZERO, %ymm2 > > vpandn %ymm1, %ymm2, %ymm1 > > vpmovmskb %ymm1, %ecx > > @@ -877,6 +1047,8 @@ L(ret_vec_page_cross_cont): > > # else > > movzbl (%rdi, %rcx), %eax > > movzbl (%rsi, %rcx), %ecx > > + TOLOWER_gpr (%rax, %eax) > > + TOLOWER_gpr (%rcx, %ecx) > > subl %ecx, %eax > > xorl %r8d, %eax > > subl %r8d, %eax > > @@ -930,7 +1102,7 @@ L(less_1x_vec_till_page): > > ja L(less_16_till_page) > > > > VMOVU (%rdi), %xmm0 > > - VPCMPEQ (%rsi), %xmm0, %xmm1 > > + CMP_R1_S2_xmm (%xmm0, (%rsi), %xmm2, %xmm1) > > VPCMPEQ %xmm0, %xmmZERO, %xmm2 > > vpandn %xmm1, %xmm2, %xmm1 > > vpmovmskb %ymm1, %ecx > > @@ -948,7 +1120,7 @@ L(less_1x_vec_till_page): > > # endif > > > > VMOVU (%rdi, %OFFSET_REG64), %xmm0 > > - VPCMPEQ (%rsi, %OFFSET_REG64), %xmm0, %xmm1 > > + CMP_R1_S2_xmm (%xmm0, (%rsi, %OFFSET_REG64), %xmm2, %xmm1) > > VPCMPEQ %xmm0, %xmmZERO, %xmm2 > > vpandn %xmm1, %xmm2, %xmm1 > > vpmovmskb %ymm1, %ecx > > @@ -986,7 +1158,7 @@ L(less_16_till_page): > > vmovq (%rdi), %xmm0 > > vmovq (%rsi), %xmm1 > > VPCMPEQ %xmm0, %xmmZERO, %xmm2 > > - VPCMPEQ %xmm1, %xmm0, %xmm1 > > + CMP_R1_R2_xmm (%xmm0, %xmm1, %xmm3, %xmm1) > > vpandn %xmm1, %xmm2, %xmm1 > > vpmovmskb %ymm1, %ecx > > incb %cl > > @@ -1006,7 +1178,7 @@ L(less_16_till_page): > > vmovq (%rdi, %OFFSET_REG64), %xmm0 > > vmovq (%rsi, %OFFSET_REG64), %xmm1 > > VPCMPEQ %xmm0, %xmmZERO, %xmm2 > > - VPCMPEQ %xmm1, %xmm0, %xmm1 > > + CMP_R1_R2_xmm (%xmm0, %xmm1, %xmm3, %xmm1) > > vpandn %xmm1, %xmm2, %xmm1 > > vpmovmskb %ymm1, %ecx > > incb %cl > > @@ -1062,7 +1234,7 @@ L(ret_less_8_wcs): > > vmovd (%rdi), %xmm0 > > vmovd (%rsi), %xmm1 > > VPCMPEQ %xmm0, %xmmZERO, %xmm2 > > - VPCMPEQ %xmm1, %xmm0, %xmm1 > > + CMP_R1_R2_xmm (%xmm0, %xmm1, %xmm3, %xmm1) > > vpandn %xmm1, %xmm2, %xmm1 > > vpmovmskb %ymm1, %ecx > > subl $0xf, %ecx > > @@ -1081,7 +1253,7 @@ L(ret_less_8_wcs): > > vmovd (%rdi, %OFFSET_REG64), %xmm0 > > vmovd (%rsi, %OFFSET_REG64), %xmm1 > > VPCMPEQ %xmm0, %xmmZERO, %xmm2 > > - VPCMPEQ %xmm1, %xmm0, %xmm1 > > + CMP_R1_R2_xmm (%xmm0, %xmm1, %xmm3, %xmm1) > > vpandn %xmm1, %xmm2, %xmm1 > > vpmovmskb %ymm1, %ecx > > subl $0xf, %ecx > > @@ -1115,7 +1287,9 @@ L(less_4_till_page): > > L(less_4_loop): > > movzbl (%rdi), %eax > > movzbl (%rsi, %rdi), %ecx > > - subl %ecx, %eax > > + TOLOWER_gpr (%rax, %eax) > > + TOLOWER_gpr (%rcx, %BYTE_LOOP_REG) > > + subl %BYTE_LOOP_REG, %eax > > jnz L(ret_less_4_loop) > > testl %ecx, %ecx > > jz L(ret_zero_4_loop) > > @@ -1142,5 +1316,6 @@ L(ret_less_4_loop): > > subl %r8d, %eax > > ret > > # endif > > -END(STRCMP) > > + cfi_endproc > > + .size STRCMP, .-STRCMP > > #endif > > diff --git a/sysdeps/x86_64/multiarch/strncase_l-avx2-rtm.S b/sysdeps/x86_64/multiarch/strncase_l-avx2-rtm.S > > new file mode 100644 > > index 0000000000..58c05dcfb8 > > --- /dev/null > > +++ b/sysdeps/x86_64/multiarch/strncase_l-avx2-rtm.S > > @@ -0,0 +1,16 @@ > > +#ifndef STRCMP > > +# define STRCMP __strncasecmp_l_avx2_rtm > > +#endif > > + > > +#define _GLABEL(x) x ## _rtm > > +#define GLABEL(x) _GLABEL(x) > > + > > +#define ZERO_UPPER_VEC_REGISTERS_RETURN \ > > + ZERO_UPPER_VEC_REGISTERS_RETURN_XTEST > > + > > +#define VZEROUPPER_RETURN jmp L(return_vzeroupper) > > + > > +#define SECTION(p) p##.avx.rtm > > +#define OVERFLOW_STRCMP __strcasecmp_l_avx2_rtm > > + > > +#include "strncase_l-avx2.S" > > diff --git a/sysdeps/x86_64/multiarch/strncase_l-avx2.S b/sysdeps/x86_64/multiarch/strncase_l-avx2.S > > new file mode 100644 > > index 0000000000..48c0aa21f8 > > --- /dev/null > > +++ b/sysdeps/x86_64/multiarch/strncase_l-avx2.S > > @@ -0,0 +1,27 @@ > > +/* strncasecmp_l optimized with AVX2. > > + Copyright (C) 2017-2022 Free Software Foundation, Inc. > > + This file is part of the GNU C Library. > > + > > + The GNU C Library is free software; you can redistribute it and/or > > + modify it under the terms of the GNU Lesser General Public > > + License as published by the Free Software Foundation; either > > + version 2.1 of the License, or (at your option) any later version. > > + > > + The GNU C Library is distributed in the hope that it will be useful, > > + but WITHOUT ANY WARRANTY; without even the implied warranty of > > + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > > + Lesser General Public License for more details. > > + > > + You should have received a copy of the GNU Lesser General Public > > + License along with the GNU C Library; if not, see > > + <https://www.gnu.org/licenses/>. */ > > + > > +#ifndef STRCMP > > +# define STRCMP __strncasecmp_l_avx2 > > +#endif > > +#define USE_AS_STRCASECMP_L > > +#define USE_AS_STRNCMP > > +#ifndef OVERFLOW_STRCMP > > +# define OVERFLOW_STRCMP __strcasecmp_l_avx2 > > +#endif > > +#include "strcmp-avx2.S" > > -- > > 2.25.1 > > > > LGTM. > > Reviewed-by: H.J. Lu <hjl.tools@gmail.com> > > Thanks. > > -- > H.J. I would like to backport this patch to release branches. Any comments or objections? Conflict resolution patch attached. --Sunil [-- Attachment #2: 0015-x86-Add-AVX2-optimized-str-n-casecmp.patch --] [-- Type: application/octet-stream, Size: 24673 bytes --] From b382e4caf50dfee62e170f9b6617b470b1289dcb Mon Sep 17 00:00:00 2001 From: Noah Goldstein <goldstein.w.n@gmail.com> Date: Thu, 24 Mar 2022 18:56:12 -0500 Subject: [PATCH 15/26] x86: Add AVX2 optimized str{n}casecmp geometric_mean(N=40) of all benchmarks AVX2 / SSE42: .702 All string/memory tests pass. Reviewed-by: H.J. Lu <hjl.tools@gmail.com> (cherry picked from commit bbf81222343fed5cd704001a2ae0d86c71544151) --- sysdeps/x86_64/multiarch/Makefile | 4 + sysdeps/x86_64/multiarch/ifunc-impl-list.c | 28 +++ sysdeps/x86_64/multiarch/ifunc-strcasecmp.h | 12 + .../x86_64/multiarch/strcasecmp_l-avx2-rtm.S | 15 ++ sysdeps/x86_64/multiarch/strcasecmp_l-avx2.S | 23 ++ sysdeps/x86_64/multiarch/strcmp-avx2.S | 237 +++++++++++++++--- .../x86_64/multiarch/strncase_l-avx2-rtm.S | 16 ++ sysdeps/x86_64/multiarch/strncase_l-avx2.S | 27 ++ 8 files changed, 331 insertions(+), 31 deletions(-) create mode 100644 sysdeps/x86_64/multiarch/strcasecmp_l-avx2-rtm.S create mode 100644 sysdeps/x86_64/multiarch/strcasecmp_l-avx2.S create mode 100644 sysdeps/x86_64/multiarch/strncase_l-avx2-rtm.S create mode 100644 sysdeps/x86_64/multiarch/strncase_l-avx2.S diff --git a/sysdeps/x86_64/multiarch/Makefile b/sysdeps/x86_64/multiarch/Makefile index e7b413edad..06e1848823 100644 --- a/sysdeps/x86_64/multiarch/Makefile +++ b/sysdeps/x86_64/multiarch/Makefile @@ -55,6 +55,8 @@ sysdep_routines += \ stpncpy-sse2-unaligned \ stpncpy-ssse3 \ strcasecmp_l-avx \ + strcasecmp_l-avx2 \ + strcasecmp_l-avx2-rtm \ strcasecmp_l-sse2 \ strcasecmp_l-sse4_2 \ strcasecmp_l-ssse3 \ @@ -93,6 +95,8 @@ sysdep_routines += \ strlen-evex \ strlen-sse2 \ strncase_l-avx \ + strncase_l-avx2 \ + strncase_l-avx2-rtm \ strncase_l-sse2 \ strncase_l-sse4_2 \ strncase_l-ssse3 \ diff --git a/sysdeps/x86_64/multiarch/ifunc-impl-list.c b/sysdeps/x86_64/multiarch/ifunc-impl-list.c index a594f4176e..3c556d07ac 100644 --- a/sysdeps/x86_64/multiarch/ifunc-impl-list.c +++ b/sysdeps/x86_64/multiarch/ifunc-impl-list.c @@ -436,6 +436,13 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, /* Support sysdeps/x86_64/multiarch/strcasecmp_l.c. */ IFUNC_IMPL (i, name, strcasecmp, + IFUNC_IMPL_ADD (array, i, strcasecmp, + CPU_FEATURE_USABLE (AVX2), + __strcasecmp_avx2) + IFUNC_IMPL_ADD (array, i, strcasecmp, + (CPU_FEATURE_USABLE (AVX2) + && CPU_FEATURE_USABLE (RTM)), + __strcasecmp_avx2_rtm) IFUNC_IMPL_ADD (array, i, strcasecmp, CPU_FEATURE_USABLE (AVX), __strcasecmp_avx) @@ -449,6 +456,13 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, /* Support sysdeps/x86_64/multiarch/strcasecmp_l.c. */ IFUNC_IMPL (i, name, strcasecmp_l, + IFUNC_IMPL_ADD (array, i, strcasecmp, + CPU_FEATURE_USABLE (AVX2), + __strcasecmp_l_avx2) + IFUNC_IMPL_ADD (array, i, strcasecmp, + (CPU_FEATURE_USABLE (AVX2) + && CPU_FEATURE_USABLE (RTM)), + __strcasecmp_l_avx2_rtm) IFUNC_IMPL_ADD (array, i, strcasecmp_l, CPU_FEATURE_USABLE (AVX), __strcasecmp_l_avx) @@ -576,6 +590,13 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, /* Support sysdeps/x86_64/multiarch/strncase_l.c. */ IFUNC_IMPL (i, name, strncasecmp, + IFUNC_IMPL_ADD (array, i, strncasecmp, + CPU_FEATURE_USABLE (AVX2), + __strncasecmp_avx2) + IFUNC_IMPL_ADD (array, i, strncasecmp, + (CPU_FEATURE_USABLE (AVX2) + && CPU_FEATURE_USABLE (RTM)), + __strncasecmp_avx2_rtm) IFUNC_IMPL_ADD (array, i, strncasecmp, CPU_FEATURE_USABLE (AVX), __strncasecmp_avx) @@ -590,6 +611,13 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, /* Support sysdeps/x86_64/multiarch/strncase_l.c. */ IFUNC_IMPL (i, name, strncasecmp_l, + IFUNC_IMPL_ADD (array, i, strncasecmp, + CPU_FEATURE_USABLE (AVX2), + __strncasecmp_l_avx2) + IFUNC_IMPL_ADD (array, i, strncasecmp, + (CPU_FEATURE_USABLE (AVX2) + && CPU_FEATURE_USABLE (RTM)), + __strncasecmp_l_avx2_rtm) IFUNC_IMPL_ADD (array, i, strncasecmp_l, CPU_FEATURE_USABLE (AVX), __strncasecmp_l_avx) diff --git a/sysdeps/x86_64/multiarch/ifunc-strcasecmp.h b/sysdeps/x86_64/multiarch/ifunc-strcasecmp.h index 9e3cc61ac0..c4de111fd0 100644 --- a/sysdeps/x86_64/multiarch/ifunc-strcasecmp.h +++ b/sysdeps/x86_64/multiarch/ifunc-strcasecmp.h @@ -23,12 +23,24 @@ extern __typeof (REDIRECT_NAME) OPTIMIZE (sse2) attribute_hidden; extern __typeof (REDIRECT_NAME) OPTIMIZE (ssse3) attribute_hidden; extern __typeof (REDIRECT_NAME) OPTIMIZE (sse42) attribute_hidden; extern __typeof (REDIRECT_NAME) OPTIMIZE (avx) attribute_hidden; +extern __typeof (REDIRECT_NAME) OPTIMIZE (avx2) attribute_hidden; +extern __typeof (REDIRECT_NAME) OPTIMIZE (avx2_rtm) attribute_hidden; static inline void * IFUNC_SELECTOR (void) { const struct cpu_features* cpu_features = __get_cpu_features (); + if (CPU_FEATURE_USABLE_P (cpu_features, AVX2) + && CPU_FEATURES_ARCH_P (cpu_features, AVX_Fast_Unaligned_Load)) + { + if (CPU_FEATURE_USABLE_P (cpu_features, RTM)) + return OPTIMIZE (avx2_rtm); + + if (!CPU_FEATURES_ARCH_P (cpu_features, Prefer_No_VZEROUPPER)) + return OPTIMIZE (avx2); + } + if (CPU_FEATURE_USABLE_P (cpu_features, AVX)) return OPTIMIZE (avx); diff --git a/sysdeps/x86_64/multiarch/strcasecmp_l-avx2-rtm.S b/sysdeps/x86_64/multiarch/strcasecmp_l-avx2-rtm.S new file mode 100644 index 0000000000..09957fc3c5 --- /dev/null +++ b/sysdeps/x86_64/multiarch/strcasecmp_l-avx2-rtm.S @@ -0,0 +1,15 @@ +#ifndef STRCMP +# define STRCMP __strcasecmp_l_avx2_rtm +#endif + +#define _GLABEL(x) x ## _rtm +#define GLABEL(x) _GLABEL(x) + +#define ZERO_UPPER_VEC_REGISTERS_RETURN \ + ZERO_UPPER_VEC_REGISTERS_RETURN_XTEST + +#define VZEROUPPER_RETURN jmp L(return_vzeroupper) + +#define SECTION(p) p##.avx.rtm + +#include "strcasecmp_l-avx2.S" diff --git a/sysdeps/x86_64/multiarch/strcasecmp_l-avx2.S b/sysdeps/x86_64/multiarch/strcasecmp_l-avx2.S new file mode 100644 index 0000000000..e2762f2a22 --- /dev/null +++ b/sysdeps/x86_64/multiarch/strcasecmp_l-avx2.S @@ -0,0 +1,23 @@ +/* strcasecmp_l optimized with AVX2. + Copyright (C) 2017-2022 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + <https://www.gnu.org/licenses/>. */ + +#ifndef STRCMP +# define STRCMP __strcasecmp_l_avx2 +#endif +#define USE_AS_STRCASECMP_L +#include "strcmp-avx2.S" diff --git a/sysdeps/x86_64/multiarch/strcmp-avx2.S b/sysdeps/x86_64/multiarch/strcmp-avx2.S index b94fc5c39a..3366d0b083 100644 --- a/sysdeps/x86_64/multiarch/strcmp-avx2.S +++ b/sysdeps/x86_64/multiarch/strcmp-avx2.S @@ -20,6 +20,10 @@ # include <sysdep.h> +# if defined USE_AS_STRCASECMP_L +# include "locale-defines.h" +# endif + # ifndef STRCMP # define STRCMP __strcmp_avx2 # endif @@ -74,13 +78,88 @@ # define VEC_OFFSET (-VEC_SIZE) # endif +# ifdef USE_AS_STRCASECMP_L +# define BYTE_LOOP_REG OFFSET_REG +# else +# define BYTE_LOOP_REG ecx +# endif + +# ifdef USE_AS_STRCASECMP_L +# ifdef USE_AS_STRNCMP +# define STRCASECMP __strncasecmp_avx2 +# define LOCALE_REG rcx +# define LOCALE_REG_LP RCX_LP +# define STRCASECMP_NONASCII __strncasecmp_l_nonascii +# else +# define STRCASECMP __strcasecmp_avx2 +# define LOCALE_REG rdx +# define LOCALE_REG_LP RDX_LP +# define STRCASECMP_NONASCII __strcasecmp_l_nonascii +# endif +# endif + # define xmmZERO xmm15 # define ymmZERO ymm15 +# define LCASE_MIN_ymm %ymm10 +# define LCASE_MAX_ymm %ymm11 +# define CASE_ADD_ymm %ymm12 + +# define LCASE_MIN_xmm %xmm10 +# define LCASE_MAX_xmm %xmm11 +# define CASE_ADD_xmm %xmm12 + + /* r11 is never use elsewhere so this is safe to maintain. */ +# define TOLOWER_BASE %r11 + # ifndef SECTION # define SECTION(p) p##.avx # endif +# ifdef USE_AS_STRCASECMP_L +# define REG(x, y) x ## y +# define TOLOWER(reg1_in, reg1_out, reg2_in, reg2_out, ext) \ + vpaddb REG(LCASE_MIN_, ext), reg1_in, REG(%ext, 8); \ + vpaddb REG(LCASE_MIN_, ext), reg2_in, REG(%ext, 9); \ + vpcmpgtb REG(LCASE_MAX_, ext), REG(%ext, 8), REG(%ext, 8); \ + vpcmpgtb REG(LCASE_MAX_, ext), REG(%ext, 9), REG(%ext, 9); \ + vpandn REG(CASE_ADD_, ext), REG(%ext, 8), REG(%ext, 8); \ + vpandn REG(CASE_ADD_, ext), REG(%ext, 9), REG(%ext, 9); \ + vpaddb REG(%ext, 8), reg1_in, reg1_out; \ + vpaddb REG(%ext, 9), reg2_in, reg2_out + +# define TOLOWER_gpr(src, dst) movl (TOLOWER_BASE, src, 4), dst +# define TOLOWER_ymm(...) TOLOWER(__VA_ARGS__, ymm) +# define TOLOWER_xmm(...) TOLOWER(__VA_ARGS__, xmm) + +# define CMP_R1_R2(s1_reg, s2_reg, scratch_reg, reg_out, ext) \ + TOLOWER (s1_reg, scratch_reg, s2_reg, s2_reg, ext); \ + VPCMPEQ scratch_reg, s2_reg, reg_out + +# define CMP_R1_S2(s1_reg, s2_mem, scratch_reg, reg_out, ext) \ + VMOVU s2_mem, reg_out; \ + CMP_R1_R2(s1_reg, reg_out, scratch_reg, reg_out, ext) + +# define CMP_R1_R2_ymm(...) CMP_R1_R2(__VA_ARGS__, ymm) +# define CMP_R1_R2_xmm(...) CMP_R1_R2(__VA_ARGS__, xmm) + +# define CMP_R1_S2_ymm(...) CMP_R1_S2(__VA_ARGS__, ymm) +# define CMP_R1_S2_xmm(...) CMP_R1_S2(__VA_ARGS__, xmm) + +# else +# define TOLOWER_gpr(...) +# define TOLOWER_ymm(...) +# define TOLOWER_xmm(...) + +# define CMP_R1_R2_ymm(s1_reg, s2_reg, scratch_reg, reg_out) \ + VPCMPEQ s2_reg, s1_reg, reg_out + +# define CMP_R1_R2_xmm(...) CMP_R1_R2_ymm(__VA_ARGS__) + +# define CMP_R1_S2_ymm(...) CMP_R1_R2_ymm(__VA_ARGS__) +# define CMP_R1_S2_xmm(...) CMP_R1_R2_xmm(__VA_ARGS__) +# endif + /* Warning! wcscmp/wcsncmp have to use SIGNED comparison for elements. strcmp/strncmp have to use UNSIGNED comparison for elements. @@ -102,8 +181,49 @@ returned. */ .section SECTION(.text), "ax", @progbits -ENTRY(STRCMP) + .align 16 + .type STRCMP, @function + .globl STRCMP + .hidden STRCMP + +# ifndef GLABEL +# define GLABEL(...) __VA_ARGS__ +# endif + +# ifdef USE_AS_STRCASECMP_L +ENTRY (GLABEL(STRCASECMP)) + movq __libc_tsd_LOCALE@gottpoff(%rip), %rax + mov %fs:(%rax), %LOCALE_REG_LP + + /* Either 1 or 5 bytes (dependeing if CET is enabled). */ + .p2align 4 +END (GLABEL(STRCASECMP)) + /* FALLTHROUGH to strcasecmp/strncasecmp_l. */ +# endif + + .p2align 4 +STRCMP: + cfi_startproc + _CET_ENDBR + CALL_MCOUNT + +# if defined USE_AS_STRCASECMP_L + /* We have to fall back on the C implementation for locales with + encodings not matching ASCII for single bytes. */ +# if LOCALE_T___LOCALES != 0 || LC_CTYPE != 0 + mov LOCALE_T___LOCALES + LC_CTYPE * LP_SIZE(%LOCALE_REG), %RAX_LP +# else + mov (%LOCALE_REG), %RAX_LP +# endif + testl $1, LOCALE_DATA_VALUES + _NL_CTYPE_NONASCII_CASE * SIZEOF_VALUES(%rax) + jne STRCASECMP_NONASCII + leaq _nl_C_LC_CTYPE_tolower + 128 * 4(%rip), TOLOWER_BASE +# endif + # ifdef USE_AS_STRNCMP + /* Don't overwrite LOCALE_REG (rcx) until we have pass + L(one_or_less). Otherwise we might use the wrong locale in + the OVERFLOW_STRCMP (strcasecmp_l). */ # ifdef __ILP32__ /* Clear the upper 32 bits. */ movl %edx, %edx @@ -128,6 +248,30 @@ ENTRY(STRCMP) # endif # endif vpxor %xmmZERO, %xmmZERO, %xmmZERO +# if defined USE_AS_STRCASECMP_L + .section .rodata.cst32, "aM", @progbits, 32 + .align 32 +L(lcase_min): + .quad 0x3f3f3f3f3f3f3f3f + .quad 0x3f3f3f3f3f3f3f3f + .quad 0x3f3f3f3f3f3f3f3f + .quad 0x3f3f3f3f3f3f3f3f +L(lcase_max): + .quad 0x9999999999999999 + .quad 0x9999999999999999 + .quad 0x9999999999999999 + .quad 0x9999999999999999 +L(case_add): + .quad 0x2020202020202020 + .quad 0x2020202020202020 + .quad 0x2020202020202020 + .quad 0x2020202020202020 + .previous + + vmovdqa L(lcase_min)(%rip), LCASE_MIN_ymm + vmovdqa L(lcase_max)(%rip), LCASE_MAX_ymm + vmovdqa L(case_add)(%rip), CASE_ADD_ymm +# endif movl %edi, %eax orl %esi, %eax sall $20, %eax @@ -138,8 +282,10 @@ ENTRY(STRCMP) L(no_page_cross): /* Safe to compare 4x vectors. */ VMOVU (%rdi), %ymm0 - /* 1s where s1 and s2 equal. */ - VPCMPEQ (%rsi), %ymm0, %ymm1 + /* 1s where s1 and s2 equal. Just VPCMPEQ if its not strcasecmp. + Otherwise converts ymm0 and load from rsi to lower. ymm2 is + scratch and ymm1 is the return. */ + CMP_R1_S2_ymm (%ymm0, (%rsi), %ymm2, %ymm1) /* 1s at null CHAR. */ VPCMPEQ %ymm0, %ymmZERO, %ymm2 /* 1s where s1 and s2 equal AND not null CHAR. */ @@ -172,6 +318,8 @@ L(return_vec_0): # else movzbl (%rdi, %rcx), %eax movzbl (%rsi, %rcx), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax # endif L(ret0): @@ -192,6 +340,10 @@ L(ret_zero): .p2align 4,, 5 L(one_or_less): +# ifdef USE_AS_STRCASECMP_L + /* Set locale argument for strcasecmp. */ + movq %LOCALE_REG, %rdx +# endif jb L(ret_zero) # ifdef USE_AS_WCSCMP /* 'nbe' covers the case where length is negative (large @@ -211,6 +363,8 @@ L(one_or_less): jnbe __strcmp_avx2 movzbl (%rdi), %eax movzbl (%rsi), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax # endif L(ret1): @@ -238,6 +392,8 @@ L(return_vec_1): # else movzbl VEC_SIZE(%rdi, %rcx), %eax movzbl VEC_SIZE(%rsi, %rcx), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax # endif L(ret2): @@ -269,6 +425,8 @@ L(return_vec_2): # else movzbl (VEC_SIZE * 2)(%rdi, %rcx), %eax movzbl (VEC_SIZE * 2)(%rsi, %rcx), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax # endif L(ret3): @@ -289,6 +447,8 @@ L(return_vec_3): # else movzbl (VEC_SIZE * 3)(%rdi, %rcx), %eax movzbl (VEC_SIZE * 3)(%rsi, %rcx), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax # endif L(ret4): @@ -299,7 +459,7 @@ L(ret4): L(more_3x_vec): /* Safe to compare 4x vectors. */ VMOVU VEC_SIZE(%rdi), %ymm0 - VPCMPEQ VEC_SIZE(%rsi), %ymm0, %ymm1 + CMP_R1_S2_ymm (%ymm0, VEC_SIZE(%rsi), %ymm2, %ymm1) VPCMPEQ %ymm0, %ymmZERO, %ymm2 vpandn %ymm1, %ymm2, %ymm1 vpmovmskb %ymm1, %ecx @@ -312,7 +472,7 @@ L(more_3x_vec): # endif VMOVU (VEC_SIZE * 2)(%rdi), %ymm0 - VPCMPEQ (VEC_SIZE * 2)(%rsi), %ymm0, %ymm1 + CMP_R1_S2_ymm (%ymm0, (VEC_SIZE * 2)(%rsi), %ymm2, %ymm1) VPCMPEQ %ymm0, %ymmZERO, %ymm2 vpandn %ymm1, %ymm2, %ymm1 vpmovmskb %ymm1, %ecx @@ -320,7 +480,7 @@ L(more_3x_vec): jnz L(return_vec_2) VMOVU (VEC_SIZE * 3)(%rdi), %ymm0 - VPCMPEQ (VEC_SIZE * 3)(%rsi), %ymm0, %ymm1 + CMP_R1_S2_ymm (%ymm0, (VEC_SIZE * 3)(%rsi), %ymm2, %ymm1) VPCMPEQ %ymm0, %ymmZERO, %ymm2 vpandn %ymm1, %ymm2, %ymm1 vpmovmskb %ymm1, %ecx @@ -395,12 +555,10 @@ L(loop_skip_page_cross_check): VMOVA (VEC_SIZE * 3)(%rdi), %ymm6 /* ymm1 all 1s where s1 and s2 equal. All 0s otherwise. */ - VPCMPEQ (VEC_SIZE * 0)(%rsi), %ymm0, %ymm1 - - VPCMPEQ (VEC_SIZE * 1)(%rsi), %ymm2, %ymm3 - VPCMPEQ (VEC_SIZE * 2)(%rsi), %ymm4, %ymm5 - VPCMPEQ (VEC_SIZE * 3)(%rsi), %ymm6, %ymm7 - + CMP_R1_S2_ymm (%ymm0, (VEC_SIZE * 0)(%rsi), %ymm3, %ymm1) + CMP_R1_S2_ymm (%ymm2, (VEC_SIZE * 1)(%rsi), %ymm5, %ymm3) + CMP_R1_S2_ymm (%ymm4, (VEC_SIZE * 2)(%rsi), %ymm7, %ymm5) + CMP_R1_S2_ymm (%ymm6, (VEC_SIZE * 3)(%rsi), %ymm13, %ymm7) /* If any mismatches or null CHAR then 0 CHAR, otherwise non- zero. */ @@ -469,6 +627,8 @@ L(return_vec_2_3_end): # else movzbl (VEC_SIZE * 2 - VEC_OFFSET)(%rdi, %LOOP_REG64), %eax movzbl (VEC_SIZE * 2 - VEC_OFFSET)(%rsi, %LOOP_REG64), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax xorl %r8d, %eax subl %r8d, %eax @@ -512,6 +672,8 @@ L(return_vec_0_end): # else movzbl (%rdi, %rcx), %eax movzbl (%rsi, %rcx), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax xorl %r8d, %eax subl %r8d, %eax @@ -534,6 +696,8 @@ L(return_vec_1_end): # else movzbl VEC_SIZE(%rdi, %rcx), %eax movzbl VEC_SIZE(%rsi, %rcx), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax xorl %r8d, %eax subl %r8d, %eax @@ -560,6 +724,8 @@ L(return_vec_2_end): # else movzbl (VEC_SIZE * 2)(%rdi, %rcx), %eax movzbl (VEC_SIZE * 2)(%rsi, %rcx), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax xorl %r8d, %eax subl %r8d, %eax @@ -587,7 +753,7 @@ L(page_cross_during_loop): jle L(less_1x_vec_till_page_cross) VMOVA (%rdi), %ymm0 - VPCMPEQ (%rsi), %ymm0, %ymm1 + CMP_R1_S2_ymm (%ymm0, (%rsi), %ymm2, %ymm1) VPCMPEQ %ymm0, %ymmZERO, %ymm2 vpandn %ymm1, %ymm2, %ymm1 vpmovmskb %ymm1, %ecx @@ -609,7 +775,7 @@ L(less_1x_vec_till_page_cross): here, it means the previous page (rdi - VEC_SIZE) has already been loaded earlier so must be valid. */ VMOVU -VEC_SIZE(%rdi, %rax), %ymm0 - VPCMPEQ -VEC_SIZE(%rsi, %rax), %ymm0, %ymm1 + CMP_R1_S2_ymm (%ymm0, -VEC_SIZE(%rsi, %rax), %ymm2, %ymm1) VPCMPEQ %ymm0, %ymmZERO, %ymm2 vpandn %ymm1, %ymm2, %ymm1 vpmovmskb %ymm1, %ecx @@ -651,6 +817,8 @@ L(return_page_cross_cmp_mem): # else movzbl VEC_OFFSET(%rdi, %rcx), %eax movzbl VEC_OFFSET(%rsi, %rcx), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax xorl %r8d, %eax subl %r8d, %eax @@ -677,7 +845,7 @@ L(more_2x_vec_till_page_cross): iteration here. */ VMOVU VEC_SIZE(%rdi), %ymm0 - VPCMPEQ VEC_SIZE(%rsi), %ymm0, %ymm1 + CMP_R1_S2_ymm (%ymm0, VEC_SIZE(%rsi), %ymm2, %ymm1) VPCMPEQ %ymm0, %ymmZERO, %ymm2 vpandn %ymm1, %ymm2, %ymm1 vpmovmskb %ymm1, %ecx @@ -693,7 +861,7 @@ L(more_2x_vec_till_page_cross): /* Safe to include comparisons from lower bytes. */ VMOVU -(VEC_SIZE * 2)(%rdi, %rax), %ymm0 - VPCMPEQ -(VEC_SIZE * 2)(%rsi, %rax), %ymm0, %ymm1 + CMP_R1_S2_ymm (%ymm0, -(VEC_SIZE * 2)(%rsi, %rax), %ymm2, %ymm1) VPCMPEQ %ymm0, %ymmZERO, %ymm2 vpandn %ymm1, %ymm2, %ymm1 vpmovmskb %ymm1, %ecx @@ -701,7 +869,7 @@ L(more_2x_vec_till_page_cross): jnz L(return_vec_page_cross_0) VMOVU -(VEC_SIZE * 1)(%rdi, %rax), %ymm0 - VPCMPEQ -(VEC_SIZE * 1)(%rsi, %rax), %ymm0, %ymm1 + CMP_R1_S2_ymm (%ymm0, -(VEC_SIZE * 1)(%rsi, %rax), %ymm2, %ymm1) VPCMPEQ %ymm0, %ymmZERO, %ymm2 vpandn %ymm1, %ymm2, %ymm1 vpmovmskb %ymm1, %ecx @@ -719,8 +887,8 @@ L(more_2x_vec_till_page_cross): VMOVA (VEC_SIZE * 2)(%rdi), %ymm4 VMOVA (VEC_SIZE * 3)(%rdi), %ymm6 - VPCMPEQ (VEC_SIZE * 2)(%rsi), %ymm4, %ymm5 - VPCMPEQ (VEC_SIZE * 3)(%rsi), %ymm6, %ymm7 + CMP_R1_S2_ymm (%ymm4, (VEC_SIZE * 2)(%rsi), %ymm7, %ymm5) + CMP_R1_S2_ymm (%ymm6, (VEC_SIZE * 3)(%rsi), %ymm13, %ymm7) vpand %ymm4, %ymm5, %ymm5 vpand %ymm6, %ymm7, %ymm7 VPMINU %ymm5, %ymm7, %ymm7 @@ -771,6 +939,8 @@ L(return_vec_page_cross_1): # else movzbl VEC_OFFSET(%rdi, %rcx), %eax movzbl VEC_OFFSET(%rsi, %rcx), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax xorl %r8d, %eax subl %r8d, %eax @@ -826,7 +996,7 @@ L(page_cross): L(page_cross_loop): VMOVU (%rdi, %OFFSET_REG64), %ymm0 - VPCMPEQ (%rsi, %OFFSET_REG64), %ymm0, %ymm1 + CMP_R1_S2_ymm (%ymm0, (%rsi, %OFFSET_REG64), %ymm2, %ymm1) VPCMPEQ %ymm0, %ymmZERO, %ymm2 vpandn %ymm1, %ymm2, %ymm1 vpmovmskb %ymm1, %ecx @@ -844,11 +1014,11 @@ L(page_cross_loop): subl %eax, %OFFSET_REG /* OFFSET_REG has distance to page cross - VEC_SIZE. Guranteed to not cross page so is safe to load. Since we have already - loaded at least 1 VEC from rsi it is also guranteed to be safe. - */ + loaded at least 1 VEC from rsi it is also guranteed to be + safe. */ VMOVU (%rdi, %OFFSET_REG64), %ymm0 - VPCMPEQ (%rsi, %OFFSET_REG64), %ymm0, %ymm1 + CMP_R1_S2_ymm (%ymm0, (%rsi, %OFFSET_REG64), %ymm2, %ymm1) VPCMPEQ %ymm0, %ymmZERO, %ymm2 vpandn %ymm1, %ymm2, %ymm1 vpmovmskb %ymm1, %ecx @@ -881,6 +1051,8 @@ L(ret_vec_page_cross_cont): # else movzbl (%rdi, %rcx), %eax movzbl (%rsi, %rcx), %ecx + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %ecx) subl %ecx, %eax xorl %r8d, %eax subl %r8d, %eax @@ -934,7 +1106,7 @@ L(less_1x_vec_till_page): ja L(less_16_till_page) VMOVU (%rdi), %xmm0 - VPCMPEQ (%rsi), %xmm0, %xmm1 + CMP_R1_S2_xmm (%xmm0, (%rsi), %xmm2, %xmm1) VPCMPEQ %xmm0, %xmmZERO, %xmm2 vpandn %xmm1, %xmm2, %xmm1 vpmovmskb %ymm1, %ecx @@ -952,7 +1124,7 @@ L(less_1x_vec_till_page): # endif VMOVU (%rdi, %OFFSET_REG64), %xmm0 - VPCMPEQ (%rsi, %OFFSET_REG64), %xmm0, %xmm1 + CMP_R1_S2_xmm (%xmm0, (%rsi, %OFFSET_REG64), %xmm2, %xmm1) VPCMPEQ %xmm0, %xmmZERO, %xmm2 vpandn %xmm1, %xmm2, %xmm1 vpmovmskb %ymm1, %ecx @@ -990,7 +1162,7 @@ L(less_16_till_page): vmovq (%rdi), %xmm0 vmovq (%rsi), %xmm1 VPCMPEQ %xmm0, %xmmZERO, %xmm2 - VPCMPEQ %xmm1, %xmm0, %xmm1 + CMP_R1_R2_xmm (%xmm0, %xmm1, %xmm3, %xmm1) vpandn %xmm1, %xmm2, %xmm1 vpmovmskb %ymm1, %ecx incb %cl @@ -1010,7 +1182,7 @@ L(less_16_till_page): vmovq (%rdi, %OFFSET_REG64), %xmm0 vmovq (%rsi, %OFFSET_REG64), %xmm1 VPCMPEQ %xmm0, %xmmZERO, %xmm2 - VPCMPEQ %xmm1, %xmm0, %xmm1 + CMP_R1_R2_xmm (%xmm0, %xmm1, %xmm3, %xmm1) vpandn %xmm1, %xmm2, %xmm1 vpmovmskb %ymm1, %ecx incb %cl @@ -1066,7 +1238,7 @@ L(ret_less_8_wcs): vmovd (%rdi), %xmm0 vmovd (%rsi), %xmm1 VPCMPEQ %xmm0, %xmmZERO, %xmm2 - VPCMPEQ %xmm1, %xmm0, %xmm1 + CMP_R1_R2_xmm (%xmm0, %xmm1, %xmm3, %xmm1) vpandn %xmm1, %xmm2, %xmm1 vpmovmskb %ymm1, %ecx subl $0xf, %ecx @@ -1085,7 +1257,7 @@ L(ret_less_8_wcs): vmovd (%rdi, %OFFSET_REG64), %xmm0 vmovd (%rsi, %OFFSET_REG64), %xmm1 VPCMPEQ %xmm0, %xmmZERO, %xmm2 - VPCMPEQ %xmm1, %xmm0, %xmm1 + CMP_R1_R2_xmm (%xmm0, %xmm1, %xmm3, %xmm1) vpandn %xmm1, %xmm2, %xmm1 vpmovmskb %ymm1, %ecx subl $0xf, %ecx @@ -1119,7 +1291,9 @@ L(less_4_till_page): L(less_4_loop): movzbl (%rdi), %eax movzbl (%rsi, %rdi), %ecx - subl %ecx, %eax + TOLOWER_gpr (%rax, %eax) + TOLOWER_gpr (%rcx, %BYTE_LOOP_REG) + subl %BYTE_LOOP_REG, %eax jnz L(ret_less_4_loop) testl %ecx, %ecx jz L(ret_zero_4_loop) @@ -1146,5 +1320,6 @@ L(ret_less_4_loop): subl %r8d, %eax ret # endif -END(STRCMP) + cfi_endproc + .size STRCMP, .-STRCMP #endif diff --git a/sysdeps/x86_64/multiarch/strncase_l-avx2-rtm.S b/sysdeps/x86_64/multiarch/strncase_l-avx2-rtm.S new file mode 100644 index 0000000000..58c05dcfb8 --- /dev/null +++ b/sysdeps/x86_64/multiarch/strncase_l-avx2-rtm.S @@ -0,0 +1,16 @@ +#ifndef STRCMP +# define STRCMP __strncasecmp_l_avx2_rtm +#endif + +#define _GLABEL(x) x ## _rtm +#define GLABEL(x) _GLABEL(x) + +#define ZERO_UPPER_VEC_REGISTERS_RETURN \ + ZERO_UPPER_VEC_REGISTERS_RETURN_XTEST + +#define VZEROUPPER_RETURN jmp L(return_vzeroupper) + +#define SECTION(p) p##.avx.rtm +#define OVERFLOW_STRCMP __strcasecmp_l_avx2_rtm + +#include "strncase_l-avx2.S" diff --git a/sysdeps/x86_64/multiarch/strncase_l-avx2.S b/sysdeps/x86_64/multiarch/strncase_l-avx2.S new file mode 100644 index 0000000000..48c0aa21f8 --- /dev/null +++ b/sysdeps/x86_64/multiarch/strncase_l-avx2.S @@ -0,0 +1,27 @@ +/* strncasecmp_l optimized with AVX2. + Copyright (C) 2017-2022 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + <https://www.gnu.org/licenses/>. */ + +#ifndef STRCMP +# define STRCMP __strncasecmp_l_avx2 +#endif +#define USE_AS_STRCASECMP_L +#define USE_AS_STRNCMP +#ifndef OVERFLOW_STRCMP +# define OVERFLOW_STRCMP __strcasecmp_l_avx2 +#endif +#include "strcmp-avx2.S" -- 2.35.1 ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-05-12 19:53 UTC | newest] Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <20220323215734.3927131-21-goldstein.w.n@gmail.com> [not found] ` <20220324235613.2445581-1-goldstein.w.n@gmail.com> [not found] ` <20220324235613.2445581-2-goldstein.w.n@gmail.com> [not found] ` <CAMe9rOp3wFrvLmM9PhtHWJis-i85QCrL17Oua8+xj=x2eSVxkg@mail.gmail.com> [not found] ` <CAFUsyf+FsB7x-O-FB87vsxNF5XaXJ8+CxOr-sgaO+5PbXcH6qQ@mail.gmail.com> 2022-05-12 19:47 ` [PATCH v4 22/23] x86: Add EVEX optimized str{n}casecmp Sunil Pandey 2022-05-12 19:52 ` Sunil Pandey [not found] ` <CAMe9rOqZ8h1dOUctD6jF3OmUv9TXjyA7_ser3p_6157myHq1Og@mail.gmail.com> 2022-05-12 19:52 ` [PATCH v4 21/23] x86: Add AVX2 " 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).