[PATCH] D135930: [X86] Add AVX-NE-CONVERT instructions.

Wed Oct 19 23:19:53 PDT 2022

pengfei added inline comments.

================
Comment at: clang/lib/Headers/immintrin.h:257

+/* FIXME: Change these When _Float16 type is supported */
+#if defined(__AVXNECONVERT__) && defined(__AVX512FP16__)
----------------
FreddyYe wrote:
> pengfei wrote:
> > craig.topper wrote:
> > > Is this FIXME still relevant? Don't we support _Float16 with SSE2 now?
> > _Float16 is supported with SSE2, but maybe we need to move `__m128h`, `__m256h` out of avx512fp16intrin.h
> Yes. This is a redundant FIXME.
I have moved FP16/BF16 vector types out of original header files. rGe0fb01e9
There should be no dependency to FP16 and BF16 feature now.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135930