Remove unneeded condition

Author: jfuentes

visa/G4_Kernel.cpp

@@ -2117,7 +2117,8 @@ unsigned G4_Kernel::getSRFInWords() {
 
 // GRF modes supported by HW
 // There must be at least one Config that is VRTEnable for each platform
-GRFMode::GRFMode(const TARGET_PLATFORM platform, Options *op) : options(op) {
+GRFMode::GRFMode(const TARGET_PLATFORM plat, Options *op)
+    : platform(plat), options(op) {
   switch (platform) {
   case Xe_XeHPSDV:
   case Xe_DG2:
@@ -2178,7 +2179,7 @@ GRFMode::GRFMode(const TARGET_PLATFORM platform, Options *op) : options(op) {
 unsigned GRFMode::setModeByRegPressure(unsigned maxRP, unsigned largestInputReg,
                                        bool forceGRFModeUp) {
   unsigned size = configs.size(), i = 0;
-  bool spillAllowed = options->getuInt32Option(vISA_SpillAllowed) > 256;
+  bool spillAllowed = options->getuInt32Option(vISA_SpillAllowed) > 0;
   // find appropiate GRF based on reg pressure
   for (; i < size; i++) {
     if (configs[i].VRTEnable && configs[i].numGRF >= lowerBoundGRF &&
@@ -2196,10 +2197,21 @@ unsigned GRFMode::setModeByRegPressure(unsigned maxRP, unsigned largestInputReg,
           currentMode = newGRFMode < maxGRFMode ? newGRFMode : maxGRFMode;
         }
 
-        if (spillAllowed && currentMode > 0)
-          return configs[--currentMode].numGRF;
-        else
-          return configs[currentMode].numGRF;
+        if (platform >= Xe3 && spillAllowed && currentMode > 0) {
+          unsigned lowerGRFNum = configs[currentMode - 1].numGRF;
+          // Select a lower GRF number in PreRA in case the register
+          // pressure computed is a bit higher (e.g. 10%) than the lower GRF
+          // config. If spills are detected, RA will still bump up the GRF
+          // number to avoid them.
+          // For example, if reg pressure is 167, we select 160GRF since
+          // we have spill threshold enabled and the diff between 167 and 160
+          // is less than 10%.
+          if (lowerGRFNum * 1.1 >= maxRP &&
+              lowerGRFNum >= (largestInputReg + 8) &&
+              lowerGRFNum >= lowerBoundGRF)
+            return configs[--currentMode].numGRF;
+        }
+        return configs[currentMode].numGRF;
       }
     }
   }

visa/G4_Kernel.hpp

@@ -266,6 +266,7 @@ class GRFMode {
   unsigned lowerBoundGRF;
   unsigned upperBoundGRF;
   unsigned GRFModeUpValue;
+  const TARGET_PLATFORM platform;
   Options *options;
 };