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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to