[PATCH] D109536: [AMDGPU] Switch PostRA sched to MachineSched

[Joe Nash via Phabricator via llvm-commits]

Joe_Nash added a comment.



In D109536#3001188 <https://reviews.llvm.org/D109536#3001188>, @foad wrote:

> In D109536#2993511 <https://reviews.llvm.org/D109536#2993511>, @foad wrote:
>
>> Can you give some rationale for the change in the commit message?
>
> Or failing that, leave a comment here with the rationale, just for the historical record?

This was my updated commit message. I didn't expect it to update the description on phab, but apparently it also didn't make it into the landed patch either. Arcanist did something I didn't expect.

[AMDGPU] Switch PostRA sched to MachineSched

The PostRA Machine scheduler is more flexible and extensible than
the old PostRA top-down list scheduler. In particular it will
make better use of cluster edges. Testing shows no regressions
on graphics workloads relative to the old scheduler.

GCNHazardRecognizer is still used as the PostRA sched hazard rec.
Updated tests for the new schedules.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109536