Apply CR comments

Author: dm-vodopyanov

sycl/include/CL/sycl/handler.hpp

@@ -498,7 +498,7 @@ class __SYCL_EXPORT handler {
                                                         LambdaArgType>() &&
         MIsHost) {
       throw cl::sycl::feature_not_supported(
-          "kernel_handler is not supported by host device.",
+          "kernel_handler is not yet supported by host device.",
           PI_INVALID_OPERATION);
     }
     MHostKernel.reset(

sycl/include/CL/sycl/kernel_handler.hpp

@@ -33,7 +33,7 @@ class kernel_handler {
     return getSpecializationConstantOnDevice<S>();
 #else
     throw cl::sycl::feature_not_supported(
-        "kernel_handler::get_specialization_constant() is not supported by "
+        "kernel_handler::get_specialization_constant() is not yet supported by "
         "host device.",
         PI_INVALID_OPERATION);
 #endif // __SYCL_DEVICE_ONLY__

sycl/include/CL/sycl/specialization_id.hpp

@@ -20,7 +20,7 @@ template <typename T> class specialization_id {
 
   template <class... Args>
   explicit constexpr specialization_id(Args &&... args)
-      : MSpecializationConstantValue(args...) {}
+      : MSpecializationConstantValue(std::forward<Args>(args)...) {}
 
   specialization_id(const specialization_id &rhs) = delete;
   specialization_id(specialization_id &&rhs) = delete;

sycl/test/on-device/basic_tests/specialization_constants/kernel_lambda_with_kernel_handler_arg.cpp

@@ -1,6 +1,10 @@
 // RUN: %clangxx -fsycl -fsycl-targets=%sycl_triple %s -o %t.out
 // RUN: %t.out
 
+// This test checks all possible scenarios of running single_task, parallel_for
+// and parallel_for_work_group to verify that this code compiles and runs
+// correctly with user's lambda with and without sycl::kernel_handler argument
+
 #include <CL/sycl.hpp>
 
 int main() {