[Bf-blender-cvs] [9325b9bf153] master: Fix T53365: OpenCL has wrong shading of brick texture

Mai Lavelle noreply at git.blender.org
Tue Nov 21 06:43:26 CET 2017


Commit: 9325b9bf1539bcc440be03c3c360e0b7dd9dc3e7
Author: Mai Lavelle
Date:   Tue Nov 21 00:38:02 2017 -0500
Branches: master
https://developer.blender.org/rB9325b9bf1539bcc440be03c3c360e0b7dd9dc3e7

Fix T53365: OpenCL has wrong shading of brick texture

Looks like some weird compiler difference with signed vs unsigned ints.

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

M	intern/cycles/kernel/shaders/node_brick_texture.osl
M	intern/cycles/kernel/svm/svm_brick.h

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

diff --git a/intern/cycles/kernel/shaders/node_brick_texture.osl b/intern/cycles/kernel/shaders/node_brick_texture.osl
index c303594681c..eaa5bfc776f 100644
--- a/intern/cycles/kernel/shaders/node_brick_texture.osl
+++ b/intern/cycles/kernel/shaders/node_brick_texture.osl
@@ -19,9 +19,9 @@
 
 /* Brick */
 
-float brick_noise(int n) /* fast integer noise */
+float brick_noise(uint n) /* fast integer noise */
 {
-	int nn;
+	uint nn;
 	n = (n + 1013) & 2147483647;
 	n = (n >> 13) ^ n;
 	nn = (n * (n * n * 60493 + 19990303) + 1376312589) & 2147483647;
diff --git a/intern/cycles/kernel/svm/svm_brick.h b/intern/cycles/kernel/svm/svm_brick.h
index 14245cf0522..90fa2a99b67 100644
--- a/intern/cycles/kernel/svm/svm_brick.h
+++ b/intern/cycles/kernel/svm/svm_brick.h
@@ -18,9 +18,9 @@ CCL_NAMESPACE_BEGIN
 
 /* Brick */
 
-ccl_device_noinline float brick_noise(int n) /* fast integer noise */
+ccl_device_noinline float brick_noise(uint n) /* fast integer noise */
 {
-	int nn;
+	uint nn;
 	n = (n + 1013) & 0x7fffffff;
 	n = (n >> 13) ^ n;
 	nn = (n * (n * n * 60493 + 19990303) + 1376312589) & 0x7fffffff;



More information about the Bf-blender-cvs mailing list