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

[Jessica Clarke via Phabricator via llvm-commits]

jrtc27 added inline comments.


================
Comment at: llvm/include/llvm/Support/MachineValueType.h:288
       // This value must be a multiple of 32.
       MAX_ALLOWED_VALUETYPE = 192,
 
----------------
We're starting to creep awfully close to this. Downstream we have 5 extra MVTs (though could get away with just 3), and Arm's Morello fork of CHERI LLVM adds another one (but we should really have more than one were we to have it in CHERI LLVM). Is there a plan for what to do when this limit finally gets hit? Because the slow but steady explosion of vector types is causing me some amount of worry for us downstream...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125247