Revert "SWDEV-359287 - uniform_work_group_size is interger type is 1 for true and 0 for false. Lighting kernel's init sets uniform work group flag and hence not need to set it from options."

This reverts commit ea25e65834.

Reason for revert: Waiting for staging results before finally merging it.

Change-Id: Iaabb510325f50147f368108e98531291217627c0
This commit is contained in:
Jaydeepkumar Patel
2024-01-17 06:16:54 -05:00
کامیت شده توسط Maneesh Gupta
والد f40c380cdb
کامیت 77be355fd9
3فایلهای تغییر یافته به همراه3 افزوده شده و 7 حذف شده
+1 -1
مشاهده پرونده
@@ -575,7 +575,7 @@ static amd_comgr_status_t populateKernelMetaV3(const amd_comgr_metadata_node_t k
kernel->SetWGPMode(buf.compare("true") == 0);
break;
case KernelField::UniformWrokGroupSize:
kernel->setUniformWorkGroupSize(buf.compare("1") == 0);
kernel->setUniformWorkGroupSize(buf.compare("true") == 0);
break;
default:
return AMD_COMGR_STATUS_ERROR;
@@ -762,9 +762,7 @@ bool LightningProgram::createKernels(void* binary, size_t binSize, bool useUnifo
}
kernels()[kernelName] = kernel;
if (codeObjectVer() < 5) {
kernel->setUniformWorkGroupSize(useUniformWorkGroupSize);
}
kernel->setUniformWorkGroupSize(useUniformWorkGroupSize);
}
}
executable_ = loader_->CreateExecutable(HSA_PROFILE_FULL, nullptr);
@@ -274,9 +274,7 @@ bool LightningProgram::createKernels(void* binary, size_t binSize, bool useUnifo
if (!aKernel->init()) {
return false;
}
if (codeObjectVer() < 5) {
aKernel->setUniformWorkGroupSize(useUniformWorkGroupSize);
}
aKernel->setUniformWorkGroupSize(useUniformWorkGroupSize);
aKernel->setInternalKernelFlag(internalKernel);
kernels()[kernelName] = aKernel;
}