[Bf-blender-cvs] [1129b5dc7fe] cycles-x: Cycles X: Remove zero_to_device from RenderBuffers::reset()
Sergey Sharybin
noreply at git.blender.org
Tue Jun 29 18:13:11 CEST 2021
Commit: 1129b5dc7fe2191c7608d9d5dad55e8415790387
Author: Sergey Sharybin
Date: Tue Jun 29 17:13:34 2021 +0200
Branches: cycles-x
https://developer.blender.org/rB1129b5dc7fe2191c7608d9d5dad55e8415790387
Cycles X: Remove zero_to_device from RenderBuffers::reset()
In all the current usages of RenderBuffers::reset the zero was
actually redundant.
This change makes it easier to allocate temporary render buffer
and copy data to it as a part of a specific stream without zero
(which happens in the default stream) conflicting.
Differential Revision: https://developer.blender.org/D11740
===================================================================
M intern/cycles/integrator/denoiser_device.cpp
M intern/cycles/render/buffers.cpp
===================================================================
diff --git a/intern/cycles/integrator/denoiser_device.cpp b/intern/cycles/integrator/denoiser_device.cpp
index e13cf467204..8a0bff8dc21 100644
--- a/intern/cycles/integrator/denoiser_device.cpp
+++ b/intern/cycles/integrator/denoiser_device.cpp
@@ -195,7 +195,6 @@ void DeviceDenoiser::denoise_buffer_on_device(Device *device,
render_buffers->copy_from_device();
- /* TODO(sergey): Avoid `zero_to_device()`. */
local_render_buffers.reset(buffer_params);
/* NOTE: The local buffer is allocated for an exact size of the effective render size, while
diff --git a/intern/cycles/render/buffers.cpp b/intern/cycles/render/buffers.cpp
index 775190fa6c0..5ce31e4e308 100644
--- a/intern/cycles/render/buffers.cpp
+++ b/intern/cycles/render/buffers.cpp
@@ -159,7 +159,6 @@ void RenderBuffers::reset(const BufferParams ¶ms_)
/* re-allocate buffer */
buffer.alloc(params.width * params.pass_stride, params.height);
- buffer.zero_to_device();
}
void RenderBuffers::zero()
More information about the Bf-blender-cvs
mailing list