From f3b9f48a49b8e67c0c8ad08649327a701ccac461 Mon Sep 17 00:00:00 2001 From: Sergey Kopienko Date: Mon, 13 Jan 2025 10:06:41 +0100 Subject: [PATCH] include/oneapi/dpl/pstl/hetero/dpcpp/parallel_backend_sycl.h - fix review comment: restore __device_backend_tag in __parallel_find_or_impl_one_wg::operator() and __parallel_find_or_impl_multiple_wgs::operator() --- .../pstl/hetero/dpcpp/parallel_backend_sycl.h | 20 +++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/include/oneapi/dpl/pstl/hetero/dpcpp/parallel_backend_sycl.h b/include/oneapi/dpl/pstl/hetero/dpcpp/parallel_backend_sycl.h index c427cd8f9d..3d17d66536 100644 --- a/include/oneapi/dpl/pstl/hetero/dpcpp/parallel_backend_sycl.h +++ b/include/oneapi/dpl/pstl/hetero/dpcpp/parallel_backend_sycl.h @@ -1786,9 +1786,9 @@ struct __parallel_find_or_impl_one_wg<__internal::__optional_kernel_name __FoundStateType - operator()(_ExecutionPolicy&& __exec, _BrickTag __brick_tag, const std::size_t __rng_n, - const std::size_t __wgroup_size, const __FoundStateType __init_value, _Predicate __pred, - _Ranges&&... __rngs) + operator()(oneapi::dpl::__internal::__device_backend_tag, _ExecutionPolicy&& __exec, _BrickTag __brick_tag, + const std::size_t __rng_n, const std::size_t __wgroup_size, const __FoundStateType __init_value, + _Predicate __pred, _Ranges&&... __rngs) { using __result_and_scratch_storage_t = __result_and_scratch_storage<_ExecutionPolicy, __FoundStateType>; __result_and_scratch_storage_t __result_storage{__exec, 1, 0}; @@ -1851,9 +1851,9 @@ struct __parallel_find_or_impl_multiple_wgs<__internal::__optional_kernel_name _AtomicType - operator()(_ExecutionPolicy&& __exec, _BrickTag __brick_tag, const std::size_t __rng_n, - const std::size_t __n_groups, const std::size_t __wgroup_size, const _AtomicType __init_value, - _Predicate __pred, _Ranges&&... __rngs) + operator()(oneapi::dpl::__internal::__device_backend_tag, _ExecutionPolicy&& __exec, _BrickTag __brick_tag, + const std::size_t __rng_n, const std::size_t __n_groups, const std::size_t __wgroup_size, + const _AtomicType __init_value, _Predicate __pred, _Ranges&&... __rngs) { auto __result = __init_value; @@ -1950,8 +1950,8 @@ __parallel_find_or(oneapi::dpl::__internal::__device_backend_tag, _ExecutionPoli // Single WG implementation __result = __parallel_find_or_impl_one_wg<_KernelName, __or_tag_check>()( - std::forward<_ExecutionPolicy>(__exec), __brick_tag, __rng_n, __wgroup_size, __init_value, __pred, - std::forward<_Ranges>(__rngs)...); + oneapi::dpl::__internal::__device_backend_tag{}, std::forward<_ExecutionPolicy>(__exec), __brick_tag, + __rng_n, __wgroup_size, __init_value, __pred, std::forward<_Ranges>(__rngs)...); } else { @@ -1963,8 +1963,8 @@ __parallel_find_or(oneapi::dpl::__internal::__device_backend_tag, _ExecutionPoli // Multiple WG implementation __result = __parallel_find_or_impl_multiple_wgs<_KernelName, __or_tag_check>()( - std::forward<_ExecutionPolicy>(__exec), __brick_tag, __rng_n, __n_groups, __wgroup_size, __init_value, - __pred, std::forward<_Ranges>(__rngs)...); + oneapi::dpl::__internal::__device_backend_tag{}, std::forward<_ExecutionPolicy>(__exec), __brick_tag, + __rng_n, __n_groups, __wgroup_size, __init_value, __pred, std::forward<_Ranges>(__rngs)...); } if constexpr (__or_tag_check)