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

[Michael Jones via Phabricator via libc-commits]

michaelrj marked an inline comment as done.
michaelrj added inline comments.


================
Comment at: libc/src/string/memory_utils/op_x86.h:30
+#ifndef __AVX512F__
 #define _mm512_cmpneq_epi8_mask(A, B) 0
+#endif //  __AVX512F__
----------------
gchatelet wrote:
> This requires `__AVX512BW__`
> https://www.intel.com/content/www/us/en/docs/intrinsics-guide/index.html#text=_mm512_cmpneq_epi8_mask&ig_expand=1249
ah, yes. Fixed


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137868