commit e4e2d0c57d1602fa06b677d0b6635c4c77f3a1ae
author Jamie Madill <jmadill@chromium.org> Fri Jun 22 08:25:05 2018 -0400
committer Commit Bot <commit-bot@chromium.org> Fri Jun 22 14:57:22 2018 +0000
tree 8ac83e466cb01177ea942af716a2e938f85da4ff
parent 16f391d4459ec1ab76df8e71f7d6d74bc9e5ab46

Vulkan: Add RenderTargetVk::getImageForRead.

This helper method will also transition the Image to the correct read
layout. We will need to revisit the implementation when working on
simulatenous read.

Bug: angleproject:2539
Change-Id: Id61404460f3ef0dbb054e6ac2dfc0b59adb78402
Reviewed-on: https://chromium-review.googlesource.com/1108378
Commit-Queue: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Luc Ferron <lucferron@chromium.org>

src/libANGLE/renderer/vulkan/RenderTargetVk.cpp

diff --git a/src/libANGLE/renderer/vulkan/RenderTargetVk.cpp b/src/libANGLE/renderer/vulkan/RenderTargetVk.cpp
index 4d6cfa9..48c2c10 100644
--- a/src/libANGLE/renderer/vulkan/RenderTargetVk.cpp
+++ b/src/libANGLE/renderer/vulkan/RenderTargetVk.cpp
@@ -104,8 +104,23 @@
     mImageView = imageView;
 }
 
-vk::ImageHelper *RenderTargetVk::getImageForWrite(Serial currentSerial,
-                                                  vk::CommandGraphResource *writingResource) const
+vk::ImageHelper *RenderTargetVk::getImageForRead(vk::CommandGraphResource *readingResource,
+                                                 VkImageLayout layout,
+                                                 VkImageAspectFlags aspectFlags,
+                                                 vk::CommandBuffer *commandBuffer)
+{
+    ASSERT(mImage && mImage->valid());
+
+    // TODO(jmadill): Better simultaneous resource access. http://anglebug.com/2679
+    mResource->addWriteDependency(readingResource);
+
+    mImage->changeLayoutWithStages(aspectFlags, layout, VK_PIPELINE_STAGE_ALL_COMMANDS_BIT,
+                                   VK_PIPELINE_STAGE_ALL_COMMANDS_BIT, commandBuffer);
+
+    return mImage;
+}
+
+vk::ImageHelper *RenderTargetVk::getImageForWrite(vk::CommandGraphResource *writingResource) const
 {
     ASSERT(mImage && mImage->valid());
     mResource->addWriteDependency(writingResource);