Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Commit 2fabf4e

Browse files
author
Jonah Williams
authored
[Impeller] remove requests for dedicated allocations. (#43686)
This change never improved performance (in fact I think it regressed it slightly), so lets revert it and see if things change.
1 parent 354bc34 commit 2fabf4e

File tree

1 file changed

+2
-11
lines changed

1 file changed

+2
-11
lines changed

impeller/renderer/backend/vulkan/allocator_vk.cc

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
#include "impeller/core/formats.h"
1212
#include "impeller/renderer/backend/vulkan/device_buffer_vk.h"
1313
#include "impeller/renderer/backend/vulkan/formats_vk.h"
14-
#include "impeller/renderer/backend/vulkan/limits_vk.h"
1514
#include "impeller/renderer/backend/vulkan/texture_vk.h"
1615

1716
namespace impeller {
@@ -264,19 +263,12 @@ ToVKTextureMemoryPropertyFlags(StorageMode mode,
264263
FML_UNREACHABLE();
265264
}
266265

267-
static VmaAllocationCreateFlags ToVmaAllocationCreateFlags(StorageMode mode,
268-
size_t size) {
266+
static VmaAllocationCreateFlags ToVmaAllocationCreateFlags(StorageMode mode) {
269267
VmaAllocationCreateFlags flags = 0;
270268
switch (mode) {
271269
case StorageMode::kHostVisible:
272-
if (size >= kImageSizeThresholdForDedicatedMemoryAllocation) {
273-
flags |= VMA_ALLOCATION_CREATE_DEDICATED_MEMORY_BIT;
274-
}
275270
return flags;
276271
case StorageMode::kDevicePrivate:
277-
if (size >= kImageSizeThresholdForDedicatedMemoryAllocation) {
278-
flags |= VMA_ALLOCATION_CREATE_DEDICATED_MEMORY_BIT;
279-
}
280272
return flags;
281273
case StorageMode::kDeviceTransient:
282274
return flags;
@@ -318,8 +310,7 @@ class AllocatedTextureSourceVK final : public TextureSourceVK {
318310
alloc_nfo.preferredFlags =
319311
static_cast<VkMemoryPropertyFlags>(ToVKTextureMemoryPropertyFlags(
320312
desc.storage_mode, supports_memoryless_textures));
321-
alloc_nfo.flags = ToVmaAllocationCreateFlags(
322-
desc.storage_mode, desc.GetByteSizeOfBaseMipLevel());
313+
alloc_nfo.flags = ToVmaAllocationCreateFlags(desc.storage_mode);
323314

324315
auto create_info_native =
325316
static_cast<vk::ImageCreateInfo::NativeType>(image_info);

0 commit comments

Comments
 (0)