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