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

[Matthias Braun via Phabricator via llvm-commits]

MatzeB 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);
 
----------------
arsenm wrote:
> Can we move this to after ExpandISelPseudos at least? It's a problem that reserved registers are frozen this early
I think that should work, we certainly could try. Maybe rename `ExpandISelPseudos` to `FinalizeISel` then?

However I'd like to do that in a separate patch then, to avoid complications when backporting this to a production branches.


Repository:
  rL LLVM

https://reviews.llvm.org/D32621