[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