Skip to content

Commit 2c2a649

Browse files
jfuentesigcbot
authored andcommitted
Make sure vISA honors forcing 128GRF number set by IGC.
Currently vISA honors forcing any GRF number except from 128, which is the dedault on all platforms. This PR updates this behavior to honor 128 as well.
1 parent c8e1ef7 commit 2c2a649

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

visa/G4_Kernel.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -991,7 +991,7 @@ void G4_Kernel::setKernelParameters(unsigned newGRF) {
991991
grfMode.setModeByNumThreads(overrideNumThreads);
992992
overrideGRFNum = 0;
993993
autoGRFSelection = false;
994-
} else if (overrideGRFNum != grfMode.getDefaultGRF()) {
994+
} else if (overrideGRFNum > 0) {
995995
// Forcing a specific number of GRFs
996996
grfMode.setModeByNumGRFs(overrideGRFNum);
997997
autoGRFSelection = false;

visa/include/VISAOptionsDefs.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -293,7 +293,7 @@ DEF_VISA_OPTION(vISA_SpillMemOffset, ET_INT32, "-spilloffset",
293293
DEF_VISA_OPTION(vISA_ReservedGRFNum, ET_INT32, "-reservedGRFNum",
294294
"USAGE: -reservedGRFNum <regNum>\n", 0)
295295
DEF_VISA_OPTION(vISA_TotalGRFNum, ET_INT32, "-TotalGRFNum",
296-
"USAGE: -TotalGRFNum <regNum>\n", 128)
296+
"USAGE: -TotalGRFNum <regNum>\n", 0)
297297
DEF_VISA_OPTION(vISA_RATrace, ET_BOOL, "-ratrace", UNUSED, false)
298298
DEF_VISA_OPTION(vISA_FastSpill, ET_BOOL, "-fasterRA", UNUSED, false)
299299
DEF_VISA_OPTION(vISA_AbortOnSpillThreshold, ET_INT32, "-abortOnSpill", UNUSED,

0 commit comments

Comments
 (0)