diff options
| author | Mark Lobodzinski <mark@lunarg.com> | 2017-01-18 14:52:22 -0700 |
|---|---|---|
| committer | Mark Lobodzinski <mark@lunarg.com> | 2017-01-19 13:46:48 -0700 |
| commit | cbcf98910b1894480436953d9d1182da5a23e958 (patch) | |
| tree | 7dba04a2ce8693702ef5fb27e3481984f33a46c4 /layers/core_validation.cpp | |
| parent | e9d84d2b0da3d1e8c5f2059325351ba2fe415a0a (diff) | |
| download | usermoji-cbcf98910b1894480436953d9d1182da5a23e958.tar.xz | |
layers: Remove redundant CV ClearAttachments check
This check was incorrectly labeled, and the actual check is present
in the image layer.
Change-Id: I29b866eb9da731e0996b95da3052b4878e906602
Diffstat (limited to 'layers/core_validation.cpp')
| -rw-r--r-- | layers/core_validation.cpp | 11 |
1 files changed, 0 insertions, 11 deletions
diff --git a/layers/core_validation.cpp b/layers/core_validation.cpp index b60a879f..3dd80450 100644 --- a/layers/core_validation.cpp +++ b/layers/core_validation.cpp @@ -8918,17 +8918,6 @@ VKAPI_ATTR void VKAPI_CALL CmdClearAttachments(VkCommandBuffer commandBuffer, ui if (image_view) { auto image_view_state = getImageViewState(dev_data, image_view); - auto aspects_present = image_view_state->create_info.subresourceRange.aspectMask; - auto extra_aspects = clear_desc->aspectMask & ~aspects_present; - // TODO: This is a different check than 01125. Need a new valid usage statement for this case, or should kill check. - if (extra_aspects) { - skip_call |= - log_msg(dev_data->report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_IMAGE_VIEW_EXT, - reinterpret_cast<uint64_t &>(image_view), __LINE__, VALIDATION_ERROR_01125, "DS", - "vkCmdClearAttachments() with aspects not present in image view: %s. %s", - string_VkImageAspectFlagBits((VkImageAspectFlagBits)extra_aspects), - validation_error_map[VALIDATION_ERROR_01125]); - } for (uint32_t j = 0; j < rectCount; j++) { // The rectangular region specified by a given element of pRects must be contained within the render area of the // current render pass instance |