clspv icon indicating copy to clipboard operation
clspv copied to clipboard

Request for fine grain image requeriments..

Open oscarbg opened this issue 8 years ago • 3 comments

You say "If images are used in the OpenCL C: The shaderStorageImageReadWithoutFormat field of VkPhysicalDeviceFeatures must be set to true. The shaderStorageImageWriteWithoutFormat field of VkPhysicalDeviceFeatures must be set to true. " also seeing vulkan.gpuinfo reports situation is better than int16 support at least modern NV and AMD GPUs support both reqs.. but Intel iGPUs don't support shaderStorageImageReadWithoutFormat neither Windows nor Linux driver..

So as OpenCL has __read_only and __write_only image access qualifiers maybe we can require only shaderStorageImageReadWithoutFormat in __read_only images and shaderStorageImageWriteWithoutFormat in __write_only images .. is that correct?

that way a kernel having only write only image access should work on Intel GPUs right?

thanks..

oscarbg avatar Jul 23 '17 04:07 oscarbg

may have been to fast.. seeing test files included seems includes only what needed.. feel free to close if that's true..

oscarbg avatar Jul 23 '17 07:07 oscarbg

@oscarbg yup! We'll only define the capability when we require it - so a kernel with write_only images would work on Intel GPUs!

sheredom avatar Jul 24 '17 09:07 sheredom

See also https://github.com/google/clspv/issues/5 which documents how the compiler is being overly conservative about declaring those capabilities.

dneto0 avatar Jul 31 '17 18:07 dneto0