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

Valery Pykhtin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 16 08:53:47 PDT 2017


vpykhtin 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:";
----------------
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 :)


Repository:
  rL LLVM

https://reviews.llvm.org/D33117





More information about the llvm-commits mailing list