pengfei added inline comments.

================
Comment at: clang/lib/Headers/avxifmaintrin.h:36
+/// __m128i
+/// _mm_madd52hi_avx_epu64 (__m128i __X, __m128i __Y, __m128i __Z)
+/// \endcode
----------------
FreddyYe wrote:
> pengfei wrote:
> > Should we provide unified intrinsic `_mm_madd52hi_epu64` like AVXVNNI?
> Good reminding. You mean adding them in intrinsic guide, right?
I mean define with macro first like this: 
https://github.com/llvm/llvm-project/blob/main/clang/lib/Headers/avx512vlvnniintrin.h#L38-L39


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135932

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

Reply via email to