================ @@ -1302,6 +1302,7 @@ ItaniumRecordLayoutBuilder::LayoutBase(const BaseSubobjectInfo *Base) { setSize(std::max(getSize(), Offset + Layout.getSize())); // Remember max struct/class alignment. + UnadjustedAlignment = std::max(UnadjustedAlignment, PreferredBaseAlign); ---------------- efriedma-quic wrote:
If other compilers agree, maybe it's not high-priority to update the spec... but it would be nice to have an explicit statement somewhere. ---- I was remembering that other targets have preferred alignment, but I guess on non-AIX targets that's only a thing outside of record layout. But for clarity, please do fix it so it doesn't interact with preferred alignment stuff. https://github.com/llvm/llvm-project/pull/135564 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits