[llvm-branch-commits] [clang] [llvm] [HLSL][RootSignature] Add optional parameters for RootConstants (PR #138007)

[Ashley Coleman via llvm-branch-commits]

================
@@ -82,6 +82,8 @@ class RootSignatureParser {
   struct ParsedConstantParams {
     std::optional<llvm::hlsl::rootsig::Register> Reg;
     std::optional<uint32_t> Num32BitConstants;
+    std::optional<uint32_t> Space;
----------------
V-FEXrt wrote:

Yeah I think I agree with Finn here. Seems to map more directly to the source to treat space as a param

https://github.com/llvm/llvm-project/pull/138007