[PATCH] D97342: [AMDGPU] Skip unclusterd rescheduling w/o ld/st

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added a comment.

In D97342#2590002 <https://reviews.llvm.org/D97342#2590002>, @vpykhtin wrote:

> Looks good, but should we use just a single dedicated pass over SUs to check if there're clustered ops after first scheduling to make the logic slightly easier?

My problem with that it has to be done in the schedule() method or somewhere else within GCNScheduleDAGMILive. The only way to get an SUnit there is to call getSUnit() passing a MachineInstr and that is a map lookup. I.e. it is simply slower and I am trying to squeeze as much speed as I could.


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

https://reviews.llvm.org/D97342