[PATCH] D133634: [clang] Allow vector of BitInt

[Eli Friedman via Phabricator via cfe-commits]

efriedma added a comment.

I think the worst of the backend issues with vectors of non-byte-size integers have been resolved?  Work on that has progressed slowly for a very long time.  There have been a constant stream of issues.  (See, for example, D129164 <https://reviews.llvm.org/D129164>.)  But LangRef does define the memory layout, and CodeGen does essentially implement that layout.

It's not clear to me we actually want to use that layout for vectors of non-power-of-two types.  Load/store for bit-packed vectors isn't efficient.

Maybe we can just enable this specifically for types which are a power of two multiple of 8 for now (where there's one obvious layout), and consider the other cases later?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133634