commit c3fd2a50ba1395b6c2240f6a688c6a1aa975a1fe
author Guillaume Chatelet <gchatelet@google.com> Tue Sep 15 20:48:08 2020 +0000
committer Guillaume Chatelet <gchatelet@google.com> Tue Sep 15 20:48:27 2020 +0000
tree a53d046a079c10ce58d4381f4a53083cd85151f8
parent 7d26d6a1b062f7ce820b02b39d102d5f8f15fa5f

[libc] Remove special case for 8 and 16 bytes

They don't seem to gain much in real apps and its better to favor less branches and smaller code.

libc/src/string/memcpy.cpp

diff --git a/libc/src/string/memcpy.cpp b/libc/src/string/memcpy.cpp
index a805671..00d66ea 100644
--- a/libc/src/string/memcpy.cpp
+++ b/libc/src/string/memcpy.cpp
@@ -44,12 +44,8 @@
     return CopyBlock<4>(dst, src);
   if (count < 8)
     return CopyBlockOverlap<4>(dst, src, count);
-  if (count == 8)
-    return CopyBlock<8>(dst, src);
   if (count < 16)
     return CopyBlockOverlap<8>(dst, src, count);
-  if (count == 16)
-    return CopyBlock<16>(dst, src);
   if (count < 32)
     return CopyBlockOverlap<16>(dst, src, count);
   if (count < 64)

libc/src/string/x86/memcpy.cpp

diff --git a/libc/src/string/x86/memcpy.cpp b/libc/src/string/x86/memcpy.cpp
index 811ce51..2e2148e 100644
--- a/libc/src/string/x86/memcpy.cpp
+++ b/libc/src/string/x86/memcpy.cpp
@@ -59,12 +59,8 @@
     return CopyBlock<4>(dst, src);
   if (count < 8)
     return CopyBlockOverlap<4>(dst, src, count);
-  if (count == 8)
-    return CopyBlock<8>(dst, src);
   if (count < 16)
     return CopyBlockOverlap<8>(dst, src, count);
-  if (count == 16)
-    return CopyBlock<16>(dst, src);
   if (count < 32)
     return CopyBlockOverlap<16>(dst, src, count);
   if (count < 64)