[PATCH] D125164: [X86] Fix some signedness errors in x86 headers

[Simon Pilgrim via Phabricator via cfe-commits]

RKSimon created this revision.
RKSimon added reviewers: pengfei, craig.topper.
Herald added a subscriber: StephenFan.
Herald added a project: All.
RKSimon requested review of this revision.
Herald added a project: clang.

Another step toward enabling -Wsystem-headers testing across all x86 headers

Fix a number of cases where the arg / return value signedness doesn't match the C/C++ intrinsic.

So far I've just added explicit casts as necessary, but we might want to address some of the mismatches directly:

e.g.

  _mm512_srli_epi16(__m512i __A, unsigned int __B)
  _mm512_mask_srli_epi16(__m512i __W, __mmask32 __U, __m512i __A, unsigned int __B)
  _mm512_maskz_srli_epi16(__mmask32 __U, __m512i __A, int __B)  // <---- unsigned int?


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D125164

Files:
  clang/lib/Headers/avx512bwintrin.h
  clang/lib/Headers/avx512fintrin.h
  clang/lib/Headers/avx512vlbf16intrin.h
  clang/lib/Headers/avx512vlbwintrin.h
  clang/lib/Headers/avx512vlintrin.h
  clang/lib/Headers/bmiintrin.h
  clang/lib/Headers/cetintrin.h
  clang/lib/Headers/immintrin.h
  clang/lib/Headers/rdseedintrin.h
  clang/lib/Headers/rtmintrin.h
  clang/test/CodeGen/X86/avx512bw-builtins.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D125164.427843.patch
Type: text/x-patch
Size: 26420 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220507/eb51365c/attachment-0001.bin>