[llvm] [RISCV] Share ArgGPRs array between SelectionDAG and GISel. (PR #74152)
Craig Topper via llvm-commits
llvm-commits at lists.llvm.org
Sun Dec 3 20:25:15 PST 2023
================
@@ -986,6 +986,9 @@ bool CC_RISCV_FastCC(const DataLayout &DL, RISCVABI::ABI ABI, unsigned ValNo,
bool CC_RISCV_GHC(unsigned ValNo, MVT ValVT, MVT LocVT,
CCValAssign::LocInfo LocInfo, ISD::ArgFlagsTy ArgFlags,
CCState &State);
+
+ArrayRef<MCPhysReg> getArgGPRs();
----------------
topperc wrote:
> What about putting this function in `RISCVRegisterInfo.h/cpp`, no need to be a class method but a function in `RISCV` namespace.
It's not a class method in this patch.
Shouldn't it stay with other calling convention functions that are the primary users?
https://github.com/llvm/llvm-project/pull/74152
More information about the llvm-commits
mailing list