From 6c185e7355b3daac9fadcdcd5543c65bd90bc9ca Mon Sep 17 00:00:00 2001 From: Lorenc Bushi <113361374+lbushi25@users.noreply.github.com> Date: Thu, 16 May 2024 06:01:01 -0400 Subject: [PATCH] [SYCL] Fix unused variable errors on post-commit for forward_progress extension (#13793) The merge of https://github.com/intel/llvm/pull/13389 has caused post-commit failures due to three unused variable warnings. This PR fixes that. One of these warnings also revealed a bug in my original PR. --------- Co-authored-by: Steffen Larsen Co-authored-by: Alexey Sachkov --- sycl/source/detail/device_impl.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/sycl/source/detail/device_impl.cpp b/sycl/source/detail/device_impl.cpp index 303d86d2fa31..661a470ffe10 100644 --- a/sycl/source/detail/device_impl.cpp +++ b/sycl/source/detail/device_impl.cpp @@ -860,8 +860,8 @@ bool device_impl::extOneapiCanCompile( // coordinationScope sycl::ext::oneapi::experimental::forward_progress_guarantee device_impl::getHostProgressGuarantee( - ext::oneapi::experimental::execution_scope threadScope, - ext::oneapi::experimental::execution_scope coordinationScope) { + ext::oneapi::experimental::execution_scope, + ext::oneapi::experimental::execution_scope) { return sycl::ext::oneapi::experimental::forward_progress_guarantee:: weakly_parallel; } @@ -877,6 +877,7 @@ device_impl::getProgressGuarantee( ext::oneapi::experimental::forward_progress_guarantee; using execution_scope = ext::oneapi::experimental::execution_scope; const int executionScopeSize = 4; + (void)coordinationScope; int threadScopeNum = static_cast(threadScope); // we get the immediate progress guarantee that is provided by each scope // between root_group and threadScope and then return the weakest of these.