[PATCH] D133711: [Sema] Reject array element types whose alignments are larger than their sizes
    Reid Kleckner via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Wed Sep 14 09:54:08 PDT 2022
    
    
  
rnk added a comment.
I agree, there's no reason to change the i686 MSVC record layout algorithm. It was written, tested, done, it's fragile. We used to have some decent continuous integration testing for ABI issues like this, but as usual these things require care and feeding and it did not survive the passage of time.
This patch implements the new GCC behavior, right? Reject arrays when size is not a multiple of alignment? That sounds good to me.
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