Media/intel-graphics-compiler
1
0
Fork 0
mirror of https://github.com/intel/intel-graphics-compiler.git synced 2025-10-30 08:18:26 +08:00
Code Issues Packages Projects Releases Wiki Activity

Fix OCL atomic benchmark regression

Browse Source 
More investigation is needed. Currently, we don't apply atomic_iadd to
atomic_inc/dec optimization to OCL cases.
This commit is contained in:
Chen, Kai
2025-10-18 05:39:07 +00:00
committed by igcbot
parent f8934ec463
commit 94f2cb27d9
3 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
IGC/AdaptorOCL/DriverInfoOCL.hpp
View File

@ -68,6 +68,7 @@ public:
bool EnableShaderDebugHashCodeInKernel() const override { return true; }
bool supportsAtomicIaddToIncDec() const override { return false; }
};
// In case some cpas are specific to NEO

1
IGC/Compiler/CISACodeGen/DriverInfo.hpp
View File

@ -400,6 +400,7 @@ public:
return (IGC_GET_FLAG_VALUE(UseNewInlineRaytracing) &
static_cast<uint32_t>(NewInlineRaytracingMask::NonRTShaders)) != 0;
}
virtual bool supportsAtomicIaddToIncDec() const { return true; }
protected:
bool autoGRFSelection = false;

6
IGC/Compiler/CustomSafeOptPass.cpp
View File

@ -523,7 +523,7 @@ void CustomSafeOptPass::visitIntAtomicIAddToIncOrDec(CallInst *I) {
// or
// %13 = call i32 @llvm.genx.GenISA.intatomicrawA64.i32.p3i32.p3i32(i32 addrspace(3)* %12, i32 addrspace(3)* %12, i32 poison, i32 2)
// %14 = call i32 @llvm.genx.GenISA.intatomicrawA64.i32.p3i32.p3i32(i32 addrspace(3)* %12, i32 addrspace(3)* %12, i32 poison, i32 3)
// clang-format on
// clang-format on
if (id == GenISAIntrinsic::GenISA_intatomicraw || id == GenISAIntrinsic::GenISA_intatomicrawA64) {
if (instr->getOperand(0)->getType()->getPointerAddressSpace() == ADDRESS_SPACE_LOCAL)
return;
@ -916,7 +916,9 @@ void CustomSafeOptPass::visitCallInst(CallInst &C) {
case GenISAIntrinsic::GenISA_intatomictyped:
case GenISAIntrinsic::GenISA_intatomicraw:
case GenISAIntrinsic::GenISA_intatomicrawA64: {
visitIntAtomicIAddToIncOrDec(inst);
if (pContext->m_DriverInfo.supportsAtomicIaddToIncDec()) {
visitIntAtomicIAddToIncOrDec(inst);
}
break;
}