public inbox for libc-stable@sourceware.org
 help / color / mirror / Atom feed
* [2.27 COMMITTED][AArch64] Backport strcmp improvements
@ 2019-01-01  0:00 Wilco Dijkstra
  0 siblings, 0 replies; only message in thread
From: Wilco Dijkstra @ 2019-01-01  0:00 UTC (permalink / raw)
  To: libc-stable; +Cc: nd

commit d5f45a29ff152a079f3ab6b3f9a8f8dce261252f
Author: Siddhesh Poyarekar <siddhesh@sourceware.org>
Date:   Thu Feb 22 23:48:13 2018 +0530

    aarch64/strcmp: fix misaligned loop jump target
    
    I accidentally set the loop jump back label as misaligned8 instead of
    do_misaligned.  The typo is harmless but it's always nice to not have
    to unnecessarily execute those two instructions.
    
        * sysdeps/aarch64/strcmp.S (do_misaligned): Jump back to
        do_misaligned, not misaligned8.
    
    (cherry picked from commit 6ca24c43481e2c93a6eec362b04c3e77a35b28e3)

diff --git a/ChangeLog b/ChangeLog
index 20d98f0..5ddc3e7 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,15 @@
 2019-09-06  Siddhesh Poyarekar  <siddhesh@sourceware.org>
 
+       * sysdeps/aarch64/strcmp.S (do_misaligned): Jump back to
+       do_misaligned, not misaligned8.
+
+2019-09-06  Siddhesh Poyarekar  <siddhesh@sourceware.org>
+
+       * sysdeps/aarch64/strcmp.S (misaligned8): Compare dword at a
+       time whenever possible.
+
+2019-09-06  Siddhesh Poyarekar  <siddhesh@sourceware.org>
+
        * sysdeps/aarch64/memcmp.S: Widen comparison to 16 bytes at a
        time.
 
diff --git a/sysdeps/aarch64/strcmp.S b/sysdeps/aarch64/strcmp.S
index 0b22f16..267aa4b 100644
--- a/sysdeps/aarch64/strcmp.S
+++ b/sysdeps/aarch64/strcmp.S
@@ -158,7 +158,7 @@ L(do_misaligned):
        ccmp    data1w, data2w, #0, cs  /* NZCV = 0b0000.  */
        b.ne    L(done)
        tst     src1, #7
-       b.ne    L(misaligned8)
+       b.ne    L(do_misaligned)
 
 L(loop_misaligned):
        /* Test if we are within the last dword of the end of a 4K page.  If



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

only message in thread, other threads:[~2019-09-06 16:45 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-01  0:00 [2.27 COMMITTED][AArch64] Backport strcmp improvements Wilco Dijkstra

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).