[PATCH] [ARM] Align stack objects passed to functions
John Brawn
john.brawn at arm.com
Mon Mar 9 11:29:58 PDT 2015
In http://reviews.llvm.org/D7908#136838, @hfinkel wrote:
> You're currently doing this only for pointers that are captured as call arguments. Do you care about pointers captured in any other way (by having their address stored somewhere, or returned, for example)?
Maybe? The original patch did that as it just aligned everything that was used in a FrameIndex instruction. I guess the question is, how much do we care if we over-align things pointlessly? When it's an argument to a memcpy call it's definitely a good idea, but in an argument to some other call or when the address is stored somewhere it may turn out to be pointless (which is possibly an argument that I should be restricting this to only memcpy/move etc. calls).
REPOSITORY
rL LLVM
http://reviews.llvm.org/D7908
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list