[SYCL] Fix memory leak in InterOp make_event

bso-intel · bso-intel · commit 0b8e5740965b · 2022-03-28T16:17:10.000-07:00
Retain should be called only for OpenCL backend.

Signed-off-by: Byoungro So &lt;byoungro.so@intel.com&gt;
diff --git a/sycl/source/detail/event_impl.cpp b/sycl/source/detail/event_impl.cpp
@@ -123,8 +123,9 @@ event_impl::event_impl(RT::PiEvent Event, const context &SyclContext)
         "clEvent.",
         PI_INVALID_CONTEXT);
   }
-
-  getPlugin().call<PiApiKind::piEventRetain>(MEvent);
+  // Ref count should be incremented for OpenCL backend only.
+  if (getPlugin().getBackend() == cl::sycl::backend::opencl)
+    getPlugin().call<PiApiKind::piEventRetain>(MEvent);
 }
 
 event_impl::event_impl(const QueueImplPtr &Queue)

Original file line number	Diff line number	Diff line change
`@@ -123,8 +123,9 @@ event_impl::event_impl(RT::PiEvent Event, const context &SyclContext)`
`123`	`123`	`"clEvent.",`
`124`	`124`	`PI_INVALID_CONTEXT);`
`125`	`125`	`}`
`126`		`-`
`127`		`- getPlugin().call<PiApiKind::piEventRetain>(MEvent);`
	`126`	`+ // Ref count should be incremented for OpenCL backend only.`
	`127`	`+ if (getPlugin().getBackend() == cl::sycl::backend::opencl)`
	`128`	`+ getPlugin().call<PiApiKind::piEventRetain>(MEvent);`
`128`	`129`	`}`
`129`	`130`
`130`	`131`	`event_impl::event_impl(const QueueImplPtr &Queue)`