[PATCH] D105268: [X86] AVX512FP16 instructions enabling 5/6

Simon Pilgrim via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 17 02:40:27 PDT 2021


RKSimon added inline comments.


================
Comment at: clang/lib/Headers/avx512fp16intrin.h:2373
+      (__v32hf)(__m512h)(A), (__v32hf)(__m512h)(B), (__v32hf)(__m512h)(C),     \
+      (__mmask32)-1, (int)(R))
+
----------------
Add outer brackets to all the defines to prevent precedence issues:
```
#define _mm512_fmadd_round_ph(A, B, C, R)                                      \
 ((__m512h) __builtin_ia32_vfmaddph512_mask(                                   \
      (__v32hf)(__m512h)(A), (__v32hf)(__m512h)(B), (__v32hf)(__m512h)(C),     \
      (__mmask32)-1, (int)(R)))
```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105268



More information about the cfe-commits mailing list