Skip to content

Commit c4fe7d5

Browse files
authored
[SYCL] Follow-up fix after #15179 (#15228)
Fix the issue caught by static analyzer after #15179
1 parent c6b611d commit c4fe7d5

File tree

2 files changed

+13
-0
lines changed

2 files changed

+13
-0
lines changed

sycl/source/detail/event_impl.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -490,6 +490,7 @@ ur_native_handle_t event_impl::getNative() {
490490
Plugin->call(urEventCreateWithNativeHandle, 0, TempContext,
491491
&NativeProperties, &UREvent);
492492
this->setHandle(UREvent);
493+
Handle = UREvent;
493494
}
494495
if (MContext->getBackend() == backend::opencl)
495496
Plugin->call(urEventRetain, Handle);

sycl/test-e2e/Plugin/interop-level-zero.cpp

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ int main() {
1818
#ifdef SYCL_EXT_ONEAPI_BACKEND_LEVEL_ZERO
1919
queue Queue{};
2020

21+
event DefaultEvent;
2122
auto Event = Queue.single_task([=]() {});
2223
auto Context = Queue.get_info<info::queue::context>();
2324
auto Device = Queue.get_info<info::queue::device>();
@@ -29,6 +30,8 @@ int main() {
2930
auto ZeContext = get_native<backend::ext_oneapi_level_zero>(Context);
3031
auto ZeQueue = get_native<backend::ext_oneapi_level_zero>(Queue);
3132
auto ZeEvent = get_native<backend::ext_oneapi_level_zero>(Event);
33+
auto DefaultZeEvent =
34+
get_native<backend::ext_oneapi_level_zero>(DefaultEvent);
3235

3336
// Create native Level-Zero context.
3437
// It then will be owned/destroyed by SYCL RT.
@@ -60,6 +63,13 @@ int main() {
6063
auto EventInterop = make_event<backend::ext_oneapi_level_zero>(
6164
EventInteropInput, ContextInterop);
6265

66+
backend_input_t<backend::ext_oneapi_level_zero, event>
67+
DefaultEventInteropInput = {DefaultZeEvent};
68+
DefaultEventInteropInput.Ownership =
69+
sycl::ext::oneapi::level_zero::ownership::keep;
70+
auto DefaultEventInterop = make_event<backend::ext_oneapi_level_zero>(
71+
DefaultEventInteropInput, ContextInterop);
72+
6373
// Check native handles
6474
assert(ZePlatform ==
6575
get_native<backend::ext_oneapi_level_zero>(PlatformInterop));
@@ -68,6 +78,8 @@ int main() {
6878
get_native<backend::ext_oneapi_level_zero>(ContextInterop));
6979
assert(ZeQueue == get_native<backend::ext_oneapi_level_zero>(QueueInterop));
7080
assert(ZeEvent == get_native<backend::ext_oneapi_level_zero>(EventInterop));
81+
assert(DefaultZeEvent ==
82+
get_native<backend::ext_oneapi_level_zero>(DefaultEventInterop));
7183

7284
// Verify re-created objects
7385
int Arr[] = {2};

0 commit comments

Comments
 (0)