[PATCH] D125018: [ValueTypes] Define MVTs for v128i2/v64i4 as well as i2 and i4.

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

What SelectionDAG support is needed for i2/i4?



================
Comment at: llvm/include/llvm/CodeGen/ValueTypes.td:29
 
-def bf16    : ValueType<16,   8>;  // 16-bit brain floating point value
-def f16     : ValueType<16,   9>;  // 16-bit floating point value
+def bf16    : ValueType<16,  8>;  // 16-bit brain floating point value
+def f16     : ValueType<16,  9>;  // 16-bit floating point value
----------------
Don't change this formatting


================
Comment at: llvm/utils/TableGen/CodeGenTarget.cpp:59
   case MVT::i1:       return "MVT::i1";
+  case MVT::i2:
+    return "MVT::i2";
----------------
Keep the formatting consistent with existing code


================
Comment at: llvm/utils/TableGen/CodeGenTarget.cpp:96
+  case MVT::v128i2:
+    return "MVT::v128i2";
+  case MVT::v64i4:
----------------
Same here


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

https://reviews.llvm.org/D125018