[PATCH] D60748: Fix i386 struct and union parameter alignment

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 16 10:20:26 PDT 2020


rjmccall added a comment.

Oh, I see you just updated your patch months ago without ever mentioning that it was ready for review.

It sounds to me like GCC retroactively added a switch specifying which version of the ABI to follow on this point, somewhat confusingly called `-malign-data`.  That's probably the right move here for us, too, especially since FreeBSD says they'd like to use it.  That also means the condition of when to use your new logic will have to change; basically, we need a CodeGenOption for this that will default to the old ABI, and the driver will pass down a different default on Linux.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60748/new/

https://reviews.llvm.org/D60748





More information about the cfe-commits mailing list