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

Reply via email to