commit ec737893f8422d70735a604b46bb0cb250ba5ce0
author Guillaume Chatelet <gchatelet@google.com> Tue Sep 15 20:48:08 2020 +0000
committer Copybara-Service <copybara-worker@google.com> Thu Oct 29 11:46:22 2020 -0700
tree 6b13b27f91ea85e2c3494fd85dec0f9406c33ca6
parent 966cc1cc2631cd7eb8ca94e9f1e367d3fc4280ca

[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.

GitOrigin-RevId: c3fd2a50ba1395b6c2240f6a688c6a1aa975a1fe

src/string/memcpy.cpp

diff --git a/src/string/memcpy.cpp b/src/string/memcpy.cpp
index a805671..00d66ea 100644
--- a/src/string/memcpy.cpp
+++ b/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)

src/string/x86/memcpy.cpp

diff --git a/src/string/x86/memcpy.cpp b/src/string/x86/memcpy.cpp
index 811ce51..2e2148e 100644
--- a/src/string/x86/memcpy.cpp
+++ b/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)