Skip to content

Commit b660e06

Browse files
jfuentesigcbot
authored andcommitted
Re-schedule only when option ForceGRFModeUp is set.
1 parent e651db8 commit b660e06

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

visa/G4_Kernel.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2178,8 +2178,7 @@ GRFMode::GRFMode(const TARGET_PLATFORM platform, Options *op) : options(op) {
21782178
unsigned GRFMode::setModeByRegPressure(unsigned maxRP, unsigned largestInputReg,
21792179
bool forceGRFModeUp) {
21802180
unsigned size = configs.size(), i = 0;
2181-
bool spillAllowed = 0;
2182-
spillAllowed = options->getuInt32Option(vISA_SpillAllowed) > 256;
2181+
bool spillAllowed = options->getuInt32Option(vISA_SpillAllowed) > 256;
21832182
// find appropiate GRF based on reg pressure
21842183
for (; i < size; i++) {
21852184
if (configs[i].VRTEnable && configs[i].numGRF >= lowerBoundGRF &&

visa/Optimizer.h

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -159,8 +159,11 @@ class Optimizer {
159159
if (kernel.useAutoGRFSelection()) {
160160
unsigned InitialGRFNumber = kernel.getNumRegTotal();
161161
Sched.runWithGRFSelection(KernelPressure);
162-
if (InitialGRFNumber != kernel.getNumRegTotal())
162+
if (kernel.getuInt32Option(vISA_ForceGRFModeUp) > 0 &&
163+
InitialGRFNumber != kernel.getNumRegTotal()) {
164+
// Only re-schedule when ForceGRFModeUP is set
163165
Sched.run(KernelPressure);
166+
}
164167
} else {
165168
Sched.run(KernelPressure);
166169
}

0 commit comments

Comments
 (0)