[PATCH] D92398: [AIX][XCOFF] emit traceback table for function in aix

[Hubert Tong via Phabricator via llvm-commits]

hubert.reinterpretcast added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCMachineFunctionInfo.cpp:67
+
+void PPCFunctionInfo::setParameterType(ParamType Type, unsigned RegNo) {
+  unsigned CopyParamType = ParameterType;
----------------
hubert.reinterpretcast wrote:
> Is `RegNo` a correct name for this parameter? It's not used like a register number (e.g., `PPC::R3`).
Perhaps this function should use and also be responsible for adjusting `FixedParamNum` and `FloatingPointParamNum`. The function should be renamed to something like `appendParameterType`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92398