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

Sean Fertile via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 18 12:39:37 PST 2019


sfertile added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:6886
+  // Area that is at least reserved in the caller of this function.
+  unsigned MinReservedArea =
+      std::max(CCInfo.getNextStackOffset(), LinkageSize + MinParameterSaveArea);
----------------
Since you have allocated the linkage area and the parameter save area `CCInfo.getNextStackOffset()` will be at least `LinkageSize + MinParameterSaveArea`, so you can omit taking the max.


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

https://reviews.llvm.org/D69578





More information about the llvm-commits mailing list