Remove vk workaround to do partial clears as draws on adreno.
Qcomm did some investigations of this workaround, and even though it was
giving some cpu savings, it was causing a big hit to GPU performance and
power. So we're removing this now to get back the various wins on the
GPU.
Bug: b/230630340
Change-Id: I01ba4e271dc02ae4ab6155fc794a5a1e3c796341
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/525638
Auto-Submit: Greg Daniel <egdaniel@google.com>
Reviewed-by: Jim Van Verth <jvanverth@google.com>
Commit-Queue: Jim Van Verth <jvanverth@google.com>
(cherry picked from commit 6368d39a9e7ce68c35f37b0e550f1b4274edf070)
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/534657
Reviewed-by: Greg Daniel <egdaniel@google.com>
diff --git a/src/gpu/vk/GrVkCaps.cpp b/src/gpu/vk/GrVkCaps.cpp
index 7ae1f2f..3a61dd5 100644
--- a/src/gpu/vk/GrVkCaps.cpp
+++ b/src/gpu/vk/GrVkCaps.cpp
@@ -679,13 +679,6 @@
if (kARM_VkVendor == properties.vendorID) {
fShouldCollapseSrcOverToSrcWhenAble = true;
}
-
- // We're seeing vkCmdClearAttachments take a lot of cpu time when clearing the color attachment.
- // We really should only be getting in there for partial clears. So instead we will do all
- // partial clears as draws.
- if (kQualcomm_VkVendor == properties.vendorID) {
- fPerformPartialClearsAsDraws = true;
- }
}
void GrVkCaps::initShaderCaps(const VkPhysicalDeviceProperties& properties,
