[PATCH] D32621: TargetLowering: Add finalizeLowering() function; NFC

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 27 17:07:32 PDT 2017


arsenm added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp:595
 
-  if (TLI->hasCopyImplyingStackAdjustment(MF))
-    MFI.setHasCopyImplyingStackAdjustment(true);
-
-  // Freeze the set of reserved registers now that MachineFrameInfo has been
-  // set up. All the information required by getReservedRegs() should be
-  // available now.
-  MRI.freezeReservedRegs(*MF);
+  TLI->finalizeLowering(*MF);
 
----------------
Can we move this to after ExpandISelPseudos at least? It's a problem that reserved registers are frozen this early


Repository:
  rL LLVM

https://reviews.llvm.org/D32621





More information about the llvm-commits mailing list