[SYCL] Remove sycl::byte alias (#4424)

sycl::byte is deprecated in SYCL 2020

Author: maximdimakov

sycl/include/CL/sycl/aliases.hpp

@@ -68,7 +68,6 @@ using half __SYCL2020_DEPRECATED("use 'sycl::half' instead") =
 
 __SYCL_INLINE_NAMESPACE(cl) {
 namespace sycl {
-using byte __SYCL2020_DEPRECATED("use std::byte instead") = std::uint8_t;
 using schar = signed char;
 using uchar = unsigned char;
 using ushort = unsigned short;

sycl/include/CL/sycl/detail/image_impl.hpp

@@ -21,7 +21,6 @@
 
 __SYCL_INLINE_NAMESPACE(cl) {
 namespace sycl {
-
 // forward declarations
 enum class image_channel_order : unsigned int;
 enum class image_channel_type : unsigned int;
@@ -36,7 +35,7 @@ class handler;
 namespace detail {
 
 // utility functions and typedefs for image_impl
-using image_allocator = aligned_allocator<byte>;
+using image_allocator = aligned_allocator<unsigned char>;
 
 // utility function: Returns the Number of Channels for a given Order.
 __SYCL_EXPORT uint8_t getImageNumberChannels(image_channel_order Order);

sycl/include/CL/sycl/image.hpp

@@ -59,9 +59,7 @@ enum class image_channel_type : unsigned int {
   fp32 = 14
 };
 
-using byte = unsigned char;
-
-using image_allocator = detail::aligned_allocator<byte>;
+using image_allocator = detail::aligned_allocator<unsigned char>;
 
 /// Defines a shared image data.
 ///

sycl/include/CL/sycl/sycl_span.hpp

@@ -138,9 +138,6 @@ template<class Container>
 __SYCL_INLINE_NAMESPACE(cl) {
 namespace sycl {
 
-// byte is unsigned char at sycl/image.hpp:58
-using byte = unsigned char;
-
 // asserts suppressed for device compatibility.
 // TODO: enable
 #if defined(__SYCL_DEVICE_ONLY__)
@@ -384,16 +381,17 @@ template <typename _Tp, size_t _Extent> class _SYCL_SPAN_TEMPLATE_VIS span {
     return rev_iterator(begin());
   }
 
-  _SYCL_SPAN_INLINE_VISIBILITY span<const byte, _Extent * sizeof(element_type)>
+  _SYCL_SPAN_INLINE_VISIBILITY
+  span<const std::byte, _Extent * sizeof(element_type)>
   __as_bytes() const noexcept {
-    return span<const byte, _Extent * sizeof(element_type)>{
-        reinterpret_cast<const byte *>(data()), size_bytes()};
+    return span<const std::byte, _Extent * sizeof(element_type)>{
+        reinterpret_cast<const std::byte *>(data()), size_bytes()};
   }
 
-  _SYCL_SPAN_INLINE_VISIBILITY span<byte, _Extent * sizeof(element_type)>
+  _SYCL_SPAN_INLINE_VISIBILITY span<std::byte, _Extent * sizeof(element_type)>
   __as_writable_bytes() const noexcept {
-    return span<byte, _Extent * sizeof(element_type)>{
-        reinterpret_cast<byte *>(data()), size_bytes()};
+    return span<std::byte, _Extent * sizeof(element_type)>{
+        reinterpret_cast<std::byte *>(data()), size_bytes()};
   }
 
 private:
@@ -577,14 +575,14 @@ class _SYCL_SPAN_TEMPLATE_VIS span<_Tp, dynamic_extent> {
     return rev_iterator(begin());
   }
 
-  _SYCL_SPAN_INLINE_VISIBILITY span<const byte, dynamic_extent>
+  _SYCL_SPAN_INLINE_VISIBILITY span<const std::byte, dynamic_extent>
   __as_bytes() const noexcept {
-    return {reinterpret_cast<const byte *>(data()), size_bytes()};
+    return {reinterpret_cast<const std::byte *>(data()), size_bytes()};
   }
 
-  _SYCL_SPAN_INLINE_VISIBILITY span<byte, dynamic_extent>
+  _SYCL_SPAN_INLINE_VISIBILITY span<std::byte, dynamic_extent>
   __as_writable_bytes() const noexcept {
-    return {reinterpret_cast<byte *>(data()), size_bytes()};
+    return {reinterpret_cast<std::byte *>(data()), size_bytes()};
   }
 
 private:

sycl/test/basic_tests/types.cpp

@@ -93,9 +93,6 @@ int main() {
   // Check the size and alignment of the SYCL vectors.
   checkVectors();
 
-  // Table 4.93: Additional scalar data types supported by SYCL.
-  static_assert(sizeof(s::byte) == sizeof(int8_t), "");
-
   // Table 4.94: Scalar data type aliases supported by SYCL
   static_assert(is_same<s::cl_bool, decltype(0 != 1)>::value, "");
   checkSizeForSignedIntegral<s::cl_char, sizeof(int8_t)>();

sycl/test/warnings/sycl_2020_deprecations.cpp

@@ -130,10 +130,6 @@ int main() {
         });
   });
 
-  // expected-warning@+1{{'byte' is deprecated: use std::byte instead}}
-  sycl::byte B;
-  (void)B;
-
   // expected-warning@+1{{'max_constant_buffer_size' is deprecated: max_constant_buffer_size is deprecated}}
   auto MCBS = sycl::info::device::max_constant_buffer_size;
   (void)MCBS;