[llvm] [MISched] Use right boundary when trying latency heuristics (PR #116592)

Min-Yih Hsu via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 19 14:18:28 PST 2024


================
@@ -3969,9 +3969,14 @@ bool PostGenericScheduler::tryCandidate(SchedCandidate &Cand,
                  TryCand, Cand, ResourceDemand))
     return TryCand.Reason != NoCand;
 
-  // Avoid serializing long latency dependence chains.
-  if (Cand.Policy.ReduceLatency && tryLatency(TryCand, Cand, Top)) {
-    return TryCand.Reason != NoCand;
+  // We only compare a subset of features when comparing nodes between
+  // Top and Bottom boundary.
+  if (Cand.AtTop == TryCand.AtTop) {
----------------
mshockwave wrote:

just want to make sure I understand correctly: this fix is to prevent us from comparing candidates from different boundaries when bidirectional is enabled, right?

https://github.com/llvm/llvm-project/pull/116592


More information about the llvm-commits mailing list