Skip to content

Commit a30d372

Browse files
sys-igcigcbot
authored andcommitted
[Autobackout][FunctionalRegression]Revert of change: 1a88d6e: Adjust pre-RA scheduling heuristic
Adjust pre-RA scheduling heuristic
1 parent f82224a commit a30d372

File tree

1 file changed

+4
-11
lines changed

1 file changed

+4
-11
lines changed

visa/LocalScheduler/G4_Sched.cpp

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1672,8 +1672,8 @@ bool BB_Scheduler::scheduleBlockForLatency(unsigned &MaxPressure,
16721672
return false;
16731673

16741674
// UpperBoundGRF == 0 means we are scheduling for the fixed number of GRF
1675-
unsigned LatencyPressureThreshold = getLatencyHidingThreshold(kernel, nr);
1676-
if (UpperBoundGRF == 0 && MaxPressure >= LatencyPressureThreshold)
1675+
if (UpperBoundGRF == 0 &&
1676+
MaxPressure >= getLatencyHidingThreshold(kernel, nr))
16771677
return false;
16781678

16791679
// simple ROI check.
@@ -1685,15 +1685,8 @@ bool BB_Scheduler::scheduleBlockForLatency(unsigned &MaxPressure,
16851685
NumOfHighLatencyInsts++;
16861686
}
16871687
}
1688-
if (kernel.fg.builder->hasFiveALUPipes()) {
1689-
// For latest platform, do scheduling for kernels with low register
1690-
// pressure to improve ILP. Currently, half pressure threshold is used.
1691-
// Maybe tuned in future.
1692-
return (NumOfHighLatencyInsts >= 2) ||
1693-
(MaxPressure < (LatencyPressureThreshold / 2));
1694-
} else {
1695-
return (NumOfHighLatencyInsts >= 2);
1696-
}
1688+
1689+
return NumOfHighLatencyInsts >= 2;
16971690
};
16981691

16991692
unsigned NumGrfs =

0 commit comments

Comments
 (0)