[PATCH] D50486: MachineScheduler: Refactor setPolicy() to limit computing remaining latency

[Andrew Trick via Phabricator via llvm-commits]

atrick added a comment.

Looks fine to me.



================
Comment at: lib/CodeGen/MachineScheduler.cpp:2429
+                                               unsigned &RemLatency) const {
+  if (CurrZone.getCurrCycle() > Rem.CriticalPath)
+    return true;
----------------
I suggest adding a brief comment here. I think this is just a quick check to bypass computeRemLatency, and ultimately we're trying to determine whether the current cycle plus remaining latency is greater than the critical path in the scheduling region.


Repository:
  rL LLVM

https://reviews.llvm.org/D50486