Skip to content

[SYCL][NFC] Fix unused parameter warning in piQueueFlush #5139

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Dec 15, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion sycl/plugins/cuda/pi_cuda.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2257,7 +2257,10 @@ pi_result cuda_piQueueFinish(pi_queue command_queue) {
// There is no CUDA counterpart for queue flushing and we don't run into the
// same problem of having to flush cross-queue dependencies as some of the
// other plugins, so it can be left as no-op.
pi_result cuda_piQueueFlush(pi_queue command_queue) { return PI_SUCCESS; }
pi_result cuda_piQueueFlush(pi_queue command_queue) {
(void)command_queue;
return PI_SUCCESS;
}

/// Gets the native CUDA handle of a PI queue object
///
Expand Down
5 changes: 4 additions & 1 deletion sycl/plugins/hip/pi_hip.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2205,7 +2205,10 @@ pi_result hip_piQueueFinish(pi_queue command_queue) {
// There is no HIP counterpart for queue flushing and we don't run into the
// same problem of having to flush cross-queue dependencies as some of the
// other plugins, so it can be left as no-op.
pi_result hip_piQueueFlush(pi_queue command_queue) { return PI_SUCCESS; }
pi_result hip_piQueueFlush(pi_queue command_queue) {
(void)command_queue;
return PI_SUCCESS;
}

/// Gets the native HIP handle of a PI queue object
///
Expand Down
5 changes: 4 additions & 1 deletion sycl/plugins/level_zero/pi_level_zero.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3031,7 +3031,10 @@ pi_result piQueueFinish(pi_queue Queue) {

// Flushing cross-queue dependencies is covered by createAndRetainPiZeEventList,
// so this can be left as a no-op.
pi_result piQueueFlush(pi_queue Queue) { return PI_SUCCESS; }
pi_result piQueueFlush(pi_queue Queue) {
(void)Queue;
return PI_SUCCESS;
}

pi_result piextQueueGetNativeHandle(pi_queue Queue,
pi_native_handle *NativeHandle) {
Expand Down