Skip to content

Add optional flags to some extensions

Romain Failliot requested to merge (removed):add-optional-flags into main

Disclaimer: I am not OpenCL or Vulkan dev, I chose the extension to flag based on the issues I received in the mesamsatrix.net repo:

Marking some extensions as optional would help mesamatrix.net expose the right state of each driver (i.e. if a driver has all the extensions of an API except the optional ones, then it implements this API version).

As suggested in #5160, I added (optional) next to extensions that are optionals. I only flagged two of them:

  • In Vulkan 1.1: VK_KHR_sampler_ycbcr_conversion, as the documentation says "However, if Vulkan 1.1 is supported and this extension is not, the samplerYcbcrConversion capability is optional." So it seems to me that this extension is optional
  • In OpenCL: "Image support", as it is apparently well-known that this extension is optional

I didn't include VK_KHR_shader_draw_parameters as I'm not sure, by reading the documentation, that it's really optional. I'll let the professionals determine that 😉

Merge request reports