[PATCH] D138708: [SROA] Assert the AllocSize of i8 to be 1
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 25 09:36:50 PST 2022
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
More information about the llvm-commits
mailing list