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

[Graham Hunter via Phabricator via llvm-commits]

huntergr 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.
----------------
craig.topper wrote:
> 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.
I'll try that as an experiment and see if anything falls over.


https://reviews.llvm.org/D32018