[PATCH] D88540: [AMDGPU] Add amdgpu_gfx calling convention

Sebastian Neubauer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 4 01:05:21 PST 2020


sebastian-ne added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIISelLowering.cpp:1783-2265
+static void processPSInputArgs(SmallVectorImpl<ISD::InputArg> &Splits,
+                               CallingConv::ID CallConv,
+                               ArrayRef<ISD::InputArg> Ins, BitVector &Skipped,
+                               FunctionType *FType,
+                               SIMachineFunctionInfo *Info) {
   for (unsigned I = 0, E = Ins.size(), PSInputNum = 0; I != E; ++I) {
     const ISD::InputArg *Arg = &Ins[I];
----------------
arsenm wrote:
> Missing equivalent GlobalISel changes
What would that be?
The change here shouldn’t change behavior apart from checking the assert for amdgpu_gfx and I can’t find the same structure in AMDGPUCallLowering.cpp.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88540



More information about the llvm-commits mailing list