[PATCH] D100368: [X86] Support some missing intrinsics

2021-04-20 Thread LiuChen via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG72e4bf12eec4: [X86] Support some missing intrinsics (authored by LiuChen3). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D100368: [X86] Support some missing intrinsics

2021-04-19 Thread Pengfei Wang via Phabricator via cfe-commits
pengfei accepted this revision. pengfei added a comment. This revision is now accepted and ready to land. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100368/new/ https://reviews.llvm.org/D100368

[PATCH] D100368: [X86] Support some missing intrinsics

2021-04-19 Thread LiuChen via Phabricator via cfe-commits
LiuChen3 added inline comments. Comment at: clang/test/CodeGen/X86/avx512f-builtins.c:10853 + +__m512i test_mm512_mask_i32logather_epi64(__m512i __v1_old, __mmask8 __mask, __m512i __index, void const *__addr) { + // CHECK-LABEL: @test_mm512_mask_i32logather_epi64

[PATCH] D100368: [X86] Support some missing intrinsics

2021-04-19 Thread LiuChen via Phabricator via cfe-commits
LiuChen3 updated this revision to Diff 338445. LiuChen3 added a comment. Fix format issue. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100368/new/ https://reviews.llvm.org/D100368 Files: clang/lib/Headers/avx512fintrin.h

[PATCH] D100368: [X86] Support some missing intrinsics

2021-04-19 Thread LiuChen via Phabricator via cfe-commits
LiuChen3 updated this revision to Diff 338443. LiuChen3 added a comment. 1. Rebase. 2. Adding _mm512_i32loscatter_epi64 and _mm512_mask_i32loscatter_epi64. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100368/new/ https://reviews.llvm.org/D100368

[PATCH] D100368: [X86] Support some missing intrinsics

2021-04-14 Thread Pengfei Wang via Phabricator via cfe-commits
pengfei added inline comments. Comment at: clang/lib/Headers/avx512fintrin.h:9611 +/// starting at location \a base_addr at packed 32-bit integer indices stored in +/// the lower half of \a vindex scaled by \a scale into dst using writemask +/// \a mask (elements are copied

[PATCH] D100368: [X86] Support some missing intrinsics

2021-04-14 Thread Craig Topper via Phabricator via cfe-commits
craig.topper added inline comments. Comment at: clang/lib/Headers/avx512fintrin.h:9593 +/// locations starting at location \a base_addr at packed 32-bit integer indices +/// stored in the lower half of \a vindex scaled by \a scale them in dst. +/// "scale them

[PATCH] D100368: [X86] Support some missing intrinsics

2021-04-13 Thread LiuChen via Phabricator via cfe-commits
LiuChen3 updated this revision to Diff 337301. LiuChen3 added a comment. Address Simon's comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100368/new/ https://reviews.llvm.org/D100368 Files: clang/lib/Headers/avx512fintrin.h

[PATCH] D100368: [X86] Support some missing intrinsics

2021-04-13 Thread Simon Pilgrim via Phabricator via cfe-commits
RKSimon added a comment. That's why I said "completeness" :) Comment at: clang/lib/Headers/avx512fintrin.h:9692 + (scale)) +/// Stores 8 packed double-precision (64-bit) floating-point elements in \a v1 +/// to memory locations starting at location \a

[PATCH] D100368: [X86] Support some missing intrinsics

2021-04-13 Thread Craig Topper via Phabricator via cfe-commits
craig.topper added a comment. In D100368#2685559 , @LiuChen3 wrote: > In D100368#2685189 , @RKSimon wrote: > >> Add _mm512_i32loscatter_epi64 and _mm512_mask_i32loscatter_epi64 for >> completeness? > > It seems

[PATCH] D100368: [X86] Support some missing intrinsics

2021-04-13 Thread LiuChen via Phabricator via cfe-commits
LiuChen3 added a comment. In D100368#2685189 , @RKSimon wrote: > Add _mm512_i32loscatter_epi64 and _mm512_mask_i32loscatter_epi64 for > completeness? It seems we doesn't support KNCNI in LLVM. Am I right? Repository: rG LLVM Github Monorepo

[PATCH] D100368: [X86] Support some missing intrinsics

2021-04-13 Thread Simon Pilgrim via Phabricator via cfe-commits
RKSimon added a comment. Add _mm512_i32loscatter_epi64 and _mm512_mask_i32loscatter_epi64 for completeness? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100368/new/ https://reviews.llvm.org/D100368

[PATCH] D100368: [X86] Support some missing intrinsics

2021-04-13 Thread LiuChen via Phabricator via cfe-commits
LiuChen3 created this revision. Herald added a subscriber: pengfei. LiuChen3 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Support for _mm512_i32logather_pd, _mm512_mask_i32logather_pd, _mm512_i32logather_epi64,