[libc-commits] [PATCH] D92236: [LIBC] Add optimized memcpy routine for AArch64
Guillaume Chatelet via Phabricator via libc-commits
libc-commits at lists.llvm.org
Mon Dec 7 06:47:26 PST 2020
gchatelet added inline comments.
================
Comment at: libc/src/string/CMakeLists.txt:84
set(MEMCPY_SRC ${LIBC_SOURCE_DIR}/src/string/x86/memcpy.cpp)
+elseif(${LIBC_TARGET_MACHINE} STREQUAL "aarch64")
+ set(LIBC_STRING_TARGET_ARCH "aarch64")
----------------
avieira wrote:
> gchatelet wrote:
> > This change is not needed: it will be handled by the `else()` clause.
> > We have a special case for x86 to be able to support 32 and 64 bits architectures with the same code.
> I'm confused by this one. If I don't do this, then a call to __llvm_libc::memcpy in the benchmark will lead to the 'default' 'src/string/memcpy.cpp' implementation rather than the aarch64 one.
Oh my bad you're right, this code has changed (it used to work differently).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D92236/new/
https://reviews.llvm.org/D92236
More information about the libc-commits
mailing list