[PATCH] D70138: [TargetLowering] Increase the storage size of NumRegistersForVT to allow the type break down for v256i1 and other types to be stored correctly

[Sanjay Patel via Phabricator via llvm-commits]

spatel accepted this revision.
spatel added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/include/llvm/CodeGen/TargetLowering.h:2744
   const TargetRegisterClass *RegClassForVT[MVT::LAST_VALUETYPE];
-  unsigned char NumRegistersForVT[MVT::LAST_VALUETYPE];
+  uint16_t NumRegistersForVT[MVT::LAST_VALUETYPE];
   MVT RegisterTypeForVT[MVT::LAST_VALUETYPE];
----------------
Not sure if it makes any difference jumping up to straight 32-bit "unsigned", but since we have an assert now, this should be ok.


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

https://reviews.llvm.org/D70138