commit 9f2dd0d07c99cc7c274287b408960cca53678bee
author Nicolas Capens <capn@google.com> Thu Mar 26 12:00:35 2020 -0400
committer Nicolas Capens <nicolascapens@google.com> Thu Mar 26 19:15:05 2020 +0000
tree cd4a7305071a9aedee55836640709cd8dff6161f
parent b138adb76149c60b84268ba2a1999e9b3b195c7f

Fix taking aspect into account for image view identifiers

Tests like dEQP.GLES3/functional_fbo_blit_depth_stencil_depth24_stencil8_basic
were not passing when running on SWANGLE.

Bug: b/151235334
Change-Id: Icc3c4709a50c96d4552af4e4835f1e5eb43dd653
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/43028
Presubmit-Ready: Nicolas Capens <nicolascapens@google.com>
Tested-by: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>

src/Vulkan/VkImageView.cpp

diff --git a/src/Vulkan/VkImageView.cpp b/src/Vulkan/VkImageView.cpp
index d9fbaf6..c88fd71 100644
--- a/src/Vulkan/VkImageView.cpp
+++ b/src/Vulkan/VkImageView.cpp
@@ -86,7 +86,7 @@
     , components(ResolveComponentMapping(pCreateInfo->components, format))
     , subresourceRange(ResolveRemainingLevelsLayers(pCreateInfo->subresourceRange, image))
     , ycbcrConversion(ycbcrConversion)
-    , id(image, viewType, format, components)
+    , id(image, viewType, format.getAspectFormat(subresourceRange.aspectMask), components)
 {
 }