NodeInCycleConstraint and DifferenceConstraint should use node IDs rather than pointers

Currently, the NodeInCycleConstraint and DifferenceConstraint passed into the scheduler use node pointers, which means that if a scheduling pass creates or destroys nodes, the pointers could be invalidated (or worse, the pointer could be silently reused). The correct way to handle this is to use node IDs instead.