Update.

1998-09-29  Andreas Schwab  <schwab@issan.informatik.uni-dortmund.de> 
 
	* string/bits/string2.h (__string2_1bptr_p): Don't use a statement 
	expression. 
	(__mempcpy, strcpy, __stpcpy, strcmp, strcspn, strspn, strpbrk, 
	__strsep): Greatly reduce size of expansion by combining common 
	expressions. 
	(__mempcpy_small, __strcpy_small, __stpcpy_small): Convert to 
	inline function. 
	(__mempcpy_args, __strcpy_args, __stpcpy_args): New macros, used 
	by the above definitions. 
 

1 files changed, 424 insertions, 449 deletions

diff --git a/string/bits/string2.h b/string/bits/string2.h
index 172c21734a..c47a73841a 100644
--- a/string/bits/string2.h
+++ b/string/bits/string2.h
@@ -89,8 +89,7 @@ __STRING2_COPY_TYPE (8);
    pointer case, so we use this instead.
    Note that __x is evaluated twice. */
 #define __string2_1bptr_p(__x) \
-	(({ const void *__a, *__b; __a = (__x) + 1; __b = (__x); \
-	    (size_t)__a - (size_t)__b; }) == 1)
+  ((size_t)(const void *)((__x) + 1) - (size_t)(const void *)(__x) == 1)
 
 /* Set N bytes of S to C.  */
 #ifndef _HAVE_STRING_ARCH_memset
@@ -108,139 +107,135 @@ __STRING2_COPY_TYPE (8);
 #  define __mempcpy(dest, src, n) \
   (__extension__ (__builtin_constant_p (src) && __builtin_constant_p (n)      \
 		  && __string2_1bptr_p (src) && n <= 8			      \
-		  ? __mempcpy_small (dest, src, n)			      \
+		  ? __mempcpy_small (dest, __mempcpy_args (src), n)	      \
 		  : __mempcpy (dest, src, n)))
 /* In glibc we use this function frequently but for namespace reasons
    we have to use the name `__mempcpy'.  */
 #  define mempcpy(dest, src, n) __mempcpy (dest, src, n)
 
 #  if _STRING_ARCH_unaligned
-#   define __mempcpy_small(dest, src, srclen) \
-  (__extension__ ({ char *__dest = (char *) (dest);			      \
-		    switch (srclen)					      \
-		      {							      \
-		      case 1:						      \
-			*__dest++ = *(char *) src;			      \
-			break;						      \
-		      case 2:						      \
-			*((__uint16_t *) __dest) =			      \
-			  __STRING2_SMALL_GET16 (src, 0);		      \
-			__dest += 2;					      \
-			break;						      \
-		      case 3:						      \
-			*((__uint16_t *) __dest) =			      \
-			  __STRING2_SMALL_GET16 (src, 0);		      \
-			__dest += sizeof (__uint16_t);			      \
-			*__dest++ = ((char *) src)[2];			      \
-			break;						      \
-		      case 4:						      \
-			*((__uint32_t *) __dest) =			      \
-			  __STRING2_SMALL_GET32 (src, 0);		      \
-			__dest += 4;					      \
-			break;						      \
-		      case 5:						      \
-			*((__uint32_t *) __dest) =			      \
-			  __STRING2_SMALL_GET32 (src, 0);		      \
-			__dest += 4;					      \
-			*__dest++ = ((char *) src)[4];			      \
-			break;						      \
-		      case 6:						      \
-			*((__uint32_t *) __dest) =			      \
-			  __STRING2_SMALL_GET32 (src, 0);		      \
-			*((__uint16_t *) (__dest + 4)) =		      \
-			  __STRING2_SMALL_GET16 (src, 4);		      \
-			__dest += 6;					      \
-			break;						      \
-		      case 7:						      \
-			*((__uint32_t *) __dest) =			      \
-			  __STRING2_SMALL_GET32 (src, 0);		      \
-			*((__uint16_t *) (__dest + 4)) =		      \
-			  __STRING2_SMALL_GET16 (src, 4);		      \
-			__dest += 6;					      \
-			*__dest++ = ((char *) src)[6];			      \
-			break;						      \
-		      case 8:						      \
-			*((__uint32_t *) __dest) =			      \
-			  __STRING2_SMALL_GET32 (src, 0);		      \
-			*((__uint32_t *) (__dest + 4)) =		      \
-			  __STRING2_SMALL_GET32 (src, 4);		      \
-			__dest += 8;					      \
-			break;						      \
-		      }							      \
-		    __dest; }))
+#   define __mempcpy_args(src) \
+     ((char *) (src))[0], ((char *) (src))[2], ((char *) (src))[4],	      \
+     ((char *) (src))[6],						      \
+     __extension__ __STRING2_SMALL_GET16 (src, 0),			      \
+     __extension__ __STRING2_SMALL_GET16 (src, 4),			      \
+     __extension__ __STRING2_SMALL_GET32 (src, 0),			      \
+     __extension__ __STRING2_SMALL_GET32 (src, 4)
+__STRING_INLINE void *
+__mempcpy_small (void *__dest1,
+		 char __src0_1, char __src2_1, char __src4_1, char __src6_1,
+		 __uint16_t __src0_2, __uint16_t __src4_2,
+		 __uint32_t __src0_4, __uint32_t __src4_4,
+		 size_t __srclen)
+{
+  char *__dest = (char *) __dest1;
+  switch (__srclen)
+    {
+    case 1:
+      *__dest++ = __src0_1;
+      break;
+    case 2:
+      *((__uint16_t *) __dest) = __src0_2;
+      __dest += 2;
+      break;
+    case 3:
+      *((__uint16_t *) __dest) = __src0_2;
+      __dest += 2;
+      *__dest++ = __src2_1;
+      break;
+    case 4:
+      *((__uint32_t *) __dest) = __src0_4;
+      __dest += 4;
+      break;
+    case 5:
+      *((__uint32_t *) __dest) = __src0_4;
+      __dest += 4;
+      *__dest++ = __src4_1;
+      break;
+    case 6:
+      *((__uint32_t *) __dest) = __src0_4;
+      *((__uint16_t *) (__dest + 4)) = __src4_2;
+      __dest += 6;
+      break;
+    case 7:
+      *((__uint32_t *) __dest) = __src0_4;
+      *((__uint16_t *) (__dest + 4)) = __src4_2;
+      __dest += 6;
+      *__dest++ = __src6_1;
+      break;
+    case 8:
+      *((__uint32_t *) __dest) = __src0_4;
+      *((__uint32_t *) (__dest + 4)) = __src4_4;
+      __dest += 8;
+      break;
+    }
+  return (void *) __dest;
+}
 #  else
-#   define __mempcpy_small(dest, src, srclen) \
-  (__extension__ ({ char *__dest = (char *) (dest);			      \
-		    switch (srclen)					      \
-		      {							      \
-		      case 1:						      \
-			*__dest = '\0';					      \
-			break;						      \
-		      case 2:						      \
-			*((__STRING2_COPY_ARR2 *) __dest) =		      \
-			  ((__STRING2_COPY_ARR2)			      \
-			   { { ((__const char *) (src))[0],		      \
-			       ((__const char *) (src))[1] } });	      \
-			break;						      \
-		      case 3:						      \
-			*((__STRING2_COPY_ARR3 *) __dest) =		      \
-			  ((__STRING2_COPY_ARR3)			      \
-			   { { ((__const char *) (src))[0],		      \
-			       ((__const char *) (src))[1],		      \
-			       ((__const char *) (src))[2] } });	      \
-			break;						      \
-		      case 4:						      \
-			*((__STRING2_COPY_ARR4 *) __dest) =		      \
-			  ((__STRING2_COPY_ARR4)			      \
-			   { { ((__const char *) (src))[0],		      \
-			       ((__const char *) (src))[1],		      \
-			       ((__const char *) (src))[2],		      \
-			       ((__const char *) (src))[3] } });	      \
-			break;						      \
-		      case 5:						      \
-			*((__STRING2_COPY_ARR5 *) __dest) =		      \
-			  ((__STRING2_COPY_ARR5)			      \
-			   { { ((__const char *) (src))[0],		      \
-			       ((__const char *) (src))[1],		      \
-			       ((__const char *) (src))[2],		      \
-			       ((__const char *) (src))[3],		      \
-			       ((__const char *) (src))[4] } });	      \
-			break;						      \
-		      case 6:						      \
-			*((__STRING2_COPY_ARR6 *) __dest) =		      \
-			  ((__STRING2_COPY_ARR6)			      \
-			   { { ((__const char *) (src))[0],		      \
-			       ((__const char *) (src))[1],		      \
-			       ((__const char *) (src))[2],		      \
-			       ((__const char *) (src))[3],		      \
-			       ((__const char *) (src))[4],		      \
-			       ((__const char *) (src))[5] } });	      \
-			break;						      \
-		      case 7:						      \
-			*((__STRING2_COPY_ARR7 *) __dest) =		      \
-			  ((__STRING2_COPY_ARR7)			      \
-			   { { ((__const char *) (src))[0],		      \
-			       ((__const char *) (src))[1],		      \
-			       ((__const char *) (src))[2],		      \
-			       ((__const char *) (src))[3],		      \
-			       ((__const char *) (src))[4],		      \
-			       ((__const char *) (src))[5],		      \
-			       ((__const char *) (src))[6] } });	      \
-			break;						      \
-		      case 8:						      \
-			*((__STRING2_COPY_ARR8 *) __dest) =		      \
-			  ((__STRING2_COPY_ARR8)			      \
-			   { { ((__const char *) (src))[0],		      \
-			       ((__const char *) (src))[1],		      \
-			       ((__const char *) (src))[2],		      \
-			       ((__const char *) (src))[3],		      \
-			       ((__const char *) (src))[4],		      \
-			       ((__const char *) (src))[5],		      \
-			       ((__const char *) (src))[6],		      \
-			       ((__const char *) (src))[7] } });	      \
-			break;						      \
-		    }							      \
-		  __dest + (srclen); }))
+#   define __mempcpy_args(src) \
+     __extension__ ((__STRING2_COPY_ARR2)				      \
+      { { ((__const char *) (src))[0], ((__const char *) (src))[1] } }),      \
+     __extension__ ((__STRING2_COPY_ARR3)				      \
+      { { ((__const char *) (src))[0], ((__const char *) (src))[1],	      \
+	  ((__const char *) (src))[2] } }),				      \
+     __extension__ ((__STRING2_COPY_ARR4)				      \
+      { { ((__const char *) (src))[0], ((__const char *) (src))[1],	      \
+	  ((__const char *) (src))[2], ((__const char *) (src))[3] } }),      \
+     __extension__ ((__STRING2_COPY_ARR5)				      \
+      { { ((__const char *) (src))[0], ((__const char *) (src))[1],	      \
+	  ((__const char *) (src))[2], ((__const char *) (src))[3],	      \
+	  ((__const char *) (src))[4] } }),				      \
+     __extension__ ((__STRING2_COPY_ARR6)				      \
+      { { ((__const char *) (src))[0], ((__const char *) (src))[1],	      \
+	  ((__const char *) (src))[2], ((__const char *) (src))[3],	      \
+	  ((__const char *) (src))[4], ((__const char *) (src))[5] } }),      \
+     __extension__ ((__STRING2_COPY_ARR7)				      \
+      { { ((__const char *) (src))[0], ((__const char *) (src))[1],	      \
+	  ((__const char *) (src))[2], ((__const char *) (src))[3],	      \
+	  ((__const char *) (src))[4], ((__const char *) (src))[5],	      \
+	  ((__const char *) (src))[6] } }),				      \
+     __extension__ ((__STRING2_COPY_ARR8)				      \
+      { { ((__const char *) (src))[0], ((__const char *) (src))[1],	      \
+	  ((__const char *) (src))[2], ((__const char *) (src))[3],	      \
+	  ((__const char *) (src))[4], ((__const char *) (src))[5],	      \
+	  ((__const char *) (src))[6], ((__const char *) (src))[7] } })
+__STRING_INLINE void *
+__mempcpy_small (void *__dest1,
+		 __STRING2_COPY_ARR2 __src2, __STRING2_COPY_ARR3 __src3,
+		 __STRING2_COPY_ARR4 __src4, __STRING2_COPY_ARR5 __src5,
+		 __STRING2_COPY_ARR6 __src6, __STRING2_COPY_ARR7 __src7,
+		 __STRING2_COPY_ARR8 __src8, size_t __srclen)
+{
+  char *__dest = (char *) __dest1;
+  switch (__srclen)
+    {
+    case 1:
+      *__dest = '\0';
+      break;
+    case 2:
+      __extension__ *((__STRING2_COPY_ARR2 *) __dest) = __src2;
+      break;
+    case 3:
+      __extension__ *((__STRING2_COPY_ARR3 *) __dest) = __src3;
+      break;
+    case 4:
+      __extension__ *((__STRING2_COPY_ARR4 *) __dest) = __src4;
+      break;
+    case 5:
+      __extension__ *((__STRING2_COPY_ARR5 *) __dest) = __src5;
+      break;
+    case 6:
+      __extension__ *((__STRING2_COPY_ARR6 *) __dest) = __src6;
+      break;
+    case 7:
+      __extension__ *((__STRING2_COPY_ARR7 *) __dest) = __src7;
+      break;
+    case 8:
+      __extension__ *((__STRING2_COPY_ARR8 *) __dest) = __src8;
+      break;
+    }
+  return (void *) (__dest + __srclen);
+}
 #  endif
 # endif
 #endif
@@ -251,130 +246,122 @@ __STRING2_COPY_TYPE (8);
 # define strcpy(dest, src) \
   (__extension__ (__builtin_constant_p (src)				      \
 		  ? (__string2_1bptr_p (src) && strlen (src) + 1 <= 8	      \
-		     ? __strcpy_small (dest, src, strlen (src) + 1)	      \
+		     ? __strcpy_small (dest, __strcpy_args (src),	      \
+				       strlen (src) + 1)		      \
 		     : (char *) memcpy (dest, src, strlen (src) + 1))	      \
 		  : strcpy (dest, src)))
 
 # if _STRING_ARCH_unaligned
-#  define __strcpy_small(dest, src, srclen) \
-  (__extension__ ({ char *__dest = (char *) (dest);			      \
-		    switch (srclen)					      \
-		      {							      \
-		      case 1:						      \
-			*__dest = '\0';					      \
-			break;						      \
-		      case 2:						      \
-			*((__uint16_t *) __dest) =			      \
-			  __STRING2_SMALL_GET16 (src, 0);		      \
-			break;						      \
-		      case 3:						      \
-			*((__uint16_t *) __dest) =			      \
-			  __STRING2_SMALL_GET16 (src, 0);		      \
-			*(__dest + 2) = '\0';				      \
-			break;						      \
-		      case 4:						      \
-			*((__uint32_t *) __dest) =			      \
-			  __STRING2_SMALL_GET32 (src, 0);		      \
-			break;						      \
-		      case 5:						      \
-			*((__uint32_t *) __dest) =			      \
-			  __STRING2_SMALL_GET32 (src, 0);		      \
-			*(__dest + 4) = '\0';				      \
-			break;						      \
-		      case 6:						      \
-			*((__uint32_t *) __dest) =			      \
-			  __STRING2_SMALL_GET32 (src, 0);		      \
-			*((__uint16_t *) (__dest + 4)) =		      \
-			  __STRING2_SMALL_GET16 (src, 4);		      \
-			break;						      \
-		      case 7:						      \
-			*((__uint32_t *) __dest) =			      \
-			  __STRING2_SMALL_GET32 (src, 0);		      \
-			*((__uint16_t *) (__dest + 4)) =		      \
-			  __STRING2_SMALL_GET16 (src, 4);		      \
-			*(__dest + 6) = '\0';				      \
-			break;						      \
-		      case 8:						      \
-			*((__uint32_t *) __dest) =			      \
-			  __STRING2_SMALL_GET32 (src, 0);		      \
-			*((__uint32_t *) (__dest + 4)) =		      \
-			  __STRING2_SMALL_GET32 (src, 4);		      \
-			break;						      \
-		      }							      \
-		    __dest; }))
+#  define __strcpy_args(src) \
+     __extension__ __STRING2_SMALL_GET16 (src, 0),			      \
+     __extension__ __STRING2_SMALL_GET16 (src, 4),			      \
+     __extension__ __STRING2_SMALL_GET32 (src, 0),			      \
+     __extension__ __STRING2_SMALL_GET32 (src, 4)
+__STRING_INLINE char *
+__strcpy_small (char *__dest,
+		__uint16_t __src0_2, __uint16_t __src4_2,
+		__uint32_t __src0_4, __uint32_t __src4_4,
+		size_t __srclen)
+{
+  switch (__srclen)
+    {
+    case 1:
+      *__dest = '\0';
+      break;
+    case 2:
+      *((__uint16_t *) __dest) = __src0_2;
+      break;
+    case 3:
+      *((__uint16_t *) __dest) = __src0_2;
+      *(__dest + 2) = '\0';
+      break;
+    case 4:
+      *((__uint32_t *) __dest) = __src0_4;
+      break;
+    case 5:
+      *((__uint32_t *) __dest) = __src0_4;
+      *(__dest + 4) = '\0';
+      break;
+    case 6:
+      *((__uint32_t *) __dest) = __src0_4;
+      *((__uint16_t *) (__dest + 4)) = __src4_2;
+      break;
+    case 7:
+      *((__uint32_t *) __dest) = __src0_4;
+      *((__uint16_t *) (__dest + 4)) = __src4_2;
+      *(__dest + 6) = '\0';
+      break;
+    case 8:
+      *((__uint32_t *) __dest) = __src0_4;
+      *((__uint32_t *) (__dest + 4)) = __src4_4;
+      break;
+    }
+  return __dest;
+}
 # else
-#  define __strcpy_small(dest, src, srclen) \
-  (__extension__ ({ char *__dest = (char *) (dest);			      \
-		    switch (srclen)					      \
-		      {							      \
-		      case 1:						      \
-			*__dest = '\0';					      \
-			break;						      \
-		      case 2:						      \
-			*((__STRING2_COPY_ARR2 *) __dest) =		      \
-			  ((__STRING2_COPY_ARR2)			      \
-			   { { ((__const char *) (src))[0],		      \
-			       '\0' } });				      \
-			break;						      \
-		      case 3:						      \
-			*((__STRING2_COPY_ARR3 *) __dest) =		      \
-			  ((__STRING2_COPY_ARR3)			      \
-			   { { ((__const char *) (src))[0],		      \
-			       ((__const char *) (src))[1],		      \
-			       '\0' } });				      \
-			break;						      \
-		      case 4:						      \
-			*((__STRING2_COPY_ARR4 *) __dest) =		      \
-			  ((__STRING2_COPY_ARR4)			      \
-			   { { ((__const char *) (src))[0],		      \
-			       ((__const char *) (src))[1],		      \
-			       ((__const char *) (src))[2],		      \
-			       '\0' } });				      \
-			break;						      \
-		      case 5:						      \
-			*((__STRING2_COPY_ARR5 *) __dest) =		      \
-			  ((__STRING2_COPY_ARR5)			      \
-			   { { ((__const char *) (src))[0],		      \
-			       ((__const char *) (src))[1],		      \
-			       ((__const char *) (src))[2],		      \
-			       ((__const char *) (src))[3],		      \
-			       '\0' } });				      \
-			break;						      \
-		      case 6:						      \
-			*((__STRING2_COPY_ARR6 *) __dest) =		      \
-			  ((__STRING2_COPY_ARR6)			      \
-			   { { ((__const char *) (src))[0],		      \
-			       ((__const char *) (src))[1],		      \
-			       ((__const char *) (src))[2],		      \
-			       ((__const char *) (src))[3],		      \
-			       ((__const char *) (src))[4],		      \
-			       '\0' } });				      \
-			break;						      \
-		      case 7:						      \
-			*((__STRING2_COPY_ARR7 *) __dest) =		      \
-			  ((__STRING2_COPY_ARR7)			      \
-			   { { ((__const char *) (src))[0],		      \
-			       ((__const char *) (src))[1],		      \
-			       ((__const char *) (src))[2],		      \
-			       ((__const char *) (src))[3],		      \
-			       ((__const char *) (src))[4],		      \
-			       ((__const char *) (src))[5],		      \
-			       '\0' } });				      \
-			break;						      \
-		      case 8:						      \
-			*((__STRING2_COPY_ARR8 *) __dest) =		      \
-			  ((__STRING2_COPY_ARR8)			      \
-			   { { ((__const char *) (src))[0],		      \
-			       ((__const char *) (src))[1],		      \
-			       ((__const char *) (src))[2],		      \
-			       ((__const char *) (src))[3],		      \
-			       ((__const char *) (src))[4],		      \
-			       ((__const char *) (src))[5],		      \
-			       ((__const char *) (src))[6],		      \
-			       '\0' } });				      \
-			break;						      \
-		    }							      \
-		  __dest; }))
+#  define __strcpy_args(src) \
+     __extension__ ((__STRING2_COPY_ARR2)				      \
+      { { ((__const char *) (src))[0], '\0' } }),			      \
+     __extension__ ((__STRING2_COPY_ARR3)				      \
+      { { ((__const char *) (src))[0], ((__const char *) (src))[1],	      \
+	  '\0' } }),							      \
+     __extension__ ((__STRING2_COPY_ARR4)				      \
+      { { ((__const char *) (src))[0], ((__const char *) (src))[1],	      \
+	  ((__const char *) (src))[2], '\0' } }),			      \
+     __extension__ ((__STRING2_COPY_ARR5)				      \
+      { { ((__const char *) (src))[0], ((__const char *) (src))[1],	      \
+	  ((__const char *) (src))[2], ((__const char *) (src))[3],	      \
+	  '\0' } }),							      \
+     __extension__ ((__STRING2_COPY_ARR6)				      \
+      { { ((__const char *) (src))[0], ((__const char *) (src))[1],	      \
+	  ((__const char *) (src))[2], ((__const char *) (src))[3],	      \
+	  ((__const char *) (src))[4], '\0' } }),			      \
+     __extension__ ((__STRING2_COPY_ARR7)				      \
+      { { ((__const char *) (src))[0], ((__const char *) (src))[1],	      \
+	  ((__const char *) (src))[2], ((__const char *) (src))[3],	      \
+	  ((__const char *) (src))[4], ((__const char *) (src))[5],	      \
+	  '\0' } }),							      \
+     __extension__ ((__STRING2_COPY_ARR8)				      \
+      { { ((__const char *) (src))[0], ((__const char *) (src))[1],	      \
+	  ((__const char *) (src))[2], ((__const char *) (src))[3],	      \
+	  ((__const char *) (src))[4], ((__const char *) (src))[5],	      \
+	  ((__const char *) (src))[6], '\0' } })
+__STRING_INLINE char *
+__strcpy_small (char *__dest,
+		__STRING2_COPY_ARR2 __src2, __STRING2_COPY_ARR3 __src3,
+		__STRING2_COPY_ARR4 __src4, __STRING2_COPY_ARR5 __src5,
+		__STRING2_COPY_ARR6 __src6, __STRING2_COPY_ARR7 __src7,
+		__STRING2_COPY_ARR8 __src8, size_t __srclen)
+{
+  switch (__srclen)
+    {
+    case 1:
+      *__dest = '\0';
+      break;
+    case 2:
+      __extension__ *((__STRING2_COPY_ARR2 *) __dest) = __src2;
+      break;
+    case 3:
+      __extension__ *((__STRING2_COPY_ARR3 *) __dest) = __src3;
+      break;
+    case 4:
+      __extension__ *((__STRING2_COPY_ARR4 *) __dest) = __src4;
+      break;
+    case 5:
+      __extension__ *((__STRING2_COPY_ARR5 *) __dest) = __src5;
+      break;
+    case 6:
+      __extension__ *((__STRING2_COPY_ARR6 *) __dest) = __src6;
+      break;
+    case 7:
+      __extension__ *((__STRING2_COPY_ARR7 *) __dest) = __src7;
+      break;
+    case 8:
+      __extension__ *((__STRING2_COPY_ARR8 *) __dest) = __src8;
+      break;
+  }
+  return __dest;
+}
 # endif
 #endif
 
@@ -385,7 +372,8 @@ __STRING2_COPY_TYPE (8);
 #  define __stpcpy(dest, src) \
   (__extension__ (__builtin_constant_p (src)				      \
 		  ? (__string2_1bptr_p (src) && strlen (src) + 1 <= 8	      \
-		     ? __stpcpy_small (dest, src, strlen (src) + 1)	      \
+		     ? __stpcpy_small (dest, __stpcpy_args (src),	      \
+				       strlen (src) + 1)		      \
 		     : ((char *) __mempcpy (dest, src, strlen (src) + 1) - 1))\
 		  : __stpcpy (dest, src)))
 /* In glibc we use this function frequently but for namespace reasons
@@ -393,132 +381,123 @@ __STRING2_COPY_TYPE (8);
 #  define stpcpy(dest, src) __stpcpy (dest, src)
 
 #  if _STRING_ARCH_unaligned
-#   define __stpcpy_small(dest, src, srclen) \
-  (__extension__ ({ char *__dest = (char *) (dest);			      \
-		    switch (srclen)					      \
-		      {							      \
-		      case 1:						      \
-			*__dest = '\0';					      \
-			break;						      \
-		      case 2:						      \
-			*((__uint16_t *) __dest) =			      \
-			  __STRING2_SMALL_GET16 (src, 0);		      \
-			++__dest;					      \
-			break;						      \
-		      case 3:						      \
-			*((__uint16_t *) __dest) =			      \
-			  __STRING2_SMALL_GET16 (src, 0);		      \
-			__dest += sizeof (__uint16_t);			      \
-			*__dest = '\0';					      \
-			break;						      \
-		      case 4:						      \
-			*((__uint32_t *) __dest) =			      \
-			  __STRING2_SMALL_GET32 (src, 0);		      \
-			__dest += 3;					      \
-			break;						      \
-		      case 5:						      \
-			*((__uint32_t *) __dest) =			      \
-			  __STRING2_SMALL_GET32 (src, 0);		      \
-			__dest += 4;					      \
-			*__dest = '\0';					      \
-			break;						      \
-		      case 6:						      \
-			*((__uint32_t *) __dest) =			      \
-			  __STRING2_SMALL_GET32 (src, 0);		      \
-			*((__uint16_t *) (__dest + 4)) =		      \
-			  __STRING2_SMALL_GET16 (src, 4);		      \
-			__dest += 5;					      \
-			break;						      \
-		      case 7:						      \
-			*((__uint32_t *) __dest) =			      \
-			  __STRING2_SMALL_GET32 (src, 0);		      \
-			*((__uint16_t *) (__dest + 4)) =		      \
-			  __STRING2_SMALL_GET16 (src, 4);		      \
-			__dest += 6;					      \
-			*__dest = '\0';					      \
-			break;						      \
-		      case 8:						      \
-			*((__uint32_t *) __dest) =			      \
-			  __STRING2_SMALL_GET32 (src, 0);		      \
-			*((__uint32_t *) (__dest + 4)) =		      \
-			  __STRING2_SMALL_GET32 (src, 4);		      \
-			__dest += 7;					      \
-			break;						      \
-		      }							      \
-		    __dest; }))
+#   define __stpcpy_args(src) \
+     __extension__ __STRING2_SMALL_GET16 (src, 0),			      \
+     __extension__ __STRING2_SMALL_GET16 (src, 4),			      \
+     __extension__ __STRING2_SMALL_GET32 (src, 0),			      \
+     __extension__ __STRING2_SMALL_GET32 (src, 4)
+__STRING_INLINE char *
+__stpcpy_small (char *__dest,
+		__uint16_t __src0_2, __uint16_t __src4_2,
+		__uint32_t __src0_4, __uint32_t __src4_4,
+		size_t __srclen)
+{
+  switch (__srclen)
+    {
+    case 1:
+      *__dest = '\0';
+      break;
+    case 2:
+      *((__uint16_t *) __dest) = __src0_2;
+      ++__dest;
+      break;
+    case 3:
+      *((__uint16_t *) __dest) = __src0_2;
+      __dest += sizeof (__uint16_t);
+      *__dest = '\0';
+      break;
+    case 4:
+      *((__uint32_t *) __dest) = __src0_4;
+      __dest += 3;
+      break;
+    case 5:
+      *((__uint32_t *) __dest) = __src0_4;
+      __dest += 4;
+      *__dest = '\0';
+      break;
+    case 6:
+      *((__uint32_t *) __dest) = __src0_4;
+      *((__uint16_t *) (__dest + 4)) = __src4_2;
+      __dest += 5;
+      break;
+    case 7:
+      *((__uint32_t *) __dest) = __src0_4;
+      *((__uint16_t *) (__dest + 4)) = __src4_2;
+      __dest += 6;
+      *__dest = '\0';
+      break;
+    case 8:
+      *((__uint32_t *) __dest) = __src0_4;
+      *((__uint32_t *) (__dest + 4)) = __src4_4;
+      __dest += 7;
+      break;
+    }
+  return __dest;
+}
 #  else
-#   define __stpcpy_small(dest, src, srclen) \
-  (__extension__ ({ char *__dest = (char *) (dest);			      \
-		    switch (srclen)					      \
-		      {							      \
-		      case 1:						      \
-			*__dest = '\0';					      \
-			break;						      \
-		      case 2:						      \
-			*((__STRING2_COPY_ARR2 *) __dest) =		      \
-			  ((__STRING2_COPY_ARR2)			      \
-			   { { ((__const char *) (src))[0],		      \
-			       '\0' } });				      \
-			break;