[llvm] [DirectX] Adding support for Root Descriptors in obj2yaml/yaml2obj (PR #137259)
via llvm-commits
llvm-commits at lists.llvm.org
Mon May 5 18:05:52 PDT 2025
================
@@ -580,7 +585,28 @@ struct ProgramSignatureElement {
static_assert(sizeof(ProgramSignatureElement) == 32,
"ProgramSignatureElement is misaligned");
+namespace RST0 {
+namespace v0 {
----------------
joaosaffran wrote:
@damyanp @spall @inbelic @bogner I remember one details, DXC uses the versions as 1 and 2: https://github.com/microsoft/DirectXShaderCompiler/blob/1198c30f05ed944873ca55e89970fae407e2aacc/include/dxc/DxilRootSignature/DxilRootSignature.h#L91C1-L92C1. The version is part of the binary container, and it will lead to the parsing of several fields, I am not sure if we should change that here, since I would still need to read and write 1 and 2 as the values identifying the version, I could change those, by doing something like `read(&version); version --;` and `write(++version);` but it seems to be adding unnecessary complexity IMHO. Thoughts ?
https://github.com/llvm/llvm-project/pull/137259
More information about the llvm-commits
mailing list