[libc-commits] [PATCH] D137868: [libc] Fix builtin definition for memory functions

[Guillaume Chatelet via Phabricator via libc-commits]

gchatelet accepted this revision.
gchatelet added a comment.
This revision is now accepted and ready to land.

LGTM just make sure to keep `AVX512F` for `memmove`, `memcpy`, `memset` and `bzero`.



================
Comment at: libc/src/string/CMakeLists.txt:385
   add_bzero(bzero_x86_64_opt_avx2   COMPILE_OPTIONS -march=haswell        REQUIRE AVX2)
-  add_bzero(bzero_x86_64_opt_avx512 COMPILE_OPTIONS -march=skylake-avx512 REQUIRE AVX512F)
+  add_bzero(bzero_x86_64_opt_avx512 COMPILE_OPTIONS -march=skylake-avx512 REQUIRE AVX512BW)
   add_bzero(bzero_opt_host          COMPILE_OPTIONS ${LIBC_COMPILE_OPTIONS_NATIVE})
----------------
Only `bcmp` and `memcmp` require `AVX512BW`, the other ones only require `AVX512F`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137868