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

[Michael Jones via Phabricator via libc-commits]

michaelrj added a comment.

I'm going to land this as is for now, but I have identified a minor issue: When building variants of the memory functions that don't need AVX512BW, it doesn't seem to set the compiler flag for it. This causes a lot of warnings when building memcpy (for example) that boil down to "the macro `_mm512_cmpneq_epi8_mask` is being redefined". I don't think this will cause any real issues since that builtin isn't used for that function, but it will cause some noise in the logs.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137868