From cd68333f867f982974806f36c756ccb98a6b2778 Mon Sep 17 00:00:00 2001 From: Lionel Debroux Date: Tue, 17 May 2022 16:39:51 +0200 Subject: [PATCH] Get rid of memcpy(), as __builtin_memcpy() is globally smaller - and faster anyway - in all current and foreseeable occurrences. --- lib/string.c | 10 ---------- lib/string.h | 3 ++- 2 files changed, 2 insertions(+), 11 deletions(-) diff --git a/lib/string.c b/lib/string.c index 4781312..257b6ce 100644 --- a/lib/string.c +++ b/lib/string.c @@ -43,16 +43,6 @@ int memcmp(const void *s1, const void *s2, size_t n) return 0; } -void *memcpy(void *dest, const void *src, size_t n) -{ - char *d = (char *)dest, *s = (char *)src; - - for (size_t i = 0; i < n; i++) { - d[i] = s[i]; - } - return dest; -} - void *memmove(void *dest, const void *src, size_t n) { char *d = (char *)dest, *s = (char *)src; diff --git a/lib/string.h b/lib/string.h index b6e877a..b24e704 100644 --- a/lib/string.h +++ b/lib/string.h @@ -24,8 +24,9 @@ int memcmp(const void *s1, const void *s2, size_t n); * Copies n bytes from the memory area pointed to by src to the memory area * pointed to by dest and returns a pointer to dest. The memory areas must * not overlap. + * void *memcpy(void *dst, const void *src, size_t n); */ -void *memcpy(void *dst, const void *src, size_t n); +#define memcpy(d, s, n) __builtin_memcpy((d), (s), (n)) /** * Copies n bytes from the memory area pointed to by src to the memory area