Skip to content

Commit 6c185e7

Browse files
lbushi25steffenlarsenAlexeySachkov
authored
[SYCL] Fix unused variable errors on post-commit for forward_progress extension (#13793)
The merge of #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 <[email protected]> Co-authored-by: Alexey Sachkov <[email protected]>
1 parent 247e5e0 commit 6c185e7

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

sycl/source/detail/device_impl.cpp

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -860,8 +860,8 @@ bool device_impl::extOneapiCanCompile(
860860
// coordinationScope
861861
sycl::ext::oneapi::experimental::forward_progress_guarantee
862862
device_impl::getHostProgressGuarantee(
863-
ext::oneapi::experimental::execution_scope threadScope,
864-
ext::oneapi::experimental::execution_scope coordinationScope) {
863+
ext::oneapi::experimental::execution_scope,
864+
ext::oneapi::experimental::execution_scope) {
865865
return sycl::ext::oneapi::experimental::forward_progress_guarantee::
866866
weakly_parallel;
867867
}
@@ -877,6 +877,7 @@ device_impl::getProgressGuarantee(
877877
ext::oneapi::experimental::forward_progress_guarantee;
878878
using execution_scope = ext::oneapi::experimental::execution_scope;
879879
const int executionScopeSize = 4;
880+
(void)coordinationScope;
880881
int threadScopeNum = static_cast<int>(threadScope);
881882
// we get the immediate progress guarantee that is provided by each scope
882883
// between root_group and threadScope and then return the weakest of these.

0 commit comments

Comments
 (0)