Skip to content

Commit 5998de1

Browse files
committed
Fix program resources release
1 parent b44cba2 commit 5998de1

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
@@ -1017,18 +1017,16 @@ void ur_program_handle_t_::ur_release_program_resources(bool deletion) {
10171017
ZE_CALL_NOCHECK(zeModuleBuildLogDestroy, (DeviceData.ZeBuildLog));
10181018
}
10191019

1020-
if (InteropZeModule && OwnZeModule) {
1021-
if (DeviceDataMap.empty()) {
1022-
// interop api
1023-
ZE_CALL_NOCHECK(zeModuleDestroy, (InteropZeModule));
1024-
} else {
1025-
for (auto &[ZeDevice, DeviceData] : this->DeviceDataMap) {
1026-
if (DeviceData.ZeModule)
1027-
ZE_CALL_NOCHECK(zeModuleDestroy, (DeviceData.ZeModule));
1028-
}
1029-
this->DeviceDataMap.clear();
1030-
}
1031-
}
1020+
// interop api
1021+
if (InteropZeModule && OwnZeModule)
1022+
ZE_CALL_NOCHECK(zeModuleDestroy, (InteropZeModule));
1023+
1024+
for (auto &[ZeDevice, DeviceData] : this->DeviceDataMap)
1025+
if (DeviceData.ZeModule)
1026+
ZE_CALL_NOCHECK(zeModuleDestroy, (DeviceData.ZeModule));
1027+
1028+
this->DeviceDataMap.clear();
1029+
10321030
resourcesReleased = true;
10331031
}
10341032
}

source/adapters/level_zero/program.hpp

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

0 commit comments

Comments
 (0)