[clang] [llvm] [HLSL][RootSignature] Add infastructure to parse parameters (PR #133800)
Finn Plummer via cfe-commits
cfe-commits at lists.llvm.org
Thu Apr 17 12:33:08 PDT 2025
inbelic wrote:
I don't have any strong arguments against structuring it like described and have updated accordingly.
Since we are changing the calling convention of the `parse.*` methods to use `std::optional` I had originally also updated the `parseDescriptorTable` and `parseDescriptorTableClause` as well. But it made the diff difficult to read.
Changes to make the calling convention consistent with those methods are in the branch [here](https://github.com/inbelic/llvm-project/tree/inbelic/rs-update-call-convs). I will create the clean up pr once this one merges
https://github.com/llvm/llvm-project/pull/133800
More information about the cfe-commits
mailing list