[PATCH] D150114: [Headers][doc] Add "add/sub/mul" intrinsic descriptions to avx2intrin.h

Phoebe Wang via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 25 18:55:09 PDT 2023


pengfei accepted this revision.
pengfei added a comment.
This revision is now accepted and ready to land.

LGTM except for a possible typo.



================
Comment at: clang/lib/Headers/avx2intrin.h:412
+///    vectors of [16 x i16] and returns the lower 16 bits of each difference
+///    in an element of the [16 x i16] result (overflow is ignored).
+///    Differences from \a __a are returned in the lower 64 bits of each
----------------
underflow?


================
Comment at: clang/lib/Headers/avx2intrin.h:448
+///    vectors of [8 x i32] and returns the lower 32 bits of each difference in
+///    an element of the [8 x i31] result (overflow is ignored). Differences
+///    from \a __a are returned in the lower 64 bits of each 128-bit half of
----------------
typo or intended?


================
Comment at: clang/lib/Headers/avx2intrin.h:448
+///    vectors of [8 x i32] and returns the lower 32 bits of each difference in
+///    an element of the [8 x i31] result (overflow is ignored). Differences
+///    from \a __a are returned in the lower 64 bits of each 128-bit half of
----------------
pengfei wrote:
> typo or intended?
underflow.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D150114/new/

https://reviews.llvm.org/D150114



More information about the cfe-commits mailing list