[PATCH] D97358: [X86] Support amx-bf16 intrinsic.

[Xiang Zhang via Phabricator via cfe-commits]

xiangzhangllvm added a comment.

+1 first, didn't see key problems.



================
Comment at: clang/lib/Headers/amxintrin.h:326
+__DEFAULT_FN_ATTRS_BF16
+static void __tile_tdpbf16ps(__tile1024i *dst, __tile1024i src1,
+                             __tile1024i src2) {
----------------
yubing wrote:
> Should we align this with "tile_dpbssd" by renaming it wth "tile_dpbf16ps"?
Yes, "t" already means "tile"


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97358