[PATCH] D73386: [AMDGPU] Attempt to reschedule withou clustering

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added a comment.

In D73386#1839876 <https://reviews.llvm.org/D73386#1839876>, @kerbowa wrote:

> This tries scheduling with all mutations disabled including macro fusion right?


Yes. Macrofusion shall have the same impact on the pressure as clustering. It is a clustering in a sense.

> Instead of adding another scheduling pass could you try always disabling mutations for the first pass, and have them enabled for the second pass with a fallback if we drop occupancy?

I though about it but have chosen not to. It will effectively double scheduling time. Note I am only rescheduling when we run out of registers. If I do it another way I will have to always attempt rescheduling. Currently the impact on the compile time should be really minimal.


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

https://reviews.llvm.org/D73386