[PATCH] D101769: [ValueTypes] Add MVTs for v256i16 and v256f16

[Fraser Cormack via Phabricator via llvm-commits]

frasercrmck added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/ValueTypes.td:185
+def nxv32f16 : ValueType<512, 147>;  // n x 32 x  f16 vector value
+def nxv1bf16 : ValueType<16,  148>;  // n x  1 x bf16 vector value
+def nxv2bf16 : ValueType<32,  149>;  // n x  2 x bf16 vector value
----------------
craig.topper wrote:
> Should we add blank lines between different element widths here too?
Ah yeah I missed that one, thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101769