Vulkan: Fix performance issue

c9e0edc2 changed the layerCount in SubresourceUpdate to VK_REMAINING_ARRAY_LAYERS in some cases. So that the deferred clear logic cannot work correctly. Bug: angleproject:4617 Change-Id: Idedada085c618ff900538094412c752522594684 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/2191551Reviewed-by: Jamie Madill <jmadill@chromium.org> Reviewed-by: Geoff Lang <geofflang@chromium.org> Commit-Queue: Charlie Lao <cclao@google.com>

Vulkan: Fix performance issue
664376e2 · Sunny Sun · Commit Bot · c4832968 · 664376e2
Commit 664376e2 authored May 09, 2020 by Sunny Sun Committed by Commit Bot May 14, 2020
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

vk_helpers.cpp src/libANGLE/renderer/vulkan/vk_helpers.cpp +3 -1

No files found.
--- a/src/libANGLE/renderer/vulkan/vk_helpers.cpp
+++ b/src/libANGLE/renderer/vulkan/vk_helpers.cpp
@@ -3720,7 +3720,9 @@ angle::Result ImageHelper::flushSingleSubresourceStagedUpdates(ContextVk *contex
            if (update.isUpdateToLayerLevel(layer, level))
            {
                // On any data update, exit out. We'll need to do a full upload.
-                if (update.updateSource != UpdateSource::Clear || update.clear.layerCount != 1)
+                if (update.updateSource != UpdateSource::Clear ||
+                    (update.clear.layerCount != 1 &&
+                     !(update.clear.layerCount == VK_REMAINING_ARRAY_LAYERS && mLayerCount == 1)))
                {
                    foundClear.reset();
                    break;