RKSimon accepted this revision. RKSimon added a comment. This revision is now accepted and ready to land.
LGTM ================ Comment at: clang/lib/Headers/__wmmintrin_pclmul.h:25 /// \code -/// __m128i _mm_clmulepi64_si128(__m128i __X, __m128i __Y, const int __I); +/// __m128i _mm_clmulepi64_si128(__m128i X, __m128i Y, const int I); /// \endcode ---------------- Its annoying that we've never gotten https://github.com/llvm/llvm-project/issues/35297 fixed to have helped with these param name mismatches.... CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124469/new/ https://reviews.llvm.org/D124469 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits