[PATCH] D76882: [AMDGPU] Implement CFI for non-kernel functions

[Christudasan Devadasan via Phabricator via llvm-commits]

cdevadas added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIFrameLowering.cpp:42
 
+static ArrayRef<MCPhysReg> getAllVGPRs(const GCNSubtarget &ST,
+                                       const MachineFunction &MF) {
----------------
I think it is good to move getAllSGPRs and getAllVGPRs into SIRegisterInfo.cpp.
There is another function getAllVGPR32 which does the same thing as getAllVGPRs.
We can unify these instances.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76882