Skip to content

Validate heap is bound#11692

Closed
ziga-lunarg wants to merge 2 commits intoKhronosGroup:mainfrom
ziga-lunarg:ziga-issue-11688
Closed

Validate heap is bound#11692
ziga-lunarg wants to merge 2 commits intoKhronosGroup:mainfrom
ziga-lunarg:ziga-issue-11688

Conversation

@ziga-lunarg
Copy link
Copy Markdown
Contributor

Closes #11688

@ziga-lunarg ziga-lunarg requested a review from a team as a code owner February 16, 2026 22:10
@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 651072.

@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 651085.

@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build # 22497 running.

return skip;
}
const char *msg = descriptor_mode != vvl::DescriptorModeUnknown
? " (The command buffer has called vkCmdBindDescriptorSets, vkCmdBindDescriptorSets2, vkCmdPushConstants "
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could you add a test that does Descriptor Buffer and Heaps (imagine if someone is porting from buffers to heap and mixing things in the command buffer)

@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build queued with queue ID 651103.

@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build # 22498 running.

@ci-tester-lunarg
Copy link
Copy Markdown
Collaborator

CI Vulkan-ValidationLayers build # 22498 passed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Missing validation for descriptor heap / pre-heap binding clobber

3 participants