[PATCH] D36272: [CodeGen][x86_64] Enable 'force_align_arg_pointer' attribute at x86_64

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 24 11:07:39 PDT 2017

erichkeane added a comment.

Are we SURE the stack alignment for this type is supposed to be 16 bit as well?  I didn't see any discussion about it in the email conversation.

I have very little understanding of this attribute, but I would (perhaps naiively) presume that it would be different on 64 bit targets.

Additionally, there is likely value to split the test run-line into 2, one that specifies 64 bit Windows and one that is 64 bit Linux.  The value being that they validate two different code paths (whereas 32 is the same code path).


More information about the cfe-commits mailing list