[PATCH] D11538: Move the Target way of overriding DAG Scheduler to a target hook

[Andrew Trick]

atrick added a comment.

Let's see what Eric has to say about including TargetSelectionDAGInfo.h from TargetSubtargetInfo.h. I'm not sure that's the right layering.

Otherwise LGTM. Thanks.

When you push this, please send a note to llvm-dev telling people that their out of tree targets should not use the SchedulerRegistry to select a selection DAG scheduler. They should use your hook. It's not a big deal because we don't want people using SelectionDAG scheduler heuristics anyway, but it is a courtesy.


http://reviews.llvm.org/D11538