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

[Jay Foad via Phabricator via llvm-commits]

foad added a comment.

In D72737#1821937 <https://reviews.llvm.org/D72737#1821937>, @rampitec wrote:

> We have moved uses of loaded values further from their loads, which is good. As far as I understand these changes are inducted by the removal of artificial edges which were created by MemOpClusterMutation. These edges were linking successors of any load to all the nodes in a cluster and restricted the scheduling.
>  In sign_extend.ll that is because of the store clustering, we have moved v_ashrrev_i32_e32 producing v2 past v_ashrrev_i32_e32 producing v3 because store cluster uses them in this order. Before it was harder to do because of the artificial edges linking all predecessors to all stores.


OK, that sounds plausible, thanks!


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

https://reviews.llvm.org/D72737