[PATCH] D79706: [CodeGen][BFloat] Add bfloat MVT type

Ties Stuij via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 27 10:17:30 PDT 2020


stuij marked 2 inline comments as done.
stuij added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/ValueTypes.td:166
+def nxv8bf16 : ValueType<128, 131>; // n x  8 x bf16 vector value
+def nxv1bf32 : ValueType<32 , 132>; // n x  1 x f32 vector value
+def nxv2f32  : ValueType<64 , 133>; // n x  2 x f32 vector value
----------------
stuij wrote:
> rogfer01 wrote:
> > @stuij shouldn't this still be `nxv1f32`?
> Ai, thanks so much Roger!
> Fixing.
Fixed by: [CodeGen] fix typo `def nxv1bf32` -> `def nxv1f32`
hash: 29f8056b54ea5ea6b333e3b8f11de2cc327d1421

thanks again
(tell me if a quick non-phab-backed patch was inappropriate here)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79706





More information about the llvm-commits mailing list