Skip to content

Commit c838a67

Browse files
committed
Fix program resources release
1 parent f1841ec commit c838a67

File tree

2 files changed

+11
-13
lines changed

2 files changed

+11
-13
lines changed

source/adapters/level_zero/program.cpp

Lines changed: 10 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1029,18 +1029,16 @@ void ur_program_handle_t_::ur_release_program_resources(bool deletion) {
10291029
ZE_CALL_NOCHECK(zeModuleBuildLogDestroy, (DeviceData.ZeBuildLog));
10301030
}
10311031

1032-
if (InteropZeModule && OwnZeModule) {
1033-
if (DeviceDataMap.empty()) {
1034-
// interop api
1035-
ZE_CALL_NOCHECK(zeModuleDestroy, (InteropZeModule));
1036-
} else {
1037-
for (auto &[ZeDevice, DeviceData] : this->DeviceDataMap) {
1038-
if (DeviceData.ZeModule)
1039-
ZE_CALL_NOCHECK(zeModuleDestroy, (DeviceData.ZeModule));
1040-
}
1041-
this->DeviceDataMap.clear();
1042-
}
1043-
}
1032+
// interop api
1033+
if (InteropZeModule && OwnZeModule)
1034+
ZE_CALL_NOCHECK(zeModuleDestroy, (InteropZeModule));
1035+
1036+
for (auto &[ZeDevice, DeviceData] : this->DeviceDataMap)
1037+
if (DeviceData.ZeModule)
1038+
ZE_CALL_NOCHECK(zeModuleDestroy, (DeviceData.ZeModule));
1039+
1040+
this->DeviceDataMap.clear();
1041+
10441042
resourcesReleased = true;
10451043
}
10461044
}

source/adapters/level_zero/program.hpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -291,5 +291,5 @@ struct ur_program_handle_t_ : _ur_object {
291291
// program. This handle is returned by UR API which allows to get the native
292292
// handle from the program.
293293
// TODO: Currently interoparability UR API does not support multiple devices.
294-
ze_module_handle_t InteropZeModule{};
294+
ze_module_handle_t InteropZeModule = nullptr;
295295
};

0 commit comments

Comments
 (0)