diff options
| author | Greg Daniel <egdaniel@google.com> | 2022-03-29 13:53:47 -0400 |
|---|---|---|
| committer | Leon Scroggins <scroggo@google.com> | 2022-04-28 19:08:39 +0000 |
| commit | ae31184592569902150ee460875b30604e8bade0 (patch) | |
| tree | 674ce8dfd37754b25c201ea5bb543fd468171b47 | |
| parent | 43d175748e29b9d299f33631d8b074dbb0597d01 (diff) | |
| download | skia-ae31184592569902150ee460875b30604e8bade0.tar.gz | |
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>
| -rw-r--r-- | src/gpu/vk/GrVkCaps.cpp | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/src/gpu/vk/GrVkCaps.cpp b/src/gpu/vk/GrVkCaps.cpp index 7ae1f2f493..3a61dd5bdb 100644 --- a/src/gpu/vk/GrVkCaps.cpp +++ b/src/gpu/vk/GrVkCaps.cpp @@ -679,13 +679,6 @@ void GrVkCaps::initGrCaps(const GrVkInterface* vkInterface, 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, |