[PATCH] D11160: Targets: commonize some stack realignment code

hfinkel at anl.gov hfinkel at anl.gov
Tue Jul 14 14:37:53 PDT 2015


hfinkel added a subscriber: hfinkel.
hfinkel added a comment.

> x86 almost: has an extra force-align-stack option, and is otherwise the same as the default implementation. Should this option be shared by all targets?


Yes, that sounds very useful.

> Should we decide to go with the breaking change, I think we should discuss on the mailing list too.


Is this a change for anyone other than Hexagon. Their check seems odd: Are they specifying the right stack alignment. We should find out from them.


http://reviews.llvm.org/D11160







More information about the llvm-commits mailing list