Skip to content

Commit 0b3b137

Browse files
kychendevigcbot
authored andcommitted
Revise register sharing heuristic.
Revise register sharing heuristic.
1 parent a6d35e7 commit 0b3b137

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

visa/LocalScheduler/G4_Sched.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ static const unsigned LARGE_BLOCK_SIZE = 20000;
2121
static const unsigned LARGE_BLOCK_SIZE_RPE = 32000;
2222
static const unsigned PRESSURE_REDUCTION_MIN_BENEFIT = 5;
2323
static const unsigned PRESSURE_REDUCTION_THRESHOLD = 110;
24-
static const unsigned PRESSURE_HIGH_THRESHOLD = 120;
24+
static const unsigned PRESSURE_HIGH_THRESHOLD = 128;
2525
static const unsigned PRESSURE_LOW_THRESHOLD = 60;
2626
static const unsigned PRESSURE_REDUCTION_THRESHOLD_SIMD32 = 120;
2727
static const unsigned LATENCY_PRESSURE_THRESHOLD = 100;
@@ -513,7 +513,7 @@ static unsigned getRPReductionThreshold(unsigned NumGrfs, unsigned simdSize)
513513
}
514514

515515
// Register pressure threshold to move to a larger GRF mode
516-
static unsigned getRPThresholdHigh(unsigned NumGrfs, unsigned simdSize)
516+
static unsigned getRPThresholdHigh(unsigned NumGrfs)
517517
{
518518
float Ratio = NumGrfs / 128.0f;
519519

@@ -728,7 +728,7 @@ bool preRA_RegSharing::run()
728728
}
729729

730730
if (!kernel.getOptions()->getuInt32Option(vISA_ForceHWThreadNumberPerEU) &&
731-
(maxPressure > getRPThresholdHigh(kernel.getNumRegTotal() - kernel.getOptions()->getuInt32Option(vISA_ReservedGRFNum), kernel.getSimdSize())))
731+
(maxPressure > getRPThresholdHigh(kernel.getNumRegTotal() - kernel.getOptions()->getuInt32Option(vISA_ReservedGRFNum))))
732732
{
733733
// Update number of threads, GRF, Acc and SWSB
734734
kernel.updateKernelByNumThreads(GrfMode.getMinNumThreads());

0 commit comments

Comments
 (0)