[PATCH] D30442: [AMDGPU] Add second pass of the scheduler

Tom Stellard via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 28 08:26:52 PST 2017


tstellar added inline comments.


================
Comment at: lib/Target/AMDGPU/GCNSchedStrategy.cpp:48-50
+  if (MF != &DAG->MF)
+    TargetOccupancy = 0;
+  MF = &DAG->MF;
----------------
tstellar wrote:
> Can you store TargetOccupancy in the SIMachineFunctionInfo object?  I think that would be a little cleaner.
To clarify:

TargetOccupancy = MFI->getTargetOccupancy();


Repository:
  rL LLVM

https://reviews.llvm.org/D30442





More information about the llvm-commits mailing list