From 36a26183760a9bedd813acb475c79eba59599a5d Mon Sep 17 00:00:00 2001 From: "Matias N. Goldberg" Date: Tue, 5 Dec 2023 12:07:06 -0300 Subject: [PATCH] Update comment --- RenderSystems/Vulkan/src/Vao/OgreVulkanVaoManager.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/RenderSystems/Vulkan/src/Vao/OgreVulkanVaoManager.cpp b/RenderSystems/Vulkan/src/Vao/OgreVulkanVaoManager.cpp index fea58bafd7b..45ab1c2829d 100644 --- a/RenderSystems/Vulkan/src/Vao/OgreVulkanVaoManager.cpp +++ b/RenderSystems/Vulkan/src/Vao/OgreVulkanVaoManager.cpp @@ -850,6 +850,8 @@ namespace Ogre // - Whether 2 is faster than 3 or viceversa may be HW specific. For example Intel only // exposes Cached + Coherent and nothing else. It seems they have no performance problems // at all. + // + // This assumes CPU write-combining is fast. If it's not, then this is wrong. const uint32 idx = mBestVkMemoryTypeIndex[CPU_WRITE_PERSISTENT_COHERENT].back(); // Prefer coherent memory if it's uncached, or if we have no choice. if( !mSupportsNonCoherentMemory ||