From: Tobin Ehlis <tobine@google.com>
Date: Wed, 4 Jan 2017 15:23:39 +0000 (-0700)
Subject: layers:Remove redundant check
X-Git-Tag: upstream/1.1.92~1800
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=22bdeadc8cf005bdd470c535d160ade123f01854;p=platform%2Fupstream%2FVulkan-Tools.git

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.
---

diff --git a/layers/image.cpp b/layers/image.cpp
index 5899dcf..45786f8 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++) {