probinson added inline comments.
================ Comment at: clang/lib/Headers/avx2intrin.h:456 +/// j := i*128 +/// result[j+31:j] := __a[j+63:j+32] - __a[j+31:j] +/// result[j+63:j+32] := __a[j+127:j+96] - __a[j+95:j+64] ---------------- craig.topper wrote: > Intel intrinsics guide says > > ``` > dst[31:0] := a[31:0] - a[63:32] > dst[63:32] := a[95:64] - a[127:96] > dst[95:64] := b[31:0] - b[63:32] > dst[127:96] := b[95:64] - b[127:96] > dst[159:128] := a[159:128] - a[191:160] > dst[191:160] := a[223:192] - a[255:224] > dst[223:192] := b[159:128] - b[191:160] > dst[255:224] := b[223:192] - b[255:224] > dst[MAX:256] := 0 > ``` > > So I think the operands are in the wrong order here? Words fail me. Also diagrams. I wanted the add and sub descriptions to look similar, and copy-pasted from add to sub without verifying the order. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D150114/new/ https://reviews.llvm.org/D150114 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits