[PATCH] D62363: [X86] Enable intrinsics that convert float and bf16 data to each other

[Kan Shengchen via Phabricator via cfe-commits]

skan marked an inline comment as done.
skan added inline comments.


================
Comment at: lib/Headers/avx512bf16intrin.h:37
+///
+/// This intrinsic corresponds to the <c> EmitX86CvtBF16ToFloatExpr </c>
+/// function.
----------------
craig.topper wrote:
> This needs to be a comment that's useful to user's of the compiler not compiler developers. It can't reference an implementation function inside the compiler
i can not think out a comment that 's needed  by users since other comments has explained this function clearly, can i just remove this comment?


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

https://reviews.llvm.org/D62363