[PATCH] D138708: [SROA] Assert the AllocSize of i8 to be 1

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a subscriber: nlopes.
lebedev.ri added a comment.

**I** would say that backend/front-end is essentially non-conforming
as far as LLVM is concerned - there is a very hard sentiment that
LLVM should only support the cases where byte is 8 bits,
which is essentially not the case here. If this doesn't hold,
e.g. the "simple GEPs" proposal, which does this same expansion,
goes out of the window.

I would suggest posting to discuss.

TBN, this wouldn't be a problem if we had a true `byte` type, @nlopes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138708