[PATCH] D132342: [X86][AVX512FP16] Relax limitation to AVX512FP16 intrinsics. NFCI

Simon Pilgrim via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 22 11:10:59 PDT 2022


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

LGTM



================
Comment at: clang/lib/Headers/immintrin.h:223
+#if !(defined(_MSC_VER) || defined(__SCE__)) || __has_feature(modules) ||      \
+    (defined(__AVX512VL__) && defined(__AVX512FP16__))
 #include <avx512vlfp16intrin.h>
----------------
(style) - other uses in this header put the defined(__AVX512VL__) after:
```
(defined(__AVX512FP16__) && defined(__AVX512VL__))
```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132342



More information about the cfe-commits mailing list