Skip to content

Commit 83a1555

Browse files
authored
Roll back to original usage of sycl::get_kernel_bundle (#1935)
To solve #1121. The original usage of `sycl::get_kernel_bundle` should not be device-specific. Since the fixing has landed on oneAPI 2025.2, kernel bundle is now created using only the kernel ID (`kid`) instead of both device and kernel ID, removing the workaround for device-specific kernel builds and associated comments.
1 parent 77cc792 commit 83a1555

File tree

1 file changed

+2
-7
lines changed

1 file changed

+2
-7
lines changed

src/comm/DeviceProperties.h

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -16,13 +16,8 @@ static int64_t syclMaxWorkGroupSize(
1616
auto dev = q.get_device();
1717

1818
auto kid = ::sycl::get_kernel_id<KernelClass>();
19-
// The kernel won't be built for devices except for the first device.
20-
// Launching kernel on devices except for the first device will raise
21-
// runtime error. Here is an alternative as a temporary solution to
22-
// provide an extra hint to SYCL runtime.
23-
// https://github.com/intel/llvm/issues/15127
24-
auto kbundle = ::sycl::get_kernel_bundle<::sycl::bundle_state::executable>(
25-
ctx, {dev}, {kid});
19+
auto kbundle =
20+
::sycl::get_kernel_bundle<::sycl::bundle_state::executable>(ctx, {kid});
2621

2722
::sycl::kernel k = kbundle.get_kernel(kid);
2823
return k.get_info<::sycl::info::kernel_device_specific::work_group_size>(dev);

0 commit comments

Comments
 (0)