Vulkan: remove unnecessary transition of external images

Since semaphore has been supported, the code for changing external texture layout and queue family is not necessary anymore. So remove them. Bug: angleproject:3289 Change-Id: I5470fa4bf7db7ee4c68f157674cf907a8e455d75 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/1964655 Commit-Queue: Michael Spang <spang@chromium.org> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org> Reviewed-by: Michael Spang <spang@chromium.org>

Vulkan: remove unnecessary transition of external images
92ba377f · Peng Huang · Commit Bot · 7718c952 · 92ba377f
Commit 92ba377f authored Dec 10, 2019 by Peng Huang Committed by Commit Bot Dec 14, 2019
Hide whitespace changes
Inline Side-by-side

Showing with 0 additions and 12 deletions

TextureVk.cpp src/libANGLE/renderer/vulkan/TextureVk.cpp +0 -12

No files found.
--- a/src/libANGLE/renderer/vulkan/TextureVk.cpp
+++ b/src/libANGLE/renderer/vulkan/TextureVk.cpp
@@ -809,18 +809,6 @@ angle::Result TextureVk::setStorageExternalMemory(const gl::Context *context,
    ANGLE_TRY(initImageViews(contextVk, format, glFormat.info->sized, static_cast<uint32_t>(levels),
                             mImage->getLayerCount()));
-    // TODO(spang): This needs to be reworked when semaphores are added.
-    // http://anglebug.com/3289
-    uint32_t rendererQueueFamilyIndex = renderer->getQueueFamilyIndex();
-    if (mImage->isQueueChangeNeccesary(rendererQueueFamilyIndex))
-    {
-        vk::CommandBuffer *commandBuffer = nullptr;
-        ANGLE_TRY(mImage->recordCommands(contextVk, &commandBuffer));
-        mImage->changeLayoutAndQueue(VK_IMAGE_ASPECT_COLOR_BIT,
-                                     vk::ImageLayout::AllGraphicsShadersReadOnly,
-                                     rendererQueueFamilyIndex, commandBuffer);
-    }
    return angle::Result::Continue;
 }