stevewan added inline comments.
================ Comment at: clang/lib/AST/RecordLayoutBuilder.cpp:1783 + if (FieldPacked) { + FieldAlign = UnpackedFieldAlign = 1; + } else if (!MaxFieldAlignment.isZero()) { ---------------- `UnpackedFieldAlign` is used to check if the packed attribute is unnecessary (-Wpacked). here the attribute is making a difference, we probably shouldn't set `FieldAlign = UnpackedFieldAlign`? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106900/new/ https://reviews.llvm.org/D106900 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits