[PATCH] D128393: [RISCV] Add macrofusion infrastructure and one example usage.

[Philip Reames via Phabricator via llvm-commits]

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

LGTM w/one required change.



================
Comment at: llvm/lib/Target/RISCV/RISCVTargetMachine.cpp:147
+  createMachineScheduler(MachineSchedContext *C) const override {
+    ScheduleDAGMILive *DAG = createGenericSchedLive(C);
+    const RISCVSubtarget &ST = C->MF->getSubtarget<RISCVSubtarget>();
----------------
I think you want this to be returned only if macro enabled.  Or at least, unconditionally enabling a machine scheduler seems like it should be a separate change.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128393