Review comments

Author: slawekptak

sycl/include/sycl/queue.hpp

@@ -219,10 +219,10 @@ auto submit_kernel_direct_parallel_for(
 template <typename KernelName = detail::auto_name, bool EventNeeded = false,
           typename PropertiesT, typename KernelTypeUniversalRef>
 auto submit_kernel_direct_single_task(
-    const queue &Queue, PropertiesT Props, KernelTypeUniversalRef &&KernelFunc,
+    const queue &Queue, [[maybe_unused]] PropertiesT Props,
+    KernelTypeUniversalRef &&KernelFunc,
     const detail::code_location &CodeLoc = detail::code_location::current()) {
   // TODO Properties not supported yet
-  (void)Props;
   static_assert(
       std::is_same_v<PropertiesT,
                      ext::oneapi::experimental::empty_properties_t>,
@@ -244,7 +244,7 @@ auto submit_kernel_direct_single_task(
   // Instantiating the kernel on the host improves debugging.
   // Passing this pointer to another translation unit prevents optimization.
 #ifndef NDEBUG
-  // TODO: call library to prevent dropping call due to optimization
+  // TODO: call library to prevent dropping call due to optimization.
   (void)detail::GetInstantiateKernelOnHostPtr<KernelType, void, 1>();
 #endif
 

sycl/test-e2e/Basic/test_num_kernel_copies.cpp

@@ -36,7 +36,7 @@ int main(int argc, char **argv) {
 
   kernel<2> krn2;
   q.single_task(krn2);
-  // The kernel is copied on the scheduler-based path only
+  // The kernel is copied on the scheduler-based path only.
   assert(copy_count == 0);
   assert(move_count == 0);
   copy_count = 0;