Skip to content

Commit 610badc

Browse files
stefan-iligcbot
authored andcommitted
Enable SIMD16 drop for more platforms
Enable abort on spills to SIMD16 for more platforms.
1 parent 937b607 commit 610badc

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

IGC/Compiler/CISACodeGen/OpenCLKernelCodeGen.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2457,9 +2457,9 @@ SIMDStatus COpenCLKernel::checkSIMDCompileCondsForMin16(SIMDMode simdMode, EmitP
24572457
uint32_t requiredSimdSize = getReqdSubGroupSize(F, pMdUtils);
24582458

24592459
// there is a requirement for specific compilation size, we can't abort on simd32
2460-
if (requiredSimdSize != 0)
2460+
if (requiredSimdSize != 0 && !(requiredSimdSize < 32 && SIMDMode::SIMD32 == simdMode)) {
24612461
EP.m_canAbortOnSpill = false;
2462-
2462+
}
24632463
bool hasSubGroupForce = hasSubGroupIntrinsicPVC(F);
24642464
uint32_t maxPressure = getMaxPressure(F, pMdUtils);
24652465

IGC/Compiler/CISACodeGen/Platform.hpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,9 @@ class CPlatform {
186186

187187
bool isCoreXE2() const { return (m_platformInfo.eRenderCoreFamily == IGFX_XE2_HPG_CORE); }
188188

189-
bool isCoreXE3() const { return (m_platformInfo.eRenderCoreFamily == IGFX_XE3_CORE); }
189+
bool isCoreXE3() const {
190+
return (m_platformInfo.eRenderCoreFamily == IGFX_XE3_CORE);
191+
}
190192

191193
// This function checks if core is child of another core
192194
bool isCoreChildOf(GFXCORE_FAMILY core) const { return m_platformInfo.eRenderCoreFamily >= core; }

0 commit comments

Comments
 (0)