Page MenuHomePhabricator

[Pipeliner] Use the Index from Topo instead of relying on NodeNum. (NFC)
ClosedPublic

Authored by sgundapa on Oct 10 2018, 12:05 PM.

Details

Summary

In future, if we may add any new DAG mutations other than artificial dependencies,
the NodeNum may not be valid. Instead the index from topological schedule DAG can be
used as long as we update it with the DAG change.

Diff Detail

Repository
rL LLVM

Event Timeline

sgundapa created this revision.Oct 10 2018, 12:05 PM

There is no test case associated with this patch as I have not found one yet.

bcahoon accepted this revision.Oct 11 2018, 9:33 AM
This revision is now accepted and ready to land.Oct 11 2018, 9:33 AM
This revision was automatically updated to reflect the committed changes.