Remove TODO comments that have now been tried.

JonPsson1 · JonPsson1 · commit 7fa69cd04a15 · 2025-09-22T11:32:39.000+02:00
Some experiments made with these, without any improvements seen.
diff --git a/llvm/lib/Target/SystemZ/SystemZMachineScheduler.cpp b/llvm/lib/Target/SystemZ/SystemZMachineScheduler.cpp
@@ -185,7 +185,6 @@ void SystemZPreRASchedStrategy::initializeStoresGroup() {
   }
 
   // Value of 8 handles a known regression (with group of 20).
-  // TODO: Would some other value be better?
   if (StoresGroup.size() < 8)
     StoresGroup.clear();
 }
@@ -334,7 +333,6 @@ bool SystemZPreRASchedStrategy::tryCandidate(SchedCandidate &Cand,
 
   if (TinyRegion) {
     // Prioritize instructions that read unbuffered resources by stall cycles.
-    // TODO: Try this in bigger regions as well.
     if (tryLess(Zone->getLatencyStallCycles(TryCand.SU),
                 Zone->getLatencyStallCycles(Cand.SU), TryCand, Cand, Stall))
       return TryCand.Reason != NoCand;

Original file line number	Diff line number	Diff line change
`@@ -185,7 +185,6 @@ void SystemZPreRASchedStrategy::initializeStoresGroup() {`
`185`	`185`	`}`
`186`	`186`
`187`	`187`	`// Value of 8 handles a known regression (with group of 20).`
`188`		`- // TODO: Would some other value be better?`
`189`	`188`	`if (StoresGroup.size() < 8)`
`190`	`189`	`StoresGroup.clear();`
`191`	`190`	`}`
`@@ -334,7 +333,6 @@ bool SystemZPreRASchedStrategy::tryCandidate(SchedCandidate &Cand,`
`334`	`333`
`335`	`334`	`if (TinyRegion) {`
`336`	`335`	`// Prioritize instructions that read unbuffered resources by stall cycles.`
`337`		`- // TODO: Try this in bigger regions as well.`
`338`	`336`	`if (tryLess(Zone->getLatencyStallCycles(TryCand.SU),`
`339`	`337`	`Zone->getLatencyStallCycles(Cand.SU), TryCand, Cand, Stall))`
`340`	`338`	`return TryCand.Reason != NoCand;`