[Bf-blender-cvs] [8b9b87aee8e] blender-v2.93-release: Cleanup: removed unnecessary multiplications in area light importance sampling

Matteo Falduto noreply at git.blender.org
Thu Apr 29 15:59:34 CEST 2021


Commit: 8b9b87aee8e5e9a68b3c211049377d77d44126c0
Author: Matteo Falduto
Date:   Thu Apr 29 15:10:16 2021 +0200
Branches: blender-v2.93-release
https://developer.blender.org/rB8b9b87aee8e5e9a68b3c211049377d77d44126c0

Cleanup: removed unnecessary multiplications in area light importance sampling

Differential Revision: https://developer.blender.org/D11114

===================================================================

M	intern/cycles/kernel/kernel_light_common.h

===================================================================

diff --git a/intern/cycles/kernel/kernel_light_common.h b/intern/cycles/kernel/kernel_light_common.h
index c21c517a353..4a683d36226 100644
--- a/intern/cycles/kernel/kernel_light_common.h
+++ b/intern/cycles/kernel/kernel_light_common.h
@@ -200,12 +200,12 @@ ccl_device bool light_spread_clamp_area_light(const float3 P,
    * uv coordinates. */
   const float new_center_u = 0.5f * (min_u + max_u);
   const float new_center_v = 0.5f * (min_v + max_v);
-  const float new_len_u = 0.5f * (max_u - min_u);
-  const float new_len_v = 0.5f * (max_v - min_v);
+  const float new_len_u = max_u - min_u;
+  const float new_len_v = max_v - min_v;
 
   *lightP = *lightP + new_center_u * u + new_center_v * v;
-  *axisu = u * new_len_u * 2.0f;
-  *axisv = v * new_len_v * 2.0f;
+  *axisu = u * new_len_u;
+  *axisv = v * new_len_v;
 
   return true;
 }



More information about the Bf-blender-cvs mailing list