[PATCH] D33117: [AMDGPU] Cache live-ins and register pressure in scheduler

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec marked 4 inline comments as done.
rampitec added inline comments.


================
Comment at: lib/Target/AMDGPU/GCNSchedStrategy.cpp:340
+
+    DEBUG(const SIRegisterInfo *SRI = static_cast<const SIRegisterInfo*>(TRI);
+          dbgs() << "Pressure before scheduling:\nRegion live-ins:";
----------------
vpykhtin wrote:
> rampitec wrote:
> > rampitec wrote:
> > > vpykhtin wrote:
> > > > Turn this into a helper debug func?
> > > I want to switch to your print() function from tracker in a short while. It is now this way to compare logs to old implementation.
> > Actually printLivesAt() is a heavy one, recomputing live set. How about factoring this into GCNRPTracker::printLiveRegs()?
> Later commit :)
D33244


Repository:
  rL LLVM

https://reviews.llvm.org/D33117