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

Thu Apr 2 14:06:41 PDT 2020

sammccall added a comment.

In D77313#1957970 <https://reviews.llvm.org/D77313#1957970>, @sammccall wrote:

> 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.
>
>
> Sounds reasonable to me, I'll try that.

This alternative is in D77335 <https://reviews.llvm.org/D77335>, which is much simpler.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77313