Skip to content

Commit 914b54d

Browse files
committed
Address review comment
1 parent c53232c commit 914b54d

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

sycl/source/detail/kernel_impl.cpp

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,9 @@ kernel_impl::kernel_impl(Managed<ur_kernel_handle_t> &&Kernel,
2727
MKernelBundleImpl(KernelBundleImpl ? KernelBundleImpl->shared_from_this()
2828
: nullptr),
2929
MIsInterop(true), MKernelArgMaskPtr{ArgMask},
30-
MInteropDeviceKernelInfoHolder(
31-
CompileTimeKernelInfoTy{MIsInterop ? getName() : std::string_view{}}),
30+
MInteropDeviceKernelInfoHolder(MIsInterop
31+
? CompileTimeKernelInfoTy{getName()}
32+
: CompileTimeKernelInfoTy{}),
3233
MDeviceKernelInfo(
3334
MIsInterop
3435
? MInteropDeviceKernelInfoHolder
@@ -60,8 +61,9 @@ kernel_impl::kernel_impl(Managed<ur_kernel_handle_t> &&Kernel,
6061
MKernelBundleImpl(KernelBundleImpl.shared_from_this()),
6162
MIsInterop(MDeviceImageImpl->getOriginMask() & ImageOriginInterop),
6263
MKernelArgMaskPtr{ArgMask}, MCacheMutex{CacheMutex},
63-
MInteropDeviceKernelInfoHolder(
64-
CompileTimeKernelInfoTy{MIsInterop ? getName() : std::string_view{}}),
64+
MInteropDeviceKernelInfoHolder(MIsInterop
65+
? CompileTimeKernelInfoTy{getName()}
66+
: CompileTimeKernelInfoTy{}),
6567
MDeviceKernelInfo(
6668
MIsInterop
6769
? MInteropDeviceKernelInfoHolder

0 commit comments

Comments
 (0)