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

LuoYuanke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 23 23:32:34 PST 2021


LuoYuanke added inline comments.


================
Comment at: clang/lib/Headers/amxintrin.h:283
 
+typedef struct __tile1024bf16_str {
+  const unsigned short row;
----------------
pengfei wrote:
> Is there much value to differentiate the type? We are using the same AMX type in the builtins. What do you think? @LuoYuanke 
My first though is that we can reuse __tile1024i for bf16 tile for 2 reasons.
1. We don't access the element of the tile. 
2. The destination element of amx-int8 is int32 and the destination element of amx-bf16 is float32, the element size is the same.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97358



More information about the llvm-commits mailing list