[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