[SYCL][Bindless][UR] moving srgb check to pupulate_ur_structs

Author: juanchuletas

sycl/include/sycl/ext/oneapi/bindless_images_descriptor.hpp

@@ -132,17 +132,6 @@ struct image_descriptor {
       throw sycl::exception(sycl::errc::invalid,
                             "Images must have 1, 2, 3, or 4 channels.");
     }
-    if (color_space == image_color_space::srgb) {
-      if (num_channels != 4) {
-        throw sycl::exception(sycl::errc::invalid,
-                              "sRGB color space requires num_channels == 4");
-      }
-      if (channel_type != image_channel_type::unorm_int8) {
-        throw sycl::exception(
-            sycl::errc::invalid,
-            "sRGB color space requires unorm_int8 channel type");
-      }
-    }
     switch (this->type) {
     case image_type::standard:
       if (this->array_size > 1) {

sycl/source/detail/bindless_images.cpp

@@ -23,6 +23,19 @@ namespace ext::oneapi::experimental {
 
 void populate_ur_structs(const image_descriptor &desc, ur_image_desc_t &urDesc,
                          ur_image_format_t &urFormat, size_t pitch = 0) {
+
+  // check for sRGB
+  if (desc.color_space == image_color_space::srgb) {
+    if (desc.num_channels != 4) {
+      throw sycl::exception(sycl::errc::invalid,
+                            "sRGB color space requires num_channels == 4");
+    }
+    if (desc.channel_type != image_channel_type::unorm_int8) {
+      throw sycl::exception(
+          sycl::errc::invalid,
+          "sRGB color space requires unorm_int8 channel type");
+    }
+  }
   urDesc = {};
   urDesc.stype = UR_STRUCTURE_TYPE_IMAGE_DESC;
   urDesc.width = desc.width;