diff options
| author | Chris Forbes <chrisforbes@google.com> | 2018-05-03 17:36:39 -0700 |
|---|---|---|
| committer | Chris Forbes <chrisf@ijw.co.nz> | 2018-05-08 15:42:02 -0700 |
| commit | 21f7f79ab1e0bf4d80626692dd53a95ebafe9334 (patch) | |
| tree | 764319c5c6dec23bc143f3f741d5332e0bf3e8d7 /layers/buffer_validation.cpp | |
| parent | 8773de85fc302d7c33c33b64b399e021e55190ed (diff) | |
| download | usermoji-21f7f79ab1e0bf4d80626692dd53a95ebafe9334.tar.xz | |
layers: Rework attachment use conflict checks
Previously buffer_validation covered part of this, but not everything.
V2: - Fix repeated ATTACHMENT_RESOLVE where I meant ATTACHMENT_PRESERVE
- Fix camelcase noise in a bunch of places
- Rename VkAttachmentReference& variables to attachment_ref to avoid
attachment.attachment etc.
- clang-format tidy-up
- Move positive case to VkPositiveLayerTest
Diffstat (limited to 'layers/buffer_validation.cpp')
| -rw-r--r-- | layers/buffer_validation.cpp | 16 |
1 files changed, 0 insertions, 16 deletions
diff --git a/layers/buffer_validation.cpp b/layers/buffer_validation.cpp index cb548703..dd44c7a5 100644 --- a/layers/buffer_validation.cpp +++ b/layers/buffer_validation.cpp @@ -2860,22 +2860,6 @@ bool ValidateLayouts(core_validation::layer_data *device_data, VkDevice device, string_VkImageLayout(subpass.pInputAttachments[j].layout)); } - VkImageLayout layout = subpass.pInputAttachments[j].layout; - bool found_layout_mismatch = subpass.pDepthStencilAttachment && - subpass.pDepthStencilAttachment->attachment == attach_index && - subpass.pDepthStencilAttachment->layout != layout; - for (uint32_t c = 0; !found_layout_mismatch && c < subpass.colorAttachmentCount; ++c) { - found_layout_mismatch = - (subpass.pColorAttachments[c].attachment == attach_index && subpass.pColorAttachments[c].layout != layout); - } - if (found_layout_mismatch) { - skip |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0, - VALIDATION_ERROR_140006ae, - "CreateRenderPass: Subpass %u pInputAttachments[%u] (%u) has layout %u, but is also used as a " - "depth/color attachment with a different layout.", - i, j, attach_index, layout); - } - if (attach_first_use[attach_index]) { skip |= ValidateLayoutVsAttachmentDescription(report_data, subpass.pInputAttachments[j].layout, attach_index, pCreateInfo->pAttachments[attach_index]); |
