[SYCL] Fix one more warning coming from SYCL headers (#1758)

Signed-off-by: Aleksander Fadeev <[email protected]>

Author: fadeeval

sycl/include/CL/sycl/types.hpp

@@ -330,8 +330,8 @@ convertImpl(T Value) {
   detail::enable_if_t<is_sint_to_sint<T, R>::value &&                          \
                           !std::is_same<OpenCLT, OpenCLR>::value &&            \
                           (std::is_same<OpenCLR, DestType>::value ||           \
-                           std::is_same<OpenCLR, signed char>::value &&        \
-                               std::is_same<DestType, char>::value),           \
+                           (std::is_same<OpenCLR, signed char>::value &&       \
+                            std::is_same<DestType, char>::value)),             \
                       R>                                                       \
   convertImpl(T Value) {                                                       \
     OpenCLT OpValue = cl::sycl::detail::convertDataToType<T, OpenCLT>(Value);  \
@@ -382,8 +382,8 @@ convertImpl(T Value) {
             typename OpenCLT, typename OpenCLR>                                \
   detail::enable_if_t<is_sint_to_float<T, R>::value &&                         \
                           (std::is_same<OpenCLR, DestType>::value ||           \
-                           std::is_same<OpenCLR, _Float16>::value &&           \
-                               std::is_same<DestType, half>::value),           \
+                           (std::is_same<OpenCLR, _Float16>::value &&          \
+                            std::is_same<DestType, half>::value)),             \
                       R>                                                       \
   convertImpl(T Value) {                                                       \
     OpenCLT OpValue = cl::sycl::detail::convertDataToType<T, OpenCLT>(Value);  \
@@ -402,8 +402,8 @@ __SYCL_GENERATE_CONVERT_IMPL(SToF, double)
             typename OpenCLT, typename OpenCLR>                                \
   detail::enable_if_t<is_uint_to_float<T, R>::value &&                         \
                           (std::is_same<OpenCLR, DestType>::value ||           \
-                           std::is_same<OpenCLR, _Float16>::value &&           \
-                               std::is_same<DestType, half>::value),           \
+                           (std::is_same<OpenCLR, _Float16>::value &&          \
+                            std::is_same<DestType, half>::value)),             \
                       R>                                                       \
   convertImpl(T Value) {                                                       \
     OpenCLT OpValue = cl::sycl::detail::convertDataToType<T, OpenCLT>(Value);  \
@@ -424,8 +424,8 @@ __SYCL_GENERATE_CONVERT_IMPL(UToF, double)
   detail::enable_if_t<is_float_to_float<T, R>::value &&                        \
                           !std::is_same<OpenCLT, OpenCLR>::value &&            \
                           (std::is_same<OpenCLR, DestType>::value ||           \
-                           std::is_same<OpenCLR, _Float16>::value &&           \
-                               std::is_same<DestType, half>::value) &&         \
+                           (std::is_same<OpenCLR, _Float16>::value &&          \
+                            std::is_same<DestType, half>::value)) &&           \
                           RoundingModeCondition<roundingMode>::value,          \
                       R>                                                       \
   convertImpl(T Value) {                                                       \

sycl/test/warnings/warnings.cpp

@@ -1,20 +1,17 @@
-// RUN: %clangxx -fsycl --no-system-header-prefix=CL/ -Wall -Wextra -Wno-ignored-attributes -Wno-deprecated-declarations -Wpessimizing-move -Wunused-variable -Wmismatched-tags -Wunneeded-internal-declaration -Werror -Wno-unknown-cuda-version %s -o %t.out
+// RUN: %clangxx -fsycl --no-system-header-prefix=CL/sycl -fsyntax-only -Wall -Wextra -Wno-ignored-attributes -Wno-deprecated-declarations -Wpessimizing-move -Wunused-variable -Wmismatched-tags -Wunneeded-internal-declaration -Werror -Wno-unknown-cuda-version %s -o %t.out
 
 #include <CL/sycl.hpp>
 
 using namespace cl::sycl;
-
-int main(void) {
-  // add a very simple kernel to see if compilation succeeds with -Werror
-  int data1[10] = {-1, -1, -1, -1, -1, -1, -1, -1, -1, -1};
-  buffer<int, 1> B(data1, range<1>(10), {property::buffer::use_host_ptr()});
-  queue Q;
-  Q.submit([&](handler &CGH) {
-    auto Accessor = B.get_access<access::mode::read_write>(CGH);
-    CGH.parallel_for<class TheSimpleKernel>(range<1>{10}, [=](id<1> Item) {
-      Accessor[Item] = 0;
+int main() {
+  vec<long, 4> newVec;
+  queue myQueue;
+  buffer<vec<long, 4>, 1> resultBuf{&newVec, range<1>{1}};
+  myQueue.submit([&](handler &cgh) {
+    auto writeResult = resultBuf.get_access<access::mode::write>(cgh);
+    cgh.single_task<class kernel_name>([=]() {
+      writeResult[0] = (vec<int, 4>{1, 2, 3, 4}).template convert<long>();
     });
   });
-
   return 0;
 }