[PATCH] D55555: [LLD][ELF][ARM] Add support for architecture v6m thunks

Peter Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 12 00:59:39 PST 2018


peter.smith added a comment.

It is true that the AAPCS only requires 8-byte stack alignment at procedure call boundaries. I have seen some interrupt handlers assuming 8 byte stack alignment, or at least not considering the case that it might not be, so I thought it better to be safe given that thunks are not easily visible to the programmer. I'm not wedded to the idea though so if you'd prefer I can remove the save of the extra register.


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

https://reviews.llvm.org/D55555





More information about the llvm-commits mailing list