[clang] [ARM, AArch64] Fix passing of structures with aligned base classes (PR #135564)

John McCall via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 15 10:23:23 PDT 2025


================
@@ -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);
----------------
rjmccall wrote:

I think it would be better to get that testing in this patch; it really should've been part of the original composite alignment patch, because the fact that some targets explicitly don't use it is a major aspect of the work.

https://github.com/llvm/llvm-project/pull/135564


More information about the cfe-commits mailing list