[SYCL][NFC] Fix Windows warnings, sync usage of class/struct with new SYCL SPEC

Signed-off-by: Klochkov <[email protected]>

Author: v-klochkov

sycl/include/CL/sycl/detail/array.hpp

@@ -14,7 +14,7 @@
 
 namespace cl {
 namespace sycl {
-template <int dimensions> struct id;
+template <int dimensions> class id;
 template <int dimensions> class range;
 namespace detail {
 

sycl/include/CL/sycl/detail/helpers.hpp

@@ -22,10 +22,10 @@ namespace cl {
 namespace sycl {
 class context;
 class event;
-template <int dimensions, bool with_offset> struct item;
+template <int dimensions, bool with_offset> class item;
 template <int dimensions> class group;
 template <int dimensions> class range;
-template <int dimensions> struct id;
+template <int dimensions> class id;
 template <int dimensions> class nd_item;
 enum class memory_order;
 template <int dimensions> class h_item;
@@ -40,7 +40,8 @@ getOrWaitEvents(std::vector<cl::sycl::event> DepEvents,
 
 void waitEvents(std::vector<cl::sycl::event> DepEvents);
 
-struct Builder {
+class Builder {
+public:
   Builder() = delete;
   template <int dimensions>
   static group<dimensions> createGroup(const cl::sycl::range<dimensions> &G,

sycl/include/CL/sycl/group.hpp

@@ -24,7 +24,7 @@
 namespace cl {
 namespace sycl {
 namespace detail {
-struct Builder;
+class Builder;
 
 // Implements a barrier accross work items within a work group.
 static inline void workGroupBarrier() {
@@ -290,7 +290,7 @@ template <int dimensions = 1> class group {
   }
 
 protected:
-  friend struct detail::Builder;
+  friend class detail::Builder;
   group(const range<dimensions> &G, const range<dimensions> &L,
         const id<dimensions> &I)
       : globalRange(G), localRange(L), index(I) {}

sycl/include/CL/sycl/id.hpp

@@ -16,7 +16,7 @@ namespace cl {
 namespace sycl {
 template <int dimensions> class range;
 template <int dimensions, bool with_offset> class item;
-template <int dimensions = 1> struct id : public detail::array<dimensions> {
+template <int dimensions = 1> class id : public detail::array<dimensions> {
 private:
   using base = detail::array<dimensions>;
   static_assert(dimensions >= 1 && dimensions <= 3,

sycl/include/CL/sycl/intel/sub_group.hpp

@@ -343,7 +343,7 @@ struct sub_group {
   }
 
 protected:
-  template <int dimensions> friend struct cl::sycl::nd_item;
+  template <int dimensions> friend class cl::sycl::nd_item;
   sub_group() = default;
 };
 } // namespace intel

sycl/include/CL/sycl/intel/sub_group_host.hpp

@@ -137,7 +137,7 @@ struct sub_group {
   }
 
 protected:
-  template <int dimensions> friend struct cl::sycl::nd_item;
+  template <int dimensions> friend class cl::sycl::nd_item;
   sub_group() {
     throw runtime_error("Subgroups are not supported on host device. ");
   }

sycl/include/CL/sycl/item.hpp

@@ -16,14 +16,14 @@
 namespace cl {
 namespace sycl {
 namespace detail {
-struct Builder;
+class Builder;
 }
-template <int dimensions> struct id;
+template <int dimensions> class id;
 template <int dimensions> class range;
 template <int dimensions> class h_item;
 
-template <int dimensions = 1, bool with_offset = true> struct item {
-
+template <int dimensions = 1, bool with_offset = true> class item {
+public:
   item() = delete;
 
   id<dimensions> get_id() const { return index; }
@@ -86,10 +86,10 @@ template <int dimensions = 1, bool with_offset = true> struct item {
 
 protected:
   // For call constructor inside conversion operator
-  friend struct item<dimensions, false>;
-  friend struct item<dimensions, true>;
-  friend struct h_item<dimensions>;
-  friend struct detail::Builder;
+  friend class item<dimensions, false>;
+  friend class item<dimensions, true>;
+  friend class h_item<dimensions>;
+  friend class detail::Builder;
 
   template <size_t W = with_offset>
   item(typename std::enable_if<(W == true), const range<dimensions>>::type &R,

sycl/include/CL/sycl/nd_item.hpp

@@ -24,10 +24,10 @@
 namespace cl {
 namespace sycl {
 namespace detail {
-struct Builder;
+class Builder;
 }
-template <int dimensions = 1> struct nd_item {
-
+template <int dimensions = 1> class nd_item {
+public:
   nd_item() = delete;
 
   id<dimensions> get_global_id() const { return globalItem.get_id(); }
@@ -154,7 +154,7 @@ template <int dimensions = 1> struct nd_item {
   }
 
 protected:
-  friend struct detail::Builder;
+  friend class detail::Builder;
   nd_item(const item<dimensions, true> &GL, const item<dimensions, false> &L,
           const group<dimensions> &GR)
       : globalItem(GL), localItem(L), Group(GR) {}

sycl/include/CL/sycl/range.hpp

@@ -13,7 +13,7 @@
 
 namespace cl {
 namespace sycl {
-template <int dimensions> struct id;
+template <int dimensions> class id;
 template <int dimensions = 1>
 class range : public detail::array<dimensions> {
   static_assert(dimensions >= 1 && dimensions <= 3,