Support for shaderSampledImageArrayDynamicIndexing physical device feature about moltenvk HOT 6 CLOSED

Oh, but it turns out we already do this! (I just need to let it be used in MSL 1.2.)

It should actually be safe to turn on shaderSampledImageArrayDynamicIndexing for Metal 1.2 or higher right now. Possibly shaderStorageImageArrayDynamicIndexing as well, with the caveat that it won't work for images that are written to.

from moltenvk.

Or maybe it will work for written images. The docs lied to me!

from moltenvk.

That sounds great, enabling shaderSampledImageArrayDynamicIndexing is enough for us at the moment.

from moltenvk.

This feature would really be appreciated, any plans for it?

from moltenvk.

@cdavis5e With your latest resource indexing enhancements...you are probably closer to this than I at present. Any thoughts about the viability of this enhancement?

from moltenvk.

We'd need to have SPIRV-Cross translate arrays of textures to metal::array objects. This is supported as of Metal 1.2. Unfortunately, this is somewhat complicated by the fact that arrays of samplers aren't supported until Metal 2.0. This is a problem primarily for combined image/sampler objects--with MSL 1.2, we'd be able to have an array of the image parts, but not the sampler parts.

from moltenvk.