Fix segmentation faults when subpass has unused attachments #826
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This fixes two issues related to color attachments in subpasses.
Serialise_vkCmdClearAttachments
uses the render pass attachment index from the subpass description without checking whether the attachment is used in the subpass.VulkanDebugManager::RenderOverlay
uses the wrong attachment indices forvkCmdClearAttachments
. The correct index is the index into the color attachment reference array of the current subpass description, not the index into the attachment description array of the render pass description.Section 17.2 from the Vulkan specification says:
Issue #2 occurs when trying to use the "Clear before draw/pass" overlays in case the attachment index in the render pass description differs from the attachment index in the subpass description.