[PATCH] D63849: AMDGPU: Make s34 the FP register

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added inline comments.


================
Comment at: lib/Target/AMDGPU/SIFrameLowering.cpp:63
+      if (!MRI.isPhysRegUsed(Reg) && LiveRegs.available(MRI, Reg))
+        return Reg;
+    }
----------------
As far as I understand that can break limits set on the number of registers by an user.


================
Comment at: lib/Target/AMDGPU/SIMachineFunctionInfo.cpp:74
+
+    // TODO: Pick a high register, and shift down, similar to a kernel.wwwwwwwwwwww
+    FrameOffsetReg = AMDGPU::SGPR34;
----------------
wwwwwwwww


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

https://reviews.llvm.org/D63849