[PATCH] D90822: [X86] use macros to split GFNI intrinsics into different kinds

[Craig Topper via Phabricator via cfe-commits]

craig.topper added a comment.

In D90822#2375372 <https://reviews.llvm.org/D90822#2375372>, @FreddyYe wrote:

> The fails are all unknown type errors on Windows, since those typedefs are declared in other header files.
> The error message goes like:
>
>   $ clang -march=tremont gfni.c
>   ......
>   ...\lib\clang\12.0.0\include\gfniintrin.h:129:37: error:
>         unknown type name '__mmask16'
>   _mm_mask_gf2p8mul_epi8(__m128i __S, __mmask16 __U, __m128i __A, __m128i __B)
>                                       ^
>   ...\lib\clang\12.0.0\include\gfniintrin.h:137:25: error:
>         unknown type name '__mmask16'
>   _mm_maskz_gf2p8mul_epi8(__mmask16 __U, __m128i __A, __m128i __B)
>                           ^
>   ......
>   ...\lib\clang\12.0.0\include\gfniintrin.h:159:43: error:
>         unknown type name '__m256i'
>   _mm256_maskz_gf2p8mul_epi8(__mmask32 __U, __m256i __A, __m256i __B)
>                                             ^
>   ...
>   fatal error: too many errors emitted, stopping now [-ferror-limit=]
>   20 errors generated.

Ok that makes sense. Maybe the easiest fix is just check that __AVXINTRIN_H or __AVX512BWINTRIN_H or __AVX512BWVLINTRIN_H is defined where there's a dependency.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90822