[clang] [clang] Respect field alignment in layout compatibility of structs (PR #84313)
Vlad Serebrennikov via cfe-commits
cfe-commits at lists.llvm.org
Thu Mar 7 10:47:48 PST 2024
================
@@ -19185,7 +19185,8 @@ static bool isLayoutCompatible(ASTContext &C, EnumDecl *ED1, EnumDecl *ED2) {
/// Check if two fields are layout-compatible.
static bool isLayoutCompatible(ASTContext &C, FieldDecl *Field1,
- FieldDecl *Field2) {
+ FieldDecl *Field2,
+ bool IgnoreAlignment = false) {
----------------
Endilll wrote:
> but mostly because it is from the wrong 'direction'
This is spot-on observation. I wasn't sure which direction should I look at this from, but seeing both of you agree, I'm happy to change this.
https://github.com/llvm/llvm-project/pull/84313
More information about the cfe-commits
mailing list