[PATCH] D29881: [PPC] Reduce stack frame size by allocating parameter area on an on-demand basis for ELFv2 ABI

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 2 06:46:13 PST 2017


hfinkel accepted this revision.
hfinkel added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D29881#690346, @inouehrs wrote:

> LowerFormalArguments_64SVR4 already includes the same analysis. So, LowerCall_64SVR4 and LowerFormalArguments_64SVR4 have been inconsistent (LowerCall_64SVR4 is more conservative than LowerFormalArguments_64SVR4).
>  What this patch does is making LowerCall_64SVR4 consistent with LowerFormalArguments_64SVR4.


Okay, please note that in the commit message. LGTM.


https://reviews.llvm.org/D29881





More information about the llvm-commits mailing list