[PATCH] D86684: [NFC][Refactor] Add the SchedHeuristic for Scheduler to allow platform customizing the heuristics

[Qing Shan Zhang via Phabricator via llvm-commits]

steven.zhang added a comment.

In D86684#2241416 <https://reviews.llvm.org/D86684#2241416>, @foad wrote:

>> It is hard for target to add new heuristic now, as the Reason and tryCandidate() are maintained by base class.
>> It is hard for target to change the priority of each heuristic(i.e. prefer cluster over stall) as it is hard code by tryCandidate.
>
> I agree. For the AMDGPU scheduler I am very interested in latency and register pressure, and not very interested in the other heuristics. I look forward to be being able to tweak this in the target sched strategy.

Yes, PowerPC also has some specific requirement on the sched strategy.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86684/new/

https://reviews.llvm.org/D86684