[PATCH] D31034: [X86][AVX512][Clang][Intrinsics] Adding missing intrinsics to Clang .

[Craig Topper via Phabricator via cfe-commits]

craig.topper added inline comments.


================
Comment at: lib/Headers/avx512fintrin.h:9633
 static __inline __m512i __DEFAULT_FN_ATTRS
+_mm512_set_epi8 (char e63, char e62, char e61, char e60, char e59,
+    char e58, char e57, char e56, char e55, char e54, char e53, char e52,
----------------
Isn't the convention to prefix all variable names in intrinsics with 2 leading underscores except when they are macros?


Repository:
  rL LLVM

https://reviews.llvm.org/D31034