[PATCH] D38014: [AMDGPU] Prevent post-RA scheduler from breaking memory clauses

Valery Pykhtin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 19 13:33:36 PDT 2017


vpykhtin accepted this revision.
vpykhtin added a comment.
This revision is now accepted and ready to land.

LGTM.



================
Comment at: lib/Target/AMDGPU/AMDGPUSubtarget.cpp:537
+    SUnit *SUa = nullptr;
+    for (SUnit &SU : DAG->SUnits) {
+      MachineInstr &MI2 = *SU.getInstr();
----------------
please add a comment about expected SU order for DAG->SUnits


https://reviews.llvm.org/D38014





More information about the llvm-commits mailing list