powerpc: Add powerpc64 strcspn optimization

This patch makes the POWER7 optimized strcspn generic by using
default doubleword stores to zero the hash, instead of VSX
instructions.  Performance on POWER7/POWER8 does not change.
This commit is contained in:
Adhemerval Zanella 2014-11-19 15:24:55 -05:00
parent 2e8a2de2da
commit bb2542e0ae
7 changed files with 33 additions and 146 deletions

View File

@ -1,4 +1,13 @@
2014-12-02 Adhemerval Zanella <azanella@linux.vnet.ibm.com>
* sysdeps/powerpc/powerpc64/multiarch/Makefile [sysdep_routines]:
Remove strcspn objects.
* sysdeps/powerpc/powerpc64/multiarch/ifunc-impl-list.c
(__libc_ifunc_impl_list): Remove strcspn implementation.
* sysdeps/powerpc/powerpc64/multiarch/strcspn-ppc64.c: Remove file.
* sysdeps/powerpc/powerpc64/multiarch/strcspn.c: Remove file.
* sysdeps/powerpc/powerpc64/power7/strcspn.S: Remove file.
* sysdeps/powerpc/powerpc64/strcspn.S: New file.
* sysdeps/powerpc/powerpc64/multiarch/Makefile [sysdep_routines]:
Remove strspn objetcs.

View File

@ -15,7 +15,6 @@ sysdep_routines += memcpy-power7 memcpy-a2 memcpy-power6 memcpy-cell \
wordcopy-power7 wordcopy-power6 wordcopy-ppc64 \
strcpy-power7 strcpy-ppc64 stpcpy-power7 stpcpy-ppc64 \
strrchr-power7 strrchr-ppc64 strncat-power7 strncat-ppc64 \
strcspn-power7 strcspn-ppc64 \
strpbrk-power7 strpbrk-ppc64 strncpy-power7 strncpy-ppc64 \
stpncpy-power7 stpncpy-ppc64 strcmp-power7 strcmp-ppc64 \
strcat-power7 strcat-ppc64 memmove-power7 memmove-ppc64 \

View File

@ -272,14 +272,6 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array,
IFUNC_IMPL_ADD (array, i, strncat, 1,
__strncat_ppc))
/* Support sysdeps/powerpc/powerpc64/multiarch/strcspn.c. */
IFUNC_IMPL (i, name, strcspn,
IFUNC_IMPL_ADD (array, i, strcspn,
hwcap & PPC_FEATURE_HAS_VSX,
__strcspn_power7)
IFUNC_IMPL_ADD (array, i, strcspn, 1,
__strcspn_ppc))
/* Support sysdeps/powerpc/powerpc64/multiarch/strpbrk.c. */
IFUNC_IMPL (i, name, strpbrk,
IFUNC_IMPL_ADD (array, i, strpbrk,

View File

@ -1,40 +0,0 @@
/* Optimized strcspn implementation for POWER7.
Copyright (C) 2014 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
<http://www.gnu.org/licenses/>. */
#include <sysdep.h>
#undef EALIGN
#define EALIGN(name, alignt, words) \
.section ".text"; \
ENTRY_2(__strcspn_power7) \
.align ALIGNARG(alignt); \
EALIGN_W_##words; \
BODY_LABEL(__strcspn_power7): \
cfi_startproc; \
LOCALENTRY(__strcspn_power7)
#undef END
#define END(name) \
cfi_endproc; \
TRACEBACK(__strcspn_power7) \
END_2(__strcspn_power7)
#undef libc_hidden_builtin_def
#define libc_hidden_builtin_def(name)
#include <sysdeps/powerpc/powerpc64/power7/strcspn.S>

View File

@ -1,30 +0,0 @@
/* Copyright (C) 2014 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
<http://www.gnu.org/licenses/>. */
#include <string.h>
#define STRCSPN __strcspn_ppc
#ifdef SHARED
# undef libc_hidden_builtin_def
# define libc_hidden_builtin_def(name) \
__hidden_ver1 (__strcspn_ppc, __GI_strcspn, __strcspn_ppc);
#endif
extern __typeof (strcspn) __strcspn_ppc attribute_hidden;
#include <string/strcspn.c>

View File

@ -1,31 +0,0 @@
/* Multiple versions of strcspn. PowerPC64 version.
Copyright (C) 2014 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
<http://www.gnu.org/licenses/>. */
#if IS_IN (libc)
# include <string.h>
# include <shlib-compat.h>
# include "init-arch.h"
extern __typeof (strcspn) __strcspn_ppc attribute_hidden;
extern __typeof (strcspn) __strcspn_power7 attribute_hidden;
libc_ifunc (strcspn,
(hwcap & PPC_FEATURE_HAS_VSX)
? __strcspn_power7
: __strcspn_ppc);
#endif

View File

@ -20,54 +20,42 @@
/* size_t [r3] strcspn (const char [r4] *s, const char [r5] *reject) */
.machine power7
EALIGN (strcspn, 4, 0)
CALL_MCOUNT 3
/* The idea to speed up the algorithm is to create a lookup table
for fast check if input character should be considered. For ASCII
or ISO-8859-X character sets it has 256 positions. */
lbz r10,0(r4)
/* First the table should be cleared and to avoid unaligned accesses
when using the VSX stores the table address is aligned to 16
bytes. */
xxlxor v0,v0,v0
/* PPC64 ELF ABI stack is aligned to 16 bytes. */
addi r9,r1,-256
li r8,48
li r5,16
li r6,32
cmpdi cr7,r10,0 /* reject[0] == '\0' ? */
addi r12,r9,64
/* Clear the table with 0 values */
stxvw4x v0,r0,r9
addi r11,r9,128
addi r7,r9,192
stxvw4x v0,r9,r5
stxvw4x v0,r9,r6
stxvw4x v0,r9,r8
stxvw4x v0,r0,r12
stxvw4x v0,r12,r5
stxvw4x v0,r12,r6
stxvw4x v0,r12,r8
stxvw4x v0,r0,r11
stxvw4x v0,r11,r5
stxvw4x v0,r11,r6
stxvw4x v0,r11,r8
stxvw4x v0,r0,r7
stxvw4x v0,r7,r5
stxvw4x v0,r7,r6
stxvw4x v0,r7,r8
li r6, 0
li r8, 4
mtctr r8
mr r10, r9
.align 4
L(zerohash):
std r6, 0(r10)
std r6, 8(r10)
std r6, 16(r10)
std r6, 24(r10)
std r6, 32(r10)
std r6, 40(r10)
std r6, 48(r10)
std r6, 56(r10)
addi r10, r10, 64
bdnz L(zerohash)
lbz r10,0(r4)
cmpdi cr7,r10,0 /* reject[0] == '\0' ? */
li r8,1
beq cr7,L(finish_table) /* If reject[0] == '\0' skip */
/* Initialize the table as:
for (i=0; reject[i]; i++
table[reject[i]]] = 1 */
.p2align 4,,15
.align 4
L(init_table):
stbx r8,r9,r10
lbzu r10,1(r4)
@ -93,7 +81,7 @@ L(finish_table):
if (table[input[i++]] == 1)
return i - 1;
} */
.p2align 4,,15
.align 4
L(unroll):
lbz r8,1(r3)
addi r10,r10,4
@ -121,17 +109,17 @@ L(mainloop):
mr r3,r10
blr
.p2align 4,,15
.align 4
L(end):
mr r3,r6
blr
.p2align 4,,15
.align 4
L(end2):
mr r3,r4
blr
.p2align 4,,15
.align 4
L(end3):
mr r3,r5
blr