[libc-commits] [PATCH] D135134: [libc] New version of the mem* framework

Guillaume Chatelet via Phabricator via libc-commits libc-commits at lists.llvm.org
Wed Oct 5 06:25:03 PDT 2022


gchatelet marked an inline comment as not done.
gchatelet added inline comments.


================
Comment at: libc/src/string/memory_utils/op_generic.h:127
+// Compute the type to handle an operation of Size bytes knowing that the
+// underlying platform only support native types up to MaxSize bytes.
+template <size_t Size, size_t MaxSize>
----------------
courbet wrote:
> Support is not necessarily a linear range of powers of `2`: e.g. x86+SSE which has 1,2,4,16
I've removed the support for `uint64_t` on 32bit platforms


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135134



More information about the libc-commits mailing list