[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 &params_)
 
   /* 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