mirrors/glibc
1
0
Fork 0
mirror of https://sourceware.org/git/glibc.git synced 2024-11-23 17:53:37 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Update.

Browse Source 
* sysdeps/x86_64/strchr.S: Don't use one register for two
	purposes, this fixes a bug noticed by test-strchr.c.
This commit is contained in:
Andreas Jaeger 2002-11-12 21:43:26 +00:00
parent 44df0cea62
commit 6e14793db0
2 changed files with 9 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
ChangeLog
View File

@ -1,5 +1,8 @@
2002-11-12 Andreas Jaeger <aj@suse.de>
* sysdeps/x86_64/strchr.S: Don't use one register for two
purposes, this fixes a bug noticed by test-strchr.c.
* sysdeps/x86_64/strcat.S: Fix algorithm to align source pointer
correctly.

12
sysdeps/x86_64/strchr.S
View File

@ -36,14 +36,14 @@ ENTRY (BP_SYM (strchr))
8-byte alignment guarantees that we never access illegal
memory if this would not also be done by the trivial
implementation (this is because all processor inherent
boundaries are multiples of 8. */
boundaries are multiples of 8). */
movq %rdi, %rcx
andl $7, %ecx /* Mask alignment bits */
movq %rdi, %rdx
andl $7, %edx /* Mask alignment bits */
movq %rdi, %rax /* duplicate destination. */
jz 1f /* aligned => start loop */
neg %ecx
addl $8, %ecx /* Align to 8 bytes. */
neg %edx
addl $8, %edx /* Align to 8 bytes. */
/* Search the first bytes directly. */
0: movb (%rax), %cl /* load byte */
@ -52,7 +52,7 @@ ENTRY (BP_SYM (strchr))
testb %cl,%cl /* is byte NUL? */
je 7f /* yes => return NULL */
incq %rax /* increment pointer */
decl %ecx
decl %edx
jnz 0b