minor refactor

Change-Id: I27a77b1d6e100c6e9dbb4461ed7ef6d547b00744

Author: Jay-Jiewu-Lu

IGC/Compiler/CISACodeGen/Platform.hpp

@@ -391,6 +391,11 @@ namespace IGC
         return m_platformInfo.eRenderCoreFamily >= IGFX_GEN11_CORE && IGC_IS_FLAG_ENABLED(EnableIntegerMad);
     }
 
+    unsigned int getMaxNumberHWThreadForEachWG() const
+    {
+        return getMaxNumberThreadPerSubslice();
+    }
+
     // ***** Below go accessor methods for testing WA data from WA_TABLE *****
 
     bool WaDoNotPushConstantsForAllPulledGSTopologies() const

IGC/Compiler/CISACodeGen/helper.h

@@ -370,11 +370,10 @@ namespace IGC
 
     inline unsigned GetHwThreadsPerWG(const IGC::CPlatform& platform)
     {
-        unsigned hwThreadPerWorkgroup = platform.getMaxNumberThreadPerSubslice();
-
+        unsigned hwThreadPerWorkgroup = platform.getMaxNumberHWThreadForEachWG();
         if (platform.supportPooledEU())
         {
-            hwThreadPerWorkgroup = platform.getMaxNumberThreadPerWorkgroupPooledMax();
+            hwThreadPerWorkgroup = std::min(platform.getMaxNumberThreadPerWorkgroupPooledMax(), (unsigned)64);
         }
         return hwThreadPerWorkgroup;
     }
@@ -385,8 +384,7 @@ namespace IGC
         {
             hwThreadPerWorkgroup = 42; //On GT1 HW, there are 7 threads/EU and 6 EU/subslice, 42 is the minimum threads/workgroup any HW can support
         }
-        if ((threadGroupSize <= hwThreadPerWorkgroup * 8) &&
-            threadGroupSize <= 512)
+        if (threadGroupSize <= hwThreadPerWorkgroup * 8)
         {
             return SIMDMode::SIMD8;
         }