[PATCH] D72737: [AMDGPU] Bundle loads before pre-RA scheduler

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 14 15:33:28 PST 2020


rampitec added a comment.

In D72737#1820814 <https://reviews.llvm.org/D72737#1820814>, @arsenm wrote:

> Is this just working around the post-RA scheduler? Can we finally replace the post-RA scheduler with misched?


Post RA scheduler in fact works pretty well. That is not really clear if we should replace it, then in any way we likely want a different set of heuristics than in pre-RA scheduler. The pre-RA shall deal with RP, ILP and clustering, where post-RA needs to deal with more subtle aspects of SQ. We just cannot address all of these (often opposite) heuristics in the same algorithm. But for sure it shall not undo clustering done pre-RA.


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

https://reviews.llvm.org/D72737





More information about the llvm-commits mailing list