[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