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

[Finn Plummer via llvm-branch-commits]

================
@@ -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
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits