summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorH.J. Lu <hongjiu.lu@intel.com>2009-11-20 08:30:20 -0800
committerUlrich Drepper <drepper@redhat.com>2009-11-20 08:30:20 -0800
commit66221417957963c1d2df8d79f43573e328df2275 (patch)
treefb61d07e4512b1aa04c27affdd5b03f3f2aacfa1
parentdf1934daa7788ca5f19d82efb02cdf9b9c495b1a (diff)
downloadglibc-66221417957963c1d2df8d79f43573e328df2275.tar.gz
Properly recover from shorter read.
-rw-r--r--ChangeLog6
-rw-r--r--sysdeps/ia64/memchr.S14
2 files changed, 17 insertions, 3 deletions
diff --git a/ChangeLog b/ChangeLog
index a60bfcc8e6..c99965f6e4 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2009-11-18 H.J. Lu <hongjiu.lu@intel.com>
+
+ [BZ #10162]
+ * sysdeps/ia64/memchr.S: Don't use a simple loop on data shorter
+ than software pipeline. Properly recover from shorter read.
+
2009-11-19 Ulrich Drepper <drepper@redhat.com>
[BZ #10958]
diff --git a/sysdeps/ia64/memchr.S b/sysdeps/ia64/memchr.S
index e721e1714f..cdd71ca5a5 100644
--- a/sysdeps/ia64/memchr.S
+++ b/sysdeps/ia64/memchr.S
@@ -71,8 +71,7 @@ ENTRY(__memchr)
cmp.ne p7, p0 = r0, r0 // clear p7
extr.u chr = in1, 0, 8 // chr = (unsigned char) in1
mov len = in2
- // use a simple loop on data shorter than software pipeline
- cmp.gtu p6, p0 = ((MEMLAT + 1) * 8), in2
+ cmp.gtu p6, p0 = 16, in2 // use a simple loop for short
(p6) br.cond.spnt .srchfew ;; // searches
sub loopcnt = 8, tmp // loopcnt = 8 - tmp
cmp.eq p6, p0 = tmp, r0
@@ -127,7 +126,16 @@ ENTRY(__memchr)
br.ret.sptk.many b0
.recovery:
- adds ret0 = -((MEMLAT + 1) * 8), ret0;;
+#if MEMLAT != 6
+# error "MEMLAT must be 6!"
+#endif
+(p[MEMLAT-6]) add ret0 = -8, ret0;;
+(p[MEMLAT-5]) add ret0 = -8, ret0;;
+(p[MEMLAT-4]) add ret0 = -8, ret0;;
+(p[MEMLAT-3]) add ret0 = -8, ret0;;
+(p[MEMLAT-2]) add ret0 = -8, ret0;;
+(p[MEMLAT-1]) add ret0 = -8, ret0;;
+(p[MEMLAT]) add ret0 = -8, ret0;;
(p[MEMLAT+1]) add ret0 = -8, ret0;;
(p[MEMLAT+2]) add ret0 = -8, ret0;;
.l4: