From 22bdeadc8cf005bdd470c535d160ade123f01854 Mon Sep 17 00:00:00 2001 From: Tobin Ehlis Date: Wed, 4 Jan 2017 08:23:39 -0700 Subject: layers:Remove redundant check Core_validation has a shared to check to verify image layout for both vkCmdClearColorImage() and vkCmdClearDepthStencilImage(). Removing the same check for color images from image layer. --- layers/image.cpp | 8 -------- 1 file changed, 8 deletions(-) diff --git a/layers/image.cpp b/layers/image.cpp index 5899dcf6..45786f8f 100644 --- a/layers/image.cpp +++ b/layers/image.cpp @@ -442,14 +442,6 @@ VKAPI_ATTR void VKAPI_CALL CmdClearColorImage(VkCommandBuffer commandBuffer, VkI bool skipCall = false; layer_data *device_data = get_my_data_ptr(get_dispatch_key(commandBuffer), layer_data_map); - if (imageLayout != VK_IMAGE_LAYOUT_GENERAL && imageLayout != VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL) { - char const str[] = - "vkCmdClearColorImage parameter, imageLayout, must be VK_IMAGE_LAYOUT_GENERAL or VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL"; - skipCall |= log_msg(device_data->report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_COMMAND_BUFFER_EXT, - (uint64_t)commandBuffer, __LINE__, VALIDATION_ERROR_01086, "IMAGE", "%s. %s", str, - validation_error_map[VALIDATION_ERROR_01086]); - } - // For each range, image aspect must be color only // TODO: this is a 'must' in the spec, so there should be a VU enum for it for (uint32_t i = 0; i < rangeCount; i++) { -- cgit v1.2.3