craig.topper added inline comments.
================ Comment at: clang/lib/Headers/avx512fp16intrin.h:92 #define _mm512_setr_ph(__h1, __h2, __h3, __h4, __h5, __h6, __h7, __h8, __h9, \ __h10, __h11, __h12, __h13, __h14, __h15, __h16, __h17, \ ---------------- Can we drop the underscores on these macro arguments? Unless that would be inconsistent with the other setr implementations from sse/avx. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107946/new/ https://reviews.llvm.org/D107946 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits