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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
