refactor: remove always true variable - supportsFloatAtomics

Signed-off-by: Mateusz Jablonski <mateusz.jablonski@intel.com>
This commit is contained in:
Mateusz Jablonski
2025-10-01 12:35:08 +00:00
committed by Compute-Runtime-Automation
parent 2192e0b90e
commit 0caa8d5297
22 changed files with 81 additions and 124 deletions

View File

@@ -61,31 +61,29 @@ void getOpenclCFeaturesList(const HardwareInfo &hwInfo, OpenClCFeaturesContainer
strcpy_s(openClCFeature.name, CL_NAME_VERSION_MAX_NAME_SIZE, "__opencl_c_subgroups");
openclCFeatures.push_back(openClCFeature);
if (hwInfo.capabilityTable.supportsFloatAtomics) {
strcpy_s(openClCFeature.name, CL_NAME_VERSION_MAX_NAME_SIZE, "__opencl_c_ext_fp32_global_atomic_add");
openclCFeatures.push_back(openClCFeature);
strcpy_s(openClCFeature.name, CL_NAME_VERSION_MAX_NAME_SIZE, "__opencl_c_ext_fp32_global_atomic_add");
openclCFeatures.push_back(openClCFeature);
strcpy_s(openClCFeature.name, CL_NAME_VERSION_MAX_NAME_SIZE, "__opencl_c_ext_fp32_local_atomic_add");
openclCFeatures.push_back(openClCFeature);
strcpy_s(openClCFeature.name, CL_NAME_VERSION_MAX_NAME_SIZE, "__opencl_c_ext_fp32_local_atomic_add");
openclCFeatures.push_back(openClCFeature);
strcpy_s(openClCFeature.name, CL_NAME_VERSION_MAX_NAME_SIZE, "__opencl_c_ext_fp32_global_atomic_min_max");
openclCFeatures.push_back(openClCFeature);
strcpy_s(openClCFeature.name, CL_NAME_VERSION_MAX_NAME_SIZE, "__opencl_c_ext_fp32_global_atomic_min_max");
openclCFeatures.push_back(openClCFeature);
strcpy_s(openClCFeature.name, CL_NAME_VERSION_MAX_NAME_SIZE, "__opencl_c_ext_fp32_local_atomic_min_max");
openclCFeatures.push_back(openClCFeature);
strcpy_s(openClCFeature.name, CL_NAME_VERSION_MAX_NAME_SIZE, "__opencl_c_ext_fp32_local_atomic_min_max");
openclCFeatures.push_back(openClCFeature);
strcpy_s(openClCFeature.name, CL_NAME_VERSION_MAX_NAME_SIZE, "__opencl_c_ext_fp16_global_atomic_load_store");
openclCFeatures.push_back(openClCFeature);
strcpy_s(openClCFeature.name, CL_NAME_VERSION_MAX_NAME_SIZE, "__opencl_c_ext_fp16_global_atomic_load_store");
openclCFeatures.push_back(openClCFeature);
strcpy_s(openClCFeature.name, CL_NAME_VERSION_MAX_NAME_SIZE, "__opencl_c_ext_fp16_local_atomic_load_store");
openclCFeatures.push_back(openClCFeature);
strcpy_s(openClCFeature.name, CL_NAME_VERSION_MAX_NAME_SIZE, "__opencl_c_ext_fp16_local_atomic_load_store");
openclCFeatures.push_back(openClCFeature);
strcpy_s(openClCFeature.name, CL_NAME_VERSION_MAX_NAME_SIZE, "__opencl_c_ext_fp16_global_atomic_min_max");
openclCFeatures.push_back(openClCFeature);
strcpy_s(openClCFeature.name, CL_NAME_VERSION_MAX_NAME_SIZE, "__opencl_c_ext_fp16_global_atomic_min_max");
openclCFeatures.push_back(openClCFeature);
strcpy_s(openClCFeature.name, CL_NAME_VERSION_MAX_NAME_SIZE, "__opencl_c_ext_fp16_local_atomic_min_max");
openclCFeatures.push_back(openClCFeature);
}
strcpy_s(openClCFeature.name, CL_NAME_VERSION_MAX_NAME_SIZE, "__opencl_c_ext_fp16_local_atomic_min_max");
openclCFeatures.push_back(openClCFeature);
}
auto forceFp64Support = debugManager.flags.OverrideDefaultFP64Settings.get();
@@ -94,7 +92,7 @@ void getOpenclCFeaturesList(const HardwareInfo &hwInfo, OpenClCFeaturesContainer
strcpy_s(openClCFeature.name, CL_NAME_VERSION_MAX_NAME_SIZE, "__opencl_c_fp64");
openclCFeatures.push_back(openClCFeature);
if (hwInfo.capabilityTable.supportsOcl21Features && hwInfo.capabilityTable.supportsFloatAtomics) {
if (hwInfo.capabilityTable.supportsOcl21Features) {
strcpy_s(openClCFeature.name, CL_NAME_VERSION_MAX_NAME_SIZE, "__opencl_c_ext_fp64_global_atomic_add");
openclCFeatures.push_back(openClCFeature);

View File

@@ -67,7 +67,6 @@ const RuntimeCapabilityTable ADLN::capabilityTable{
.supportsMediaBlock = true,
.fusedEuEnabled = true,
.l0DebuggerSupported = false,
.supportsFloatAtomics = true,
.cxlType = 0};
WorkaroundTable ADLN::workaroundTable = {};

View File

@@ -67,7 +67,6 @@ const RuntimeCapabilityTable ADLP::capabilityTable{
.supportsMediaBlock = true,
.fusedEuEnabled = true,
.l0DebuggerSupported = false,
.supportsFloatAtomics = true,
.cxlType = 0};
WorkaroundTable ADLP::workaroundTable = {};

View File

@@ -67,7 +67,6 @@ const RuntimeCapabilityTable ADLS::capabilityTable{
.supportsMediaBlock = true,
.fusedEuEnabled = true,
.l0DebuggerSupported = false,
.supportsFloatAtomics = true,
.cxlType = 0};
WorkaroundTable ADLS::workaroundTable = {};

View File

@@ -67,7 +67,6 @@ const RuntimeCapabilityTable DG1::capabilityTable{
.supportsMediaBlock = true,
.fusedEuEnabled = true,
.l0DebuggerSupported = true,
.supportsFloatAtomics = true,
.cxlType = 0};
WorkaroundTable DG1::workaroundTable = {};

View File

@@ -67,7 +67,6 @@ const RuntimeCapabilityTable RKL::capabilityTable{
.supportsMediaBlock = true,
.fusedEuEnabled = true,
.l0DebuggerSupported = false,
.supportsFloatAtomics = true,
.cxlType = 0};
WorkaroundTable RKL::workaroundTable = {};

View File

@@ -67,7 +67,6 @@ const RuntimeCapabilityTable TGLLP::capabilityTable{
.supportsMediaBlock = true,
.fusedEuEnabled = true,
.l0DebuggerSupported = false,
.supportsFloatAtomics = true,
.cxlType = 0};
WorkaroundTable TGLLP::workaroundTable = {};

View File

@@ -54,7 +54,6 @@ struct RuntimeCapabilityTable {
bool supportsMediaBlock;
bool fusedEuEnabled;
bool l0DebuggerSupported;
bool supportsFloatAtomics;
uint32_t cxlType;
bool operator==(const RuntimeCapabilityTable &) const = default;

View File

@@ -70,7 +70,6 @@ const RuntimeCapabilityTable BMG::capabilityTable{
.supportsMediaBlock = false,
.fusedEuEnabled = false,
.l0DebuggerSupported = true,
.supportsFloatAtomics = true,
.cxlType = 0};
void BMG::setupFeatureAndWorkaroundTable(HardwareInfo *hwInfo, const ReleaseHelper &releaseHelper) {

View File

@@ -68,7 +68,6 @@ const RuntimeCapabilityTable LNL::capabilityTable{
.supportsMediaBlock = false,
.fusedEuEnabled = false,
.l0DebuggerSupported = true,
.supportsFloatAtomics = true,
.cxlType = 0};
void LNL::setupFeatureAndWorkaroundTable(HardwareInfo *hwInfo, const ReleaseHelper &releaseHelper) {

View File

@@ -69,7 +69,6 @@ const RuntimeCapabilityTable PTL::capabilityTable{
.supportsMediaBlock = false,
.fusedEuEnabled = false,
.l0DebuggerSupported = true,
.supportsFloatAtomics = true,
.cxlType = 0};
void PTL::setupFeatureAndWorkaroundTable(HardwareInfo *hwInfo, const ReleaseHelper &releaseHelper) {

View File

@@ -80,7 +80,6 @@ const RuntimeCapabilityTable PVC::capabilityTable{
.supportsMediaBlock = false,
.fusedEuEnabled = false,
.l0DebuggerSupported = true,
.supportsFloatAtomics = true,
.cxlType = 0};
void PVC::setupFeatureAndWorkaroundTable(HardwareInfo *hwInfo, const ReleaseHelper &releaseHelper) {

View File

@@ -65,7 +65,6 @@ const RuntimeCapabilityTable ARL::capabilityTable{
.supportsMediaBlock = true,
.fusedEuEnabled = true,
.l0DebuggerSupported = true,
.supportsFloatAtomics = true,
.cxlType = 0};
WorkaroundTable ARL::workaroundTable = {};

View File

@@ -71,7 +71,6 @@ const RuntimeCapabilityTable DG2::capabilityTable{
.supportsMediaBlock = true,
.fusedEuEnabled = true,
.l0DebuggerSupported = true,
.supportsFloatAtomics = true,
.cxlType = 0};
WorkaroundTable DG2::workaroundTable = {};

View File

@@ -67,7 +67,6 @@ const RuntimeCapabilityTable MTL::capabilityTable{
.supportsMediaBlock = true,
.fusedEuEnabled = true,
.l0DebuggerSupported = true,
.supportsFloatAtomics = true,
.cxlType = 0};
WorkaroundTable MTL::workaroundTable = {};