craig.topper added inline comments.

================
Comment at: clang/lib/Headers/avx512fp16intrin.h:953
+#define _mm512_cvt_roundpd_ph(A, R)                                            
\
+  (__m128h) __builtin_ia32_vcvtpd2ph512_mask(                                  
\
+      (__v8df)(A), (__v8hf)_mm_undefined_ph(), (__mmask8)(-1), (int)(R))
----------------
Put extra parentheses here to avoid repeating PR51324. Same for any macro that 
has a typecast of builtin result.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105265

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

Reply via email to