[PATCH] D29452: AMDGPU/SI: DAGMutation for removing deps between CSEs

[Tom Stellard via Phabricator via llvm-commits]

tstellar added a comment.

In https://reviews.llvm.org/D29452#673738, @arsenm wrote:

> Should this try to delete the redundant moves itself? MachineCSE runs earlier on SSA, so this isn't getting cleaned up. It helps with the clustering, but I get sequences of re-initing m0 to -1 like:


I tried to have the DAGMutation delete the redundant moves, but I started getting assertion failures all over (maybe you can get this to work, not sure).  Might have to just run MachineCSE after scheduling.


https://reviews.llvm.org/D29452