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

Qiu Chaofan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun May 16 22:57:42 PDT 2021


qiucf added a comment.

I think if the intention is to re-use some generic code in `tryCandidate`, can we use a lighter approach? For example, make return value type of `tryCandidate` from `void` to `bool`, so that target override can call `GenericScheduler::tryCandidate`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86684



More information about the llvm-commits mailing list