[PATCH] D64908: Fix and test inter-procedural register allocation for ARM

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 2 03:07:53 PDT 2019


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

LGTM. I think this is a sensible way to deal with r12. It is a bit special.



================
Comment at: llvm/include/llvm/CodeGen/TargetRegisterInfo.h:453
+  /// sequences of long-branch veneers.
+  virtual const ArrayRef<MCPhysReg>
+  getIntraCallClobberedRegs(const MachineFunction *MF) const {
----------------
Nit: is it worth making an ArrayRef const?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D64908





More information about the llvm-commits mailing list