[PATCH] D113150: Lift VLIWResourceModel, VLIWMachineScheduler, and ConvergingVLIWScheduler into CodeGen/VLIWMachineScheduler

[Matthias Braun via Phabricator via llvm-commits]

MatzeB added a comment.

FWIW: I like how this carefully extends the existing scheduler interfaces without affecting the non-VLIW cases!

I'll leave a detailed review to the users of this pass.



================
Comment at: llvm/lib/CodeGen/VLIWMachineScheduler.cpp:9-10
+//
+// MachineScheduler schedules machine instructions after phi elimination. It
+// preserves LiveIntervals so it can be invoked before register allocation.
+//
----------------
This seems copy&pasted from MachineScheduler.cpp and needs an update :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113150