Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 1 | // |
| 2 | // Copyright 2016 The ANGLE Project Authors. All rights reserved. |
| 3 | // Use of this source code is governed by a BSD-style license that can be |
| 4 | // found in the LICENSE file. |
| 5 | // |
| 6 | // FramebufferVk.cpp: |
| 7 | // Implements the class methods for FramebufferVk. |
| 8 | // |
| 9 | |
| 10 | #include "libANGLE/renderer/vulkan/FramebufferVk.h" |
| 11 | |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 12 | #include <vulkan/vulkan.h> |
Jamie Madill | 231c7f5 | 2017-04-26 13:45:37 -0400 | [diff] [blame] | 13 | #include <array> |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 14 | |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 15 | #include "common/debug.h" |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 16 | #include "image_util/imageformats.h" |
Jamie Madill | c564c07 | 2017-06-01 12:45:42 -0400 | [diff] [blame] | 17 | #include "libANGLE/Context.h" |
| 18 | #include "libANGLE/Display.h" |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 19 | #include "libANGLE/formatutils.h" |
| 20 | #include "libANGLE/renderer/renderer_utils.h" |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 21 | #include "libANGLE/renderer/vulkan/CommandBufferNode.h" |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 22 | #include "libANGLE/renderer/vulkan/ContextVk.h" |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 23 | #include "libANGLE/renderer/vulkan/DisplayVk.h" |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 24 | #include "libANGLE/renderer/vulkan/RenderTargetVk.h" |
| 25 | #include "libANGLE/renderer/vulkan/RendererVk.h" |
| 26 | #include "libANGLE/renderer/vulkan/SurfaceVk.h" |
Jamie Madill | 3c424b4 | 2018-01-19 12:35:09 -0500 | [diff] [blame] | 27 | #include "libANGLE/renderer/vulkan/vk_format_utils.h" |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 28 | |
| 29 | namespace rx |
| 30 | { |
| 31 | |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 32 | namespace |
| 33 | { |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 34 | gl::ErrorOrResult<const gl::InternalFormat *> GetReadAttachmentInfo( |
Jamie Madill | 4928b7c | 2017-06-20 12:57:39 -0400 | [diff] [blame] | 35 | const gl::Context *context, |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 36 | const gl::FramebufferAttachment *readAttachment) |
| 37 | { |
| 38 | RenderTargetVk *renderTarget = nullptr; |
Jamie Madill | 4928b7c | 2017-06-20 12:57:39 -0400 | [diff] [blame] | 39 | ANGLE_TRY(readAttachment->getRenderTarget(context, &renderTarget)); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 40 | |
Jamie Madill | 1d7be50 | 2017-10-29 18:06:50 -0400 | [diff] [blame] | 41 | GLenum implFormat = renderTarget->format->textureFormat().fboImplementationInternalFormat; |
Geoff Lang | ca27139 | 2017-04-05 12:30:00 -0400 | [diff] [blame] | 42 | return &gl::GetSizedInternalFormatInfo(implFormat); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 43 | } |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 44 | } // anonymous namespace |
| 45 | |
| 46 | // static |
| 47 | FramebufferVk *FramebufferVk::CreateUserFBO(const gl::FramebufferState &state) |
| 48 | { |
| 49 | return new FramebufferVk(state); |
| 50 | } |
| 51 | |
| 52 | // static |
| 53 | FramebufferVk *FramebufferVk::CreateDefaultFBO(const gl::FramebufferState &state, |
| 54 | WindowSurfaceVk *backbuffer) |
| 55 | { |
| 56 | return new FramebufferVk(state, backbuffer); |
| 57 | } |
| 58 | |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 59 | FramebufferVk::FramebufferVk(const gl::FramebufferState &state) |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 60 | : FramebufferImpl(state), |
| 61 | mBackbuffer(nullptr), |
| 62 | mRenderPassDesc(), |
| 63 | mFramebuffer(), |
Jamie Madill | 47c8ea3 | 2018-01-03 18:27:59 -0500 | [diff] [blame] | 64 | mLastRenderNodeSerial() |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 65 | { |
| 66 | } |
| 67 | |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 68 | FramebufferVk::FramebufferVk(const gl::FramebufferState &state, WindowSurfaceVk *backbuffer) |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 69 | : FramebufferImpl(state), |
| 70 | mBackbuffer(backbuffer), |
| 71 | mRenderPassDesc(), |
| 72 | mFramebuffer(), |
Jamie Madill | 47c8ea3 | 2018-01-03 18:27:59 -0500 | [diff] [blame] | 73 | mLastRenderNodeSerial() |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 74 | { |
| 75 | } |
| 76 | |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 77 | FramebufferVk::~FramebufferVk() |
| 78 | { |
| 79 | } |
| 80 | |
Jamie Madill | c564c07 | 2017-06-01 12:45:42 -0400 | [diff] [blame] | 81 | void FramebufferVk::destroy(const gl::Context *context) |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 82 | { |
Jamie Madill | e1f3ad4 | 2017-10-28 23:00:42 -0400 | [diff] [blame] | 83 | RendererVk *renderer = vk::GetImpl(context)->getRenderer(); |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 84 | |
Jamie Madill | 526543c | 2017-10-28 10:59:16 -0400 | [diff] [blame] | 85 | renderer->releaseResource(*this, &mFramebuffer); |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 86 | } |
| 87 | |
Jamie Madill | c564c07 | 2017-06-01 12:45:42 -0400 | [diff] [blame] | 88 | void FramebufferVk::destroyDefault(const egl::Display *display) |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 89 | { |
Jamie Madill | e1f3ad4 | 2017-10-28 23:00:42 -0400 | [diff] [blame] | 90 | VkDevice device = vk::GetImpl(display)->getRenderer()->getDevice(); |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 91 | |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 92 | mFramebuffer.destroy(device); |
| 93 | } |
| 94 | |
Jamie Madill | 4928b7c | 2017-06-20 12:57:39 -0400 | [diff] [blame] | 95 | gl::Error FramebufferVk::discard(const gl::Context *context, |
| 96 | size_t count, |
| 97 | const GLenum *attachments) |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 98 | { |
| 99 | UNIMPLEMENTED(); |
Yuly Novikov | c4d18aa | 2017-03-09 18:45:02 -0500 | [diff] [blame] | 100 | return gl::InternalError(); |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 101 | } |
| 102 | |
Jamie Madill | 4928b7c | 2017-06-20 12:57:39 -0400 | [diff] [blame] | 103 | gl::Error FramebufferVk::invalidate(const gl::Context *context, |
| 104 | size_t count, |
| 105 | const GLenum *attachments) |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 106 | { |
| 107 | UNIMPLEMENTED(); |
Yuly Novikov | c4d18aa | 2017-03-09 18:45:02 -0500 | [diff] [blame] | 108 | return gl::InternalError(); |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 109 | } |
| 110 | |
Jamie Madill | 4928b7c | 2017-06-20 12:57:39 -0400 | [diff] [blame] | 111 | gl::Error FramebufferVk::invalidateSub(const gl::Context *context, |
| 112 | size_t count, |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 113 | const GLenum *attachments, |
| 114 | const gl::Rectangle &area) |
| 115 | { |
| 116 | UNIMPLEMENTED(); |
Yuly Novikov | c4d18aa | 2017-03-09 18:45:02 -0500 | [diff] [blame] | 117 | return gl::InternalError(); |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 118 | } |
| 119 | |
Jamie Madill | c564c07 | 2017-06-01 12:45:42 -0400 | [diff] [blame] | 120 | gl::Error FramebufferVk::clear(const gl::Context *context, GLbitfield mask) |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 121 | { |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 122 | if (mState.getDepthAttachment() && (mask & GL_DEPTH_BUFFER_BIT) != 0) |
| 123 | { |
| 124 | // TODO(jmadill): Depth clear |
| 125 | UNIMPLEMENTED(); |
| 126 | } |
| 127 | |
| 128 | if (mState.getStencilAttachment() && (mask & GL_STENCIL_BUFFER_BIT) != 0) |
| 129 | { |
| 130 | // TODO(jmadill): Stencil clear |
| 131 | UNIMPLEMENTED(); |
| 132 | } |
| 133 | |
| 134 | if ((mask & GL_COLOR_BUFFER_BIT) == 0) |
| 135 | { |
| 136 | return gl::NoError(); |
| 137 | } |
| 138 | |
| 139 | const auto &glState = context->getGLState(); |
| 140 | const auto &clearColor = glState.getColorClearValue(); |
| 141 | VkClearColorValue clearColorValue; |
| 142 | clearColorValue.float32[0] = clearColor.red; |
| 143 | clearColorValue.float32[1] = clearColor.green; |
| 144 | clearColorValue.float32[2] = clearColor.blue; |
| 145 | clearColorValue.float32[3] = clearColor.alpha; |
| 146 | |
| 147 | // TODO(jmadill): Scissored clears. |
| 148 | const auto *attachment = mState.getFirstNonNullAttachment(); |
| 149 | ASSERT(attachment && attachment->isAttached()); |
| 150 | const auto &size = attachment->getSize(); |
| 151 | const gl::Rectangle renderArea(0, 0, size.width, size.height); |
| 152 | |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 153 | RendererVk *renderer = vk::GetImpl(context)->getRenderer(); |
| 154 | |
| 155 | vk::CommandBuffer *commandBuffer = nullptr; |
| 156 | ANGLE_TRY(recordWriteCommands(renderer, &commandBuffer)); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 157 | |
Jamie Madill | efb5a5c | 2018-01-29 15:56:59 -0500 | [diff] [blame^] | 158 | Serial currentSerial = renderer->getCurrentQueueSerial(); |
| 159 | |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 160 | for (const auto &colorAttachment : mState.getColorAttachments()) |
| 161 | { |
| 162 | if (colorAttachment.isAttached()) |
| 163 | { |
| 164 | RenderTargetVk *renderTarget = nullptr; |
Jamie Madill | 4928b7c | 2017-06-20 12:57:39 -0400 | [diff] [blame] | 165 | ANGLE_TRY(colorAttachment.getRenderTarget(context, &renderTarget)); |
Jamie Madill | 47c8ea3 | 2018-01-03 18:27:59 -0500 | [diff] [blame] | 166 | |
Jamie Madill | efb5a5c | 2018-01-29 15:56:59 -0500 | [diff] [blame^] | 167 | renderTarget->resource->setWriteNode(getCurrentWriteNode(currentSerial), currentSerial); |
| 168 | |
Jamie Madill | 47c8ea3 | 2018-01-03 18:27:59 -0500 | [diff] [blame] | 169 | renderTarget->image->changeLayoutWithStages( |
| 170 | VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL, |
| 171 | VK_PIPELINE_STAGE_ALL_COMMANDS_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, commandBuffer); |
| 172 | |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 173 | commandBuffer->clearSingleColorImage(*renderTarget->image, clearColorValue); |
| 174 | } |
| 175 | } |
| 176 | |
Jamie Madill | efb5a5c | 2018-01-29 15:56:59 -0500 | [diff] [blame^] | 177 | // TODO(jmadill): Depth/stencil clear. |
| 178 | |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 179 | return gl::NoError(); |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 180 | } |
| 181 | |
Jamie Madill | c564c07 | 2017-06-01 12:45:42 -0400 | [diff] [blame] | 182 | gl::Error FramebufferVk::clearBufferfv(const gl::Context *context, |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 183 | GLenum buffer, |
| 184 | GLint drawbuffer, |
| 185 | const GLfloat *values) |
| 186 | { |
| 187 | UNIMPLEMENTED(); |
Yuly Novikov | c4d18aa | 2017-03-09 18:45:02 -0500 | [diff] [blame] | 188 | return gl::InternalError(); |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 189 | } |
| 190 | |
Jamie Madill | c564c07 | 2017-06-01 12:45:42 -0400 | [diff] [blame] | 191 | gl::Error FramebufferVk::clearBufferuiv(const gl::Context *context, |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 192 | GLenum buffer, |
| 193 | GLint drawbuffer, |
| 194 | const GLuint *values) |
| 195 | { |
| 196 | UNIMPLEMENTED(); |
Yuly Novikov | c4d18aa | 2017-03-09 18:45:02 -0500 | [diff] [blame] | 197 | return gl::InternalError(); |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 198 | } |
| 199 | |
Jamie Madill | c564c07 | 2017-06-01 12:45:42 -0400 | [diff] [blame] | 200 | gl::Error FramebufferVk::clearBufferiv(const gl::Context *context, |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 201 | GLenum buffer, |
| 202 | GLint drawbuffer, |
| 203 | const GLint *values) |
| 204 | { |
| 205 | UNIMPLEMENTED(); |
Yuly Novikov | c4d18aa | 2017-03-09 18:45:02 -0500 | [diff] [blame] | 206 | return gl::InternalError(); |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 207 | } |
| 208 | |
Jamie Madill | c564c07 | 2017-06-01 12:45:42 -0400 | [diff] [blame] | 209 | gl::Error FramebufferVk::clearBufferfi(const gl::Context *context, |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 210 | GLenum buffer, |
| 211 | GLint drawbuffer, |
| 212 | GLfloat depth, |
| 213 | GLint stencil) |
| 214 | { |
| 215 | UNIMPLEMENTED(); |
Yuly Novikov | c4d18aa | 2017-03-09 18:45:02 -0500 | [diff] [blame] | 216 | return gl::InternalError(); |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 217 | } |
| 218 | |
Jamie Madill | 4928b7c | 2017-06-20 12:57:39 -0400 | [diff] [blame] | 219 | GLenum FramebufferVk::getImplementationColorReadFormat(const gl::Context *context) const |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 220 | { |
Jamie Madill | 4928b7c | 2017-06-20 12:57:39 -0400 | [diff] [blame] | 221 | auto errOrResult = GetReadAttachmentInfo(context, mState.getReadAttachment()); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 222 | |
| 223 | // TODO(jmadill): Handle getRenderTarget error. |
| 224 | if (errOrResult.isError()) |
| 225 | { |
Yuly Novikov | d73f852 | 2017-01-13 17:48:57 -0500 | [diff] [blame] | 226 | ERR() << "Internal error in FramebufferVk::getImplementationColorReadFormat."; |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 227 | return GL_NONE; |
| 228 | } |
| 229 | |
| 230 | return errOrResult.getResult()->format; |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 231 | } |
| 232 | |
Jamie Madill | 4928b7c | 2017-06-20 12:57:39 -0400 | [diff] [blame] | 233 | GLenum FramebufferVk::getImplementationColorReadType(const gl::Context *context) const |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 234 | { |
Jamie Madill | 4928b7c | 2017-06-20 12:57:39 -0400 | [diff] [blame] | 235 | auto errOrResult = GetReadAttachmentInfo(context, mState.getReadAttachment()); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 236 | |
| 237 | // TODO(jmadill): Handle getRenderTarget error. |
| 238 | if (errOrResult.isError()) |
| 239 | { |
Yuly Novikov | d73f852 | 2017-01-13 17:48:57 -0500 | [diff] [blame] | 240 | ERR() << "Internal error in FramebufferVk::getImplementationColorReadFormat."; |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 241 | return GL_NONE; |
| 242 | } |
| 243 | |
| 244 | return errOrResult.getResult()->type; |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 245 | } |
| 246 | |
Jamie Madill | c564c07 | 2017-06-01 12:45:42 -0400 | [diff] [blame] | 247 | gl::Error FramebufferVk::readPixels(const gl::Context *context, |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 248 | const gl::Rectangle &area, |
| 249 | GLenum format, |
| 250 | GLenum type, |
Jamie Madill | d482615 | 2017-09-21 11:18:59 -0400 | [diff] [blame] | 251 | void *pixels) |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 252 | { |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 253 | const auto &glState = context->getGLState(); |
| 254 | const auto *readFramebuffer = glState.getReadFramebuffer(); |
| 255 | const auto *readAttachment = readFramebuffer->getReadColorbuffer(); |
| 256 | |
| 257 | RenderTargetVk *renderTarget = nullptr; |
Jamie Madill | 4928b7c | 2017-06-20 12:57:39 -0400 | [diff] [blame] | 258 | ANGLE_TRY(readAttachment->getRenderTarget(context, &renderTarget)); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 259 | |
Jamie Madill | e1f3ad4 | 2017-10-28 23:00:42 -0400 | [diff] [blame] | 260 | ContextVk *contextVk = vk::GetImpl(context); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 261 | RendererVk *renderer = contextVk->getRenderer(); |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 262 | VkDevice device = renderer->getDevice(); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 263 | |
| 264 | vk::Image *readImage = renderTarget->image; |
| 265 | vk::StagingImage stagingImage; |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 266 | ANGLE_TRY(renderer->createStagingImage(TextureDimension::TEX_2D, *renderTarget->format, |
Jamie Madill | 035fd6b | 2017-10-03 15:43:22 -0400 | [diff] [blame] | 267 | renderTarget->extents, vk::StagingUsage::Read, |
| 268 | &stagingImage)); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 269 | |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 270 | vk::CommandBuffer *commandBuffer = nullptr; |
| 271 | ANGLE_TRY(recordWriteCommands(renderer, &commandBuffer)); |
Jamie Madill | d482615 | 2017-09-21 11:18:59 -0400 | [diff] [blame] | 272 | |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 273 | stagingImage.getImage().changeLayoutTop(VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL, |
| 274 | commandBuffer); |
| 275 | |
Jamie Madill | d33c77c | 2017-11-09 13:08:30 -0500 | [diff] [blame] | 276 | readImage->changeLayoutWithStages( |
| 277 | VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_TRANSFER_SRC_OPTIMAL, |
| 278 | VK_PIPELINE_STAGE_ALL_COMMANDS_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, commandBuffer); |
Jamie Madill | 815a6c9 | 2017-10-21 14:33:04 -0400 | [diff] [blame] | 279 | |
| 280 | VkImageCopy region; |
| 281 | region.srcSubresource.aspectMask = VK_IMAGE_ASPECT_COLOR_BIT; |
| 282 | region.srcSubresource.mipLevel = 0; |
| 283 | region.srcSubresource.baseArrayLayer = 0; |
| 284 | region.srcSubresource.layerCount = 1; |
| 285 | region.srcOffset.x = area.x; |
| 286 | region.srcOffset.y = area.y; |
| 287 | region.srcOffset.z = 0; |
| 288 | region.dstSubresource.aspectMask = VK_IMAGE_ASPECT_COLOR_BIT; |
| 289 | region.dstSubresource.mipLevel = 0; |
| 290 | region.dstSubresource.baseArrayLayer = 0; |
| 291 | region.dstSubresource.layerCount = 1; |
| 292 | region.dstOffset.x = 0; |
| 293 | region.dstOffset.y = 0; |
| 294 | region.dstOffset.z = 0; |
| 295 | region.extent.width = area.width; |
| 296 | region.extent.height = area.height; |
| 297 | region.extent.depth = 1; |
| 298 | |
| 299 | commandBuffer->copyImage(*readImage, stagingImage.getImage(), 1, ®ion); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 300 | |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 301 | // Triggers a full finish. |
| 302 | // TODO(jmadill): Don't block on asynchronous readback. |
| 303 | ANGLE_TRY(renderer->finish(context)); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 304 | |
| 305 | // TODO(jmadill): parameters |
| 306 | uint8_t *mapPointer = nullptr; |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 307 | ANGLE_TRY( |
| 308 | stagingImage.getDeviceMemory().map(device, 0, stagingImage.getSize(), 0, &mapPointer)); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 309 | |
Jamie Madill | 1d7be50 | 2017-10-29 18:06:50 -0400 | [diff] [blame] | 310 | const auto &angleFormat = renderTarget->format->textureFormat(); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 311 | |
| 312 | // TODO(jmadill): Use pixel bytes from the ANGLE format directly. |
Geoff Lang | ca27139 | 2017-04-05 12:30:00 -0400 | [diff] [blame] | 313 | const auto &glFormat = gl::GetSizedInternalFormatInfo(angleFormat.glInternalFormat); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 314 | int inputPitch = glFormat.pixelBytes * area.width; |
| 315 | |
| 316 | PackPixelsParams params; |
| 317 | params.area = area; |
| 318 | params.format = format; |
| 319 | params.type = type; |
| 320 | params.outputPitch = inputPitch; |
Corentin Wallez | 336129f | 2017-10-17 15:55:40 -0400 | [diff] [blame] | 321 | params.packBuffer = glState.getTargetBuffer(gl::BufferBinding::PixelPack); |
Corentin Wallez | cda6af1 | 2017-10-30 19:20:37 -0400 | [diff] [blame] | 322 | params.pack = glState.getPackState(); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 323 | |
| 324 | PackPixels(params, angleFormat, inputPitch, mapPointer, reinterpret_cast<uint8_t *>(pixels)); |
| 325 | |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 326 | stagingImage.getDeviceMemory().unmap(device); |
Jamie Madill | e88ec8e | 2017-10-31 17:18:14 -0400 | [diff] [blame] | 327 | renderer->releaseObject(renderer->getCurrentQueueSerial(), &stagingImage); |
Jamie Madill | f651c77 | 2017-02-21 15:03:51 -0500 | [diff] [blame] | 328 | |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 329 | return vk::NoError(); |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 330 | } |
| 331 | |
Jamie Madill | c564c07 | 2017-06-01 12:45:42 -0400 | [diff] [blame] | 332 | gl::Error FramebufferVk::blit(const gl::Context *context, |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 333 | const gl::Rectangle &sourceArea, |
| 334 | const gl::Rectangle &destArea, |
| 335 | GLbitfield mask, |
| 336 | GLenum filter) |
| 337 | { |
| 338 | UNIMPLEMENTED(); |
Yuly Novikov | c4d18aa | 2017-03-09 18:45:02 -0500 | [diff] [blame] | 339 | return gl::InternalError(); |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 340 | } |
| 341 | |
Kenneth Russell | ce8602a | 2017-10-03 18:23:08 -0700 | [diff] [blame] | 342 | bool FramebufferVk::checkStatus(const gl::Context *context) const |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 343 | { |
Jamie Madill | b79e7bb | 2017-10-24 13:55:50 -0400 | [diff] [blame] | 344 | return true; |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 345 | } |
| 346 | |
Jamie Madill | c564c07 | 2017-06-01 12:45:42 -0400 | [diff] [blame] | 347 | void FramebufferVk::syncState(const gl::Context *context, |
| 348 | const gl::Framebuffer::DirtyBits &dirtyBits) |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 349 | { |
Jamie Madill | e1f3ad4 | 2017-10-28 23:00:42 -0400 | [diff] [blame] | 350 | ContextVk *contextVk = vk::GetImpl(context); |
Jamie Madill | 7bd1666 | 2017-10-28 19:40:50 -0400 | [diff] [blame] | 351 | RendererVk *renderer = contextVk->getRenderer(); |
Jamie Madill | dd43e6c | 2017-03-24 14:18:49 -0400 | [diff] [blame] | 352 | |
| 353 | ASSERT(dirtyBits.any()); |
| 354 | |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 355 | // TODO(jmadill): Smarter update. |
Jamie Madill | 9f2a861 | 2017-11-30 12:43:09 -0500 | [diff] [blame] | 356 | mRenderPassDesc.reset(); |
Jamie Madill | 7bd1666 | 2017-10-28 19:40:50 -0400 | [diff] [blame] | 357 | renderer->releaseResource(*this, &mFramebuffer); |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 358 | |
| 359 | // Trigger a new set of secondary commands next time we render to this FBO,. |
Jamie Madill | 47c8ea3 | 2018-01-03 18:27:59 -0500 | [diff] [blame] | 360 | mLastRenderNodeSerial = Serial(); |
Jamie Madill | 7210656 | 2017-03-24 14:18:50 -0400 | [diff] [blame] | 361 | |
Jamie Madill | 7210656 | 2017-03-24 14:18:50 -0400 | [diff] [blame] | 362 | contextVk->invalidateCurrentPipeline(); |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 363 | } |
| 364 | |
Jamie Madill | 9f2a861 | 2017-11-30 12:43:09 -0500 | [diff] [blame] | 365 | const vk::RenderPassDesc &FramebufferVk::getRenderPassDesc(const gl::Context *context) |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 366 | { |
Jamie Madill | 9f2a861 | 2017-11-30 12:43:09 -0500 | [diff] [blame] | 367 | if (mRenderPassDesc.valid()) |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 368 | { |
Jamie Madill | 9f2a861 | 2017-11-30 12:43:09 -0500 | [diff] [blame] | 369 | return mRenderPassDesc.value(); |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 370 | } |
| 371 | |
Jamie Madill | 0b684ce | 2017-11-23 12:57:39 -0500 | [diff] [blame] | 372 | vk::RenderPassDesc desc; |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 373 | |
| 374 | const auto &colorAttachments = mState.getColorAttachments(); |
| 375 | for (size_t attachmentIndex = 0; attachmentIndex < colorAttachments.size(); ++attachmentIndex) |
| 376 | { |
| 377 | const auto &colorAttachment = colorAttachments[attachmentIndex]; |
| 378 | if (colorAttachment.isAttached()) |
| 379 | { |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 380 | RenderTargetVk *renderTarget = nullptr; |
Jamie Madill | 9f2a861 | 2017-11-30 12:43:09 -0500 | [diff] [blame] | 381 | ANGLE_SWALLOW_ERR(colorAttachment.getRenderTarget(context, &renderTarget)); |
Jamie Madill | bef918c | 2017-12-13 13:11:30 -0500 | [diff] [blame] | 382 | desc.packColorAttachment(*renderTarget->format, colorAttachment.getSamples()); |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 383 | } |
| 384 | } |
| 385 | |
| 386 | const auto *depthStencilAttachment = mState.getDepthStencilAttachment(); |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 387 | |
Jamie Madill | 0b684ce | 2017-11-23 12:57:39 -0500 | [diff] [blame] | 388 | if (depthStencilAttachment && depthStencilAttachment->isAttached()) |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 389 | { |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 390 | RenderTargetVk *renderTarget = nullptr; |
Jamie Madill | 9f2a861 | 2017-11-30 12:43:09 -0500 | [diff] [blame] | 391 | ANGLE_SWALLOW_ERR(depthStencilAttachment->getRenderTarget(context, &renderTarget)); |
Jamie Madill | bef918c | 2017-12-13 13:11:30 -0500 | [diff] [blame] | 392 | desc.packDepthStencilAttachment(*renderTarget->format, |
| 393 | depthStencilAttachment->getSamples()); |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 394 | } |
| 395 | |
Jamie Madill | 9f2a861 | 2017-11-30 12:43:09 -0500 | [diff] [blame] | 396 | mRenderPassDesc = desc; |
| 397 | return mRenderPassDesc.value(); |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 398 | } |
| 399 | |
Jamie Madill | 4928b7c | 2017-06-20 12:57:39 -0400 | [diff] [blame] | 400 | gl::ErrorOrResult<vk::Framebuffer *> FramebufferVk::getFramebuffer(const gl::Context *context, |
Jamie Madill | 9f2a861 | 2017-11-30 12:43:09 -0500 | [diff] [blame] | 401 | RendererVk *rendererVk) |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 402 | { |
| 403 | // If we've already created our cached Framebuffer, return it. |
Jamie Madill | dd43e6c | 2017-03-24 14:18:49 -0400 | [diff] [blame] | 404 | if (mFramebuffer.valid()) |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 405 | { |
| 406 | return &mFramebuffer; |
| 407 | } |
| 408 | |
Jamie Madill | 9f2a861 | 2017-11-30 12:43:09 -0500 | [diff] [blame] | 409 | const vk::RenderPassDesc &desc = getRenderPassDesc(context); |
| 410 | |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 411 | vk::RenderPass *renderPass = nullptr; |
Jamie Madill | 9f2a861 | 2017-11-30 12:43:09 -0500 | [diff] [blame] | 412 | ANGLE_TRY(rendererVk->getCompatibleRenderPass(desc, &renderPass)); |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 413 | |
| 414 | // If we've a Framebuffer provided by a Surface (default FBO/backbuffer), query it. |
Jamie Madill | 9f2a861 | 2017-11-30 12:43:09 -0500 | [diff] [blame] | 415 | VkDevice device = rendererVk->getDevice(); |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 416 | if (mBackbuffer) |
| 417 | { |
| 418 | return mBackbuffer->getCurrentFramebuffer(device, *renderPass); |
| 419 | } |
| 420 | |
| 421 | // Gather VkImageViews over all FBO attachments, also size of attached region. |
| 422 | std::vector<VkImageView> attachments; |
| 423 | gl::Extents attachmentsSize; |
| 424 | |
| 425 | const auto &colorAttachments = mState.getColorAttachments(); |
| 426 | for (size_t attachmentIndex = 0; attachmentIndex < colorAttachments.size(); ++attachmentIndex) |
| 427 | { |
| 428 | const auto &colorAttachment = colorAttachments[attachmentIndex]; |
| 429 | if (colorAttachment.isAttached()) |
| 430 | { |
| 431 | RenderTargetVk *renderTarget = nullptr; |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 432 | ANGLE_TRY(colorAttachment.getRenderTarget(context, &renderTarget)); |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 433 | attachments.push_back(renderTarget->imageView->getHandle()); |
| 434 | |
| 435 | ASSERT(attachmentsSize.empty() || attachmentsSize == colorAttachment.getSize()); |
| 436 | attachmentsSize = colorAttachment.getSize(); |
| 437 | } |
| 438 | } |
| 439 | |
| 440 | const auto *depthStencilAttachment = mState.getDepthStencilAttachment(); |
| 441 | if (depthStencilAttachment && depthStencilAttachment->isAttached()) |
| 442 | { |
| 443 | RenderTargetVk *renderTarget = nullptr; |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 444 | ANGLE_TRY(depthStencilAttachment->getRenderTarget(context, &renderTarget)); |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 445 | attachments.push_back(renderTarget->imageView->getHandle()); |
| 446 | |
| 447 | ASSERT(attachmentsSize.empty() || attachmentsSize == depthStencilAttachment->getSize()); |
| 448 | attachmentsSize = depthStencilAttachment->getSize(); |
| 449 | } |
| 450 | |
| 451 | ASSERT(!attachments.empty()); |
| 452 | |
| 453 | VkFramebufferCreateInfo framebufferInfo; |
| 454 | |
| 455 | framebufferInfo.sType = VK_STRUCTURE_TYPE_FRAMEBUFFER_CREATE_INFO; |
| 456 | framebufferInfo.pNext = nullptr; |
| 457 | framebufferInfo.flags = 0; |
Jamie Madill | 9f2a861 | 2017-11-30 12:43:09 -0500 | [diff] [blame] | 458 | framebufferInfo.renderPass = renderPass->getHandle(); |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 459 | framebufferInfo.attachmentCount = static_cast<uint32_t>(attachments.size()); |
| 460 | framebufferInfo.pAttachments = attachments.data(); |
| 461 | framebufferInfo.width = static_cast<uint32_t>(attachmentsSize.width); |
| 462 | framebufferInfo.height = static_cast<uint32_t>(attachmentsSize.height); |
| 463 | framebufferInfo.layers = 1; |
| 464 | |
Jamie Madill | 25301b6 | 2017-10-28 20:59:31 -0400 | [diff] [blame] | 465 | ANGLE_TRY(mFramebuffer.init(device, framebufferInfo)); |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 466 | |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 467 | return &mFramebuffer; |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 468 | } |
| 469 | |
JiangYizhou | bddc46b | 2016-12-09 09:50:51 +0800 | [diff] [blame] | 470 | gl::Error FramebufferVk::getSamplePosition(size_t index, GLfloat *xy) const |
| 471 | { |
| 472 | UNIMPLEMENTED(); |
| 473 | return gl::InternalError() << "getSamplePosition is unimplemented."; |
| 474 | } |
| 475 | |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 476 | gl::Error FramebufferVk::getRenderNode(const gl::Context *context, vk::CommandBufferNode **nodeOut) |
Jamie Madill | df68a6f | 2017-01-13 17:29:53 -0500 | [diff] [blame] | 477 | { |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 478 | ContextVk *contextVk = vk::GetImpl(context); |
| 479 | RendererVk *renderer = contextVk->getRenderer(); |
| 480 | Serial currentSerial = renderer->getCurrentQueueSerial(); |
Jamie Madill | bef918c | 2017-12-13 13:11:30 -0500 | [diff] [blame] | 481 | |
Jamie Madill | 47c8ea3 | 2018-01-03 18:27:59 -0500 | [diff] [blame] | 482 | if (isCurrentlyRecording(currentSerial) && mLastRenderNodeSerial == currentSerial) |
Jamie Madill | 4c26fc2 | 2017-02-24 11:04:10 -0500 | [diff] [blame] | 483 | { |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 484 | *nodeOut = getCurrentWriteNode(currentSerial); |
| 485 | ASSERT((*nodeOut)->getInsideRenderPassCommands()->valid()); |
| 486 | return gl::NoError(); |
Jamie Madill | 4c26fc2 | 2017-02-24 11:04:10 -0500 | [diff] [blame] | 487 | } |
| 488 | |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 489 | vk::CommandBufferNode *node = getNewWriteNode(renderer); |
Jamie Madill | 4c26fc2 | 2017-02-24 11:04:10 -0500 | [diff] [blame] | 490 | |
Jamie Madill | df68a6f | 2017-01-13 17:29:53 -0500 | [diff] [blame] | 491 | vk::Framebuffer *framebuffer = nullptr; |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 492 | ANGLE_TRY_RESULT(getFramebuffer(context, renderer), framebuffer); |
Jamie Madill | df68a6f | 2017-01-13 17:29:53 -0500 | [diff] [blame] | 493 | |
Jamie Madill | 1b03824 | 2017-11-01 15:14:36 -0400 | [diff] [blame] | 494 | const gl::State &glState = context->getGLState(); |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 495 | |
| 496 | // Hard-code RenderPass to clear the first render target to the current clear value. |
| 497 | // TODO(jmadill): Proper clear value implementation. |
Jamie Madill | df68a6f | 2017-01-13 17:29:53 -0500 | [diff] [blame] | 498 | VkClearColorValue colorClear; |
| 499 | memset(&colorClear, 0, sizeof(VkClearColorValue)); |
| 500 | colorClear.float32[0] = glState.getColorClearValue().red; |
| 501 | colorClear.float32[1] = glState.getColorClearValue().green; |
| 502 | colorClear.float32[2] = glState.getColorClearValue().blue; |
| 503 | colorClear.float32[3] = glState.getColorClearValue().alpha; |
| 504 | |
| 505 | std::vector<VkClearValue> attachmentClearValues; |
| 506 | attachmentClearValues.push_back({colorClear}); |
| 507 | |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 508 | node->storeRenderPassInfo(*framebuffer, glState.getViewport(), attachmentClearValues); |
Jamie Madill | df68a6f | 2017-01-13 17:29:53 -0500 | [diff] [blame] | 509 | |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 510 | // Initialize RenderPass info. |
| 511 | // TODO(jmadill): Could cache this info, would require dependent state change messaging. |
| 512 | const auto &colorAttachments = mState.getColorAttachments(); |
| 513 | for (size_t attachmentIndex = 0; attachmentIndex < colorAttachments.size(); ++attachmentIndex) |
Jamie Madill | 4c26fc2 | 2017-02-24 11:04:10 -0500 | [diff] [blame] | 514 | { |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 515 | const auto &colorAttachment = colorAttachments[attachmentIndex]; |
| 516 | if (colorAttachment.isAttached()) |
| 517 | { |
| 518 | RenderTargetVk *renderTarget = nullptr; |
| 519 | ANGLE_SWALLOW_ERR(colorAttachment.getRenderTarget(context, &renderTarget)); |
| 520 | |
| 521 | // TODO(jmadill): May need layout transition. |
| 522 | renderTarget->image->updateLayout(VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL); |
| 523 | node->appendColorRenderTarget(currentSerial, renderTarget); |
| 524 | } |
Jamie Madill | 4c26fc2 | 2017-02-24 11:04:10 -0500 | [diff] [blame] | 525 | } |
| 526 | |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 527 | const gl::FramebufferAttachment *depthStencilAttachment = mState.getDepthOrStencilAttachment(); |
| 528 | if (depthStencilAttachment && depthStencilAttachment->isAttached()) |
| 529 | { |
| 530 | RenderTargetVk *renderTarget = nullptr; |
| 531 | ANGLE_SWALLOW_ERR(depthStencilAttachment->getRenderTarget(context, &renderTarget)); |
| 532 | |
| 533 | // TODO(jmadill): May need layout transition. |
| 534 | renderTarget->image->updateLayout(VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL); |
| 535 | node->appendDepthStencilRenderTarget(currentSerial, renderTarget); |
| 536 | } |
| 537 | |
Jamie Madill | 47c8ea3 | 2018-01-03 18:27:59 -0500 | [diff] [blame] | 538 | mLastRenderNodeSerial = currentSerial; |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 539 | |
| 540 | *nodeOut = node; |
Jamie Madill | df68a6f | 2017-01-13 17:29:53 -0500 | [diff] [blame] | 541 | return gl::NoError(); |
| 542 | } |
| 543 | |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 544 | } // namespace rx |