[PATCH] D142529: [MISched] Dump the execution trace of the schedule.

Francis Visoiu Mistrih via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 25 16:51:05 PST 2023


thegameg accepted this revision.
thegameg added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/CodeGen/MachineScheduler.cpp:1042
+  for (MachineInstr &MI : *this) {
+    if (SUnit *SU = getSUnit(&MI)) {
+      std::string NodeName("SU(");
----------------
can we put the !SU -> continue path first to avoid the extra indent? 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142529



More information about the llvm-commits mailing list