[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