[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