[libc-commits] [PATCH] D136595: [libc] Switch to new implementation of mem* functions

Clement Courbet via Phabricator via libc-commits libc-commits at lists.llvm.org
Mon Oct 24 08:04:38 PDT 2022


courbet added inline comments.


================
Comment at: libc/src/string/memory_utils/memcpy_implementations.h:127
+#elif defined(LLVM_LIBC_ARCH_ARM)
+  return inline_memcpy_embedded_tiny(dst, src, count);
 #else
----------------
why is `ARM` always `embedded_tiny` ? Shouldn't that be orthogonal ? I could imagine some tiny aarch64 processors...


================
Comment at: libc/src/string/memory_utils/memset_implementations.h:108
+#elif defined(LLVM_LIBC_ARCH_ARM)
+  return inline_memset_embedded_tiny(dst, value, count);
+#else
----------------
ditto


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D136595/new/

https://reviews.llvm.org/D136595



More information about the libc-commits mailing list