[PATCH] D31587: MachineScheduler/ScheduleDAG: Add support for getSUTopoIndex

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 3 15:25:13 PDT 2017


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

LGTM.

> Should the result be const as well ?

The result is passed by value so no need for const (it wouldn't change anything). You should make the `getSUTopoIndex()` method const though.


Repository:
  rL LLVM

https://reviews.llvm.org/D31587





More information about the llvm-commits mailing list