[PATCH] D32018: [MVT][SVE] Scalable vector MVTs (2/3)

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: include/llvm/CodeGen/MachineValueType.h:31
   public:
-    enum SimpleValueType : int8_t {
+    enum SimpleValueType : int16_t {
       // Simple value types less than zero are considered extended value types.
----------------
I wonder if we should move INVALID_SIMPLE_VALUE_TYPE up to 0 and shift everything so we can keep this as a byte. Not sure if anything depends on Other being 0.


https://reviews.llvm.org/D32018