aboutsummaryrefslogtreecommitdiff
path: root/layers/buffer_validation.cpp
diff options
context:
space:
mode:
authorChris Forbes <chrisforbes@google.com>2018-05-03 17:36:39 -0700
committerChris Forbes <chrisf@ijw.co.nz>2018-05-08 15:42:02 -0700
commit21f7f79ab1e0bf4d80626692dd53a95ebafe9334 (patch)
tree764319c5c6dec23bc143f3f741d5332e0bf3e8d7 /layers/buffer_validation.cpp
parent8773de85fc302d7c33c33b64b399e021e55190ed (diff)
downloadusermoji-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.cpp16
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]);