[llvm-branch-commits] [llvm] [DirectX] Adding missing descriptor table validations (PR #153276)

Finn Plummer via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Tue Aug 26 09:51:06 PDT 2025


================
@@ -547,21 +550,25 @@ Error validateDescriptorTableRegisterOverflow(mcdxbc::DescriptorTable Table,
     dxbc::DescriptorRangeType RangeType =
         static_cast<dxbc::DescriptorRangeType>(Range.RangeType);
 
-    if (verifyOffsetOverflow(Range.OffsetInDescriptorsFromTableStart,
-                             AppendingRegister))
-      return make_error<TableRegisterOverflowError>(
+    uint64_t StartSlot = AppendingRegister;
----------------
inbelic wrote:

nit: `StartSlot` can just be replaced in all uses by `AppendingRegister`. Or if you think `StartSlot` is a better name then we can just use that.

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


More information about the llvm-branch-commits mailing list