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

Javed Absar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 18 20:58:29 PDT 2017


javed.absar added inline comments.


================
Comment at: lib/Target/AMDGPU/AMDGPUSubtarget.cpp:531
+
+  struct MemOpClusterMutation : ScheduleDAGMutation {
+    const SIInstrInfo *TII;
----------------
It might be neater and easier for future extension to take out definition of MemOpClusterMutation outside of getPostRAMutation.


https://reviews.llvm.org/D38014





More information about the llvm-commits mailing list