diff options
| author | Chris Forbes <chrisforbes@google.com> | 2017-06-01 11:07:16 -0700 |
|---|---|---|
| committer | Chris Forbes <chrisf@ijw.co.nz> | 2017-06-05 11:56:37 -0700 |
| commit | 393dedb8a2a860e906527dc10bff51232eeb5869 (patch) | |
| tree | 2a5789c8014afc016750ad0baf140405d537f378 /layers/core_validation.cpp | |
| parent | df303c318ddfc6154deb26cbc388a32f94980ce8 (diff) | |
| download | usermoji-393dedb8a2a860e906527dc10bff51232eeb5869.tar.xz | |
layers: remove useless check from GetSwapchainImagesKHR
This is a relic from a /very/ old version of the WSI extensions, and
appears to be checking driver correctness more than app correctness.
Diffstat (limited to 'layers/core_validation.cpp')
| -rw-r--r-- | layers/core_validation.cpp | 13 |
1 files changed, 0 insertions, 13 deletions
diff --git a/layers/core_validation.cpp b/layers/core_validation.cpp index d4179bfc..0c7a3363 100644 --- a/layers/core_validation.cpp +++ b/layers/core_validation.cpp @@ -10388,19 +10388,6 @@ VKAPI_ATTR VkResult VKAPI_CALL GetSwapchainImagesKHR(VkDevice device, VkSwapchai std::lock_guard<std::mutex> lock(global_lock); const size_t count = *pCount; auto swapchain_node = GetSwapchainNode(dev_data, swapchain); - if (swapchain_node && !swapchain_node->images.empty()) { - // TODO : Not sure I like the memcmp here, but it works - const bool mismatch = (swapchain_node->images.size() != count || - memcmp(&swapchain_node->images[0], pSwapchainImages, sizeof(swapchain_node->images[0]) * count)); - if (mismatch) { - // TODO: Verify against Valid Usage section of extension - log_msg(dev_data->report_data, VK_DEBUG_REPORT_WARNING_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_SWAPCHAIN_KHR_EXT, - HandleToUint64(swapchain), __LINE__, MEMTRACK_NONE, "SWAP_CHAIN", - "vkGetSwapchainInfoKHR(0x%" PRIx64 - ", VK_SWAP_CHAIN_INFO_TYPE_PERSISTENT_IMAGES_KHR) returned mismatching data", - HandleToUint64(swapchain)); - } - } for (uint32_t i = 0; i < *pCount; ++i) { IMAGE_LAYOUT_NODE image_layout_node; image_layout_node.layout = VK_IMAGE_LAYOUT_UNDEFINED; |
