[PATCH] D30626: MachineScheduler/ScheduleDAG: Add support for GetSubGraph

[Valery Pykhtin via Phabricator via llvm-commits]

vpykhtin added inline comments.


================
Comment at: include/llvm/CodeGen/ScheduleDAG.h:723
+    /// StartSU, else it is true.
+    std::vector<int> GetSubGraph(const SUnit &StartSU, const SUnit &TargetSU,
+                                 bool &Success);
----------------
Isn't it enough to return empty vector on failure, without error flag?


Repository:
  rL LLVM

https://reviews.llvm.org/D30626