[PATCH] D69578: [AIX] Add support for lowering int, float and double formal arguments.

[Sean Fertile via Phabricator via llvm-commits]

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

Only a couple minor comments. Otherwise LGTM.



================
Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:6828
+  MachineFunction &MF = DAG.getMachineFunction();
+  PPCFunctionInfo *FuncInfo = MF.getInfo<PPCFunctionInfo>();
+  assert((CallConv == CallingConv::C || CallConv == CallingConv::Cold ||
----------------
Minor nit: This should be defined just before it use (`FuncInfo->setMinReservedArea(MinReservedArea);)`


================
Comment at: llvm/test/CodeGen/PowerPC/aix_cc_abi.ll:130
+
+; 32BIT:       liveins:
+; 32BIT-NEXT:  - { reg: '$r3', virtual-reg: '' }
----------------
Looks good. We should also have the same function but with a zeroext arg attribute and show that we don't end up with a clear instruction in this case. The new function doesn't need to be called. 


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

https://reviews.llvm.org/D69578