Skip to content

Commit 2b75023

Browse files
pthamminigcbot
authored andcommitted
Fix vISA_MinGRFNum and vISA_MaxGRFNum override by IGC
Fix vISA_MinGRFNum and vISA_MaxGRFNum override by IGC
1 parent 7929aac commit 2b75023

File tree

3 files changed

+6
-2
lines changed

3 files changed

+6
-2
lines changed

IGC/Compiler/CISACodeGen/CISABuilder.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5645,12 +5645,12 @@ namespace IGC
56455645
SaveOption(vISA_AutoGRFSelection, true);
56465646
}
56475647

5648-
if (lowerBoundGRF > 0)
5648+
if (lowerBoundGRF > 0 && vbuilder->GetuInt32Option(vISA_MinGRFNum) == 0)
56495649
{
56505650
SaveOption(vISA_MinGRFNum, lowerBoundGRF);
56515651
}
56525652

5653-
if (upperBoundGRF > 0)
5653+
if (upperBoundGRF > 0 && vbuilder->GetuInt32Option(vISA_MaxGRFNum) == 0)
56545654
{
56555655
SaveOption(vISA_MaxGRFNum, upperBoundGRF);
56565656
}

visa/BuildCISAIR.h

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -76,6 +76,9 @@ class CISA_IR_Builder : public VISABuilder {
7676
VISA_BUILDER_API int Compile(const char *isaasmFileName,
7777
bool emit_visa_only = false) override;
7878

79+
VISA_BUILDER_API int GetuInt32Option(vISAOptions option) override {
80+
return m_options.getuInt32Option(option);
81+
}
7982
VISA_BUILDER_API void SetOption(vISAOptions option, bool val) override {
8083
m_options.setOption(option, val);
8184
}

visa/include/VISABuilderAPIDefinition.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1321,6 +1321,7 @@ class VISABuilder {
13211321
VISA_BUILDER_API virtual int Compile(const char *isaasmFileName,
13221322
bool emit_visa_only = false) = 0;
13231323

1324+
VISA_BUILDER_API virtual int GetuInt32Option(vISAOptions option) = 0;
13241325
VISA_BUILDER_API virtual void SetOption(vISAOptions option, bool val) = 0;
13251326
VISA_BUILDER_API virtual void SetOption(vISAOptions option, uint32_t val) = 0;
13261327
VISA_BUILDER_API virtual void SetOption(vISAOptions option,

0 commit comments

Comments
 (0)