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

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 26 16:15:38 PDT 2019


arsenm marked an inline comment as done.
arsenm added inline comments.


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


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

https://reviews.llvm.org/D63849





More information about the llvm-commits mailing list