From 7e63046ce9542d910b7eeb16fabc909182e1f81d Mon Sep 17 00:00:00 2001 From: Andrew Trick Date: Mon, 17 Jun 2013 21:45:07 +0000 Subject: [PATCH] MI-Sched: Remove another heuristic that is sensitive to queue order. llvm-svn: 184130 --- llvm/lib/CodeGen/MachineScheduler.cpp | 6 ------ 1 file changed, 6 deletions(-) diff --git a/llvm/lib/CodeGen/MachineScheduler.cpp b/llvm/lib/CodeGen/MachineScheduler.cpp index 5a9c4e45132c..55e58969a041 100644 --- a/llvm/lib/CodeGen/MachineScheduler.cpp +++ b/llvm/lib/CodeGen/MachineScheduler.cpp @@ -2113,15 +2113,9 @@ void ConvergingScheduler::tryCandidate(SchedCandidate &Cand, return; // Weak edges are for clustering and other constraints. - // - // Deferring TryCand here does not change Cand's reason. This is good in the - // sense that a bad candidate shouldn't affect a previous candidate's - // goodness, but bad in that it is assymetric and depends on queue order. - CandReason OrigReason = Cand.Reason; if (tryLess(getWeakLeft(TryCand.SU, Zone.isTop()), getWeakLeft(Cand.SU, Zone.isTop()), TryCand, Cand, Weak)) { - Cand.Reason = OrigReason; return; } // Avoid critical resource consumption and balance the schedule.