Apply comments

Author: sergey-semenov

sycl/include/CL/sycl/detail/pi.h

@@ -44,7 +44,7 @@
 //
 #include "CL/cl.h"
 #define _PI_H_VERSION_MAJOR 5
-#define _PI_H_VERSION_MINOR 7
+#define _PI_H_VERSION_MINOR 8
 
 #define _PI_STRING_HELPER(a) #a
 #define _PI_CONCAT(a, b) _PI_STRING_HELPER(a.b)

sycl/plugins/cuda/pi_cuda.cpp

@@ -2252,6 +2252,9 @@ pi_result cuda_piQueueFinish(pi_queue command_queue) {
   return result;
 }
 
+// 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; }
 
 /// Gets the native CUDA handle of a PI queue object

sycl/plugins/hip/pi_hip.cpp

@@ -2202,6 +2202,9 @@ pi_result hip_piQueueFinish(pi_queue command_queue) {
   return result;
 }
 
+// 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; }
 
 /// Gets the native HIP handle of a PI queue object

sycl/plugins/level_zero/pi_level_zero.cpp

@@ -2997,6 +2997,8 @@ pi_result piQueueFinish(pi_queue Queue) {
   return PI_SUCCESS;
 }
 
+// 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 piextQueueGetNativeHandle(pi_queue Queue,