[PATCH] D39719: [X86][AVX512] lowering kunpack intrinsic - clang part

2017-12-05 Thread jina via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC319777: [x86][AVX512] Lowering kunpack intrinsics to LLVM IR (authored by jina.nahias). Repository: rC Clang https://reviews.llvm.org/D39719 Files: lib/Headers/avx512bwintrin.h lib/Headers/avx512f

[PATCH] D39719: [X86][AVX512] lowering kunpack intrinsic - clang part

2017-12-05 Thread jina via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL319777: [x86][AVX512] Lowering kunpack intrinsics to LLVM IR (authored by jina.nahias). Changed prior to commit: https://reviews.llvm.org/D39719?vs=122617&id=125525#toc Repository: rL LLVM https://r

[PATCH] D39719: [X86][AVX512] lowering kunpack intrinsic - clang part

2017-12-05 Thread jina via Phabricator via cfe-commits
jina.nahias added a comment. ping https://reviews.llvm.org/D39719 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D39719: [X86][AVX512] lowering kunpack intrinsic - clang part

2017-11-13 Thread jina via Phabricator via cfe-commits
jina.nahias updated this revision to Diff 122617. https://reviews.llvm.org/D39719 Files: lib/Headers/avx512bwintrin.h lib/Headers/avx512fintrin.h test/CodeGen/avx512bw-builtins.c test/CodeGen/avx512f-builtins.c Index: test/CodeGen/avx512f-builtins.c =

[PATCH] D39719: [X86][AVX512] lowering kunpack intrinsic - clang part

2017-11-12 Thread jina via Phabricator via cfe-commits
jina.nahias added inline comments. Comment at: test/CodeGen/avx512f-builtins.c:6231 + // CHECK: bitcast <16 x i1> %{{.*}} to i16 + // CHECK: and i32 %{{.*}}, 255 + // CHECK: shl i32 %{{.*}}, 8 craig.topper wrote: > Does this really produce kunpackb in the back

[PATCH] D39719: [X86][AVX512] lowering kunpack intrinsic - clang part

2017-11-07 Thread Craig Topper via Phabricator via cfe-commits
craig.topper added inline comments. Comment at: test/CodeGen/avx512f-builtins.c:6231 + // CHECK: bitcast <16 x i1> %{{.*}} to i16 + // CHECK: and i32 %{{.*}}, 255 + // CHECK: shl i32 %{{.*}}, 8 Does this really produce kunpackb in the backend? The type promoti

[PATCH] D39719: [X86][AVX512] lowering kunpack intrinsic - clang part

2017-11-07 Thread jina via Phabricator via cfe-commits
jina.nahias added inline comments. Comment at: lib/Headers/avx512bwintrin.h:2045 { - return (__mmask64) __builtin_ia32_kunpckdi ((__mmask64) __A, -(__mmask64) __B); + return (__mmask64) (( __B & 0x) | ((__mmask64) __A << 32)); } RKSi

[PATCH] D39719: [X86][AVX512] lowering kunpack intrinsic - clang part

2017-11-07 Thread jina via Phabricator via cfe-commits
jina.nahias updated this revision to Diff 121899. https://reviews.llvm.org/D39719 Files: lib/Headers/avx512bwintrin.h lib/Headers/avx512fintrin.h test/CodeGen/avx512bw-builtins.c test/CodeGen/avx512f-builtins.c Index: test/CodeGen/avx512f-builtins.c =

[PATCH] D39719: [X86][AVX512] lowering kunpack intrinsic - clang part

2017-11-07 Thread Simon Pilgrim via Phabricator via cfe-commits
RKSimon added inline comments. Comment at: lib/Headers/avx512bwintrin.h:2045 { - return (__mmask64) __builtin_ia32_kunpckdi ((__mmask64) __A, -(__mmask64) __B); + return (__mmask64) (( __B & 0x) | ((__mmask64) __A << 32)); } Is this

[PATCH] D39719: [X86][AVX512] lowering kunpack intrinsic - clang part

2017-11-07 Thread jina via Phabricator via cfe-commits
jina.nahias created this revision. https://reviews.llvm.org/D39719 Files: lib/Headers/avx512bwintrin.h lib/Headers/avx512fintrin.h test/CodeGen/avx512bw-builtins.c test/CodeGen/avx512f-builtins.c Index: test/CodeGen/avx512f-builtins.c