[PATCH] D101657: [PowerPC] Handle inline assembly clobber of link regsiter

Nemanja Ivanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 12 12:04:54 PDT 2021


nemanjai accepted this revision.
nemanjai added a comment.
This revision is now accepted and ready to land.

LGTM other than the allocatable nit. Let's go with this fix for now to unblock the kernel build and we'll follow-up with a more complete fix for other reserved registers later.



================
Comment at: llvm/lib/Target/PowerPC/PPCRegisterInfo.td:413
+def LRRC : RegisterClass<"PPC", [i32], 32, (add LR)>;
+def LR8RC : RegisterClass<"PPC", [i64], 64, (add LR8)>;
+
----------------
Should these have the `let isAllocatable = 0` as well? Seems like they should.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101657



More information about the llvm-commits mailing list