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