[llvm] [DirectX] Fix order of `v2::DescriptorRange` (PR #145555)
Justin Bogner via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 24 12:53:31 PDT 2025
================
@@ -207,11 +207,8 @@ struct DescriptorTableView : RootParameterView {
support::endian::read<uint32_t, llvm::endianness::little>(Current);
Current += sizeof(uint32_t);
- size_t RangeSize = sizeof(T);
-
- Table.Ranges.Stride = RangeSize;
Table.Ranges.Data =
- ParamData.substr(2 * sizeof(uint32_t), Table.NumRanges * RangeSize);
+ ParamData.substr(2 * sizeof(uint32_t), Table.NumRanges * sizeof(T));
----------------
bogner wrote:
Would it be clearer to use `Table.NumRanges * Tabled.Ranges.Stride` here?
https://github.com/llvm/llvm-project/pull/145555
More information about the llvm-commits
mailing list