Mon Jun 24 19:57:01 1996 Roland McGrath <roland@delasyd.gnu.ai.mit.edu>
[kopensolaris-gnu/glibc.git] / sysdeps / i386 / memset.c
index ab88789..2987445 100644 (file)
@@ -1,6 +1,6 @@
 /* memset -- set a block of memory to some byte value.
    For Intel 80x86, x>=3.
-   Copyright (C) 1991, 1992 Free Software Foundation, Inc.
+   Copyright (C) 1991, 1992, 1993 Free Software Foundation, Inc.
    Contributed by Torbjorn Granlund (tege@sics.se).
 
 The GNU C Library is free software; you can redistribute it and/or
@@ -34,10 +34,7 @@ DEFUN(memset, (dstpp, c, len),
      improves code very much indeed.  */
   register op_t x asm("ax");
 
-  /* Fill X with four copies of the char we want to fill with.  */
   x = (unsigned char) c;
-  x |= (x << 8);
-  x |= (x << 16);
 
   /* Clear the direction flag, so filling will move forward.  */
   asm volatile("cld");
@@ -45,6 +42,13 @@ DEFUN(memset, (dstpp, c, len),
   /* This threshold value is optimal.  */
   if (len >= 12)
     {
+      /* Fill X with four copies of the char we want to fill with.  */
+      x |= (x << 8);
+      x |= (x << 16);
+
+      /* Adjust LEN for the bytes handled in the first loop.  */
+      len -= (-dstp) % OPSIZ;
+
       /* There are at least some bytes to set.
         No need to test for LEN == 0 in this alignment loop.  */
 
@@ -54,7 +58,6 @@ DEFUN(memset, (dstpp, c, len),
                   "=D" (dstp) :
                   "0" (dstp), "c" ((-dstp) % OPSIZ), "a" (x) :
                   "cx");
-      len -= (-dstp) % OPSIZ;
 
       /* Fill longwords.  */
       asm volatile("rep\n"
@@ -76,5 +79,5 @@ DEFUN(memset, (dstpp, c, len),
 }
 
 #else
-#include <sysdeps/generic/bzero.c>
+#include <sysdeps/generic/memset.c>
 #endif