[PATCH] D77313: [AST] Allow VectorTypes of 1-256 elements, and powers of two up to 2**31.

[Sam McCall via Phabricator via cfe-commits]

sammccall added a comment.

In D77313#1957690 <https://reviews.llvm.org/D77313#1957690>, @efriedma wrote:

> I think I would rather just pay the extra 8 bytes per VectorType, and expand this to support all vector types supported by LLVM.  It's not like we allocate enough VectorTypes for it to matter, anyway.


llvm::VectorType seems to accept arbitrary `unsigned` size, so IIUC you'd suggest putting just putting `unsigned NumElements` in `VectorType`, which would cost 4 bytes and another 4 in padding. 
Sounds reasonable to me, I'll try that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77313