[PATCH] D38279: [MachineScheduler] Enable latency heuristic based on scheduled lat.

Javed Absar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 28 02:23:01 PST 2017


javed.absar added inline comments.


================
Comment at: lib/CodeGen/MachineScheduler.cpp:2445
+    unsigned Issued = CurrZone.isTop() ? CurrZone.getCurrCycle() :
+                                         CurrZone.getScheduledLatency();
+    if (IsPostRA || (RemLatency + Issued > Rem.CriticalPath)) {
----------------
Bit strange. getScheduledLatency returns max of CurrCycle and ExpectedLatency which I dont quite find set anywhere (other than 0). 


https://reviews.llvm.org/D38279





More information about the llvm-commits mailing list