[PATCH] D75126: [AIX] Extend integer arguments to register width when passed in stack memory.

Chris Bowler via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 25 08:39:20 PST 2020


cebowleratibm created this revision.
cebowleratibm added reviewers: ZarkoCA, sfertile, hubert.reinterpretcast.
Herald added subscribers: llvm-commits, jsji, kbarton, hiraditya, nemanjai.
Herald added a project: LLVM.

This is a follow up to the previous patch: [AIX] Implement caller arguments passed in stack memory. https://reviews.llvm.org/rGb373ec8ce76a00cd079f70d108b13129c3a4a7b9

This corrects a defect in AIX 64-bit where an i32 is written to the stack with stw (4 bytes) rather than the expected std (8 bytes.)  Integer arguments pass on the stack as images of their register representation.

I also took the opportunity to tidy up some of the calling convention AIX tests I added in my last commit.  This patch adds the missed assembly expected output for the stack arg int case, which would have caught this problem.  I also took the opportunity to reorder some of the calling convention stack arg tests into a more logical order to make them more readable.


https://reviews.llvm.org/D75126

Files:
  llvm/lib/Target/PowerPC/PPCISelLowering.cpp
  llvm/test/CodeGen/PowerPC/aix-cc-abi.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D75126.246479.patch
Type: text/x-patch
Size: 24716 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200225/a26b6811/attachment.bin>


More information about the llvm-commits mailing list