[PATCH] D133711: [Sema] Reject array element types whose alignments are larger than their sizes
Akira Hatanaka via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Sep 13 15:46:40 PDT 2022
ahatanak added a comment.
No, I was thinking perhaps clang has to round up the size to the alignment when the target is 32-bit MSVC too. The code in https://godbolt.org/z/c1EzdYqPc is perfectly valid so I don't think we want to reject it, but we don't want to keep emitting buggy code either.
I was just wondering whether we'd run into problems if clang started behaving differently than MSVC (i.e., increase `T2`'s size to 8B from 5B).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D133711/new/
https://reviews.llvm.org/D133711
More information about the cfe-commits
mailing list