microsoft / DirectXShaderCompiler

This repo hosts the source for the DirectX Shader Compiler which is based on LLVM/Clang.
Other
3.05k stars 677 forks source link

[SPIR-V] Support arrays of RW/append/consume structured buffers #3164

Closed mattruschnv closed 1 year ago

mattruschnv commented 3 years ago

Currently I'm trying to port a shader which uses the following to Vulkan: RWStructuredBuffer VisTLASInstances[] : register(u0, space3);

It would be awesome if this were supported for spirv as well as dxil.

jaebaek commented 3 years ago

@mattruschnv thank you for reporting this issue. First of all, we want to understand exactly what you want. Could you please share a full shader that I can compile?

In addition, if this is not supported for DXIL, then I recommend you not specifying it as "SPIRV" issue. If it is in that case, we need a dicussion with DXIL developers to understand what we are missing (I guess probably AST parser does not support it) and it is possible (or reasonable) to support it.

mattruschnv commented 3 years ago

To clarify, it IS supported for DXIL.

d11mshepcar commented 3 years ago

We'd very much like to see support for this too to enable a bindless model for RWStructuredBuffers in vulkan as well as dx.

s-perron commented 1 year ago

With will be handled under #5440. I want all of the issues in a central place with a fresh conversation.