[PATCH] D39719: [X86][AVX512] lowering kunpack intrinsic - clang part
Simon Pilgrim via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Nov 7 05:11:00 PST 2017
RKSimon added inline comments.
================
Comment at: lib/Headers/avx512bwintrin.h:2045
{
- return (__mmask64) __builtin_ia32_kunpckdi ((__mmask64) __A,
- (__mmask64) __B);
+ return (__mmask64) (( __B & 0xFFFFFFFF) | ((__mmask64) __A << 32));
}
----------------
Is this right? The Intel docs says it should be:
```
k[31:0] := a[31:0]
k[63:32] := b[31:0]
k[MAX:64] := 0
```
Also, is the cast on __A necessary?
Same for the others.
https://reviews.llvm.org/D39719
More information about the cfe-commits
mailing list