Cycles: enable improved closure sampling, this should give less noise for mix, add

and glass shaders. How well this will work on non-fermi GPU's is unclear still, it's
a bit heavy on register usage.

diff --git a/intern/cycles/device/device.cpp b/intern/cycles/device/device.cpp
index f43ccffe4614afb420fbda639b99c090d530f269..01c5081770943fbed0290d3397700ffbb0420a1e 100644 (file)
--- a/intern/cycles/device/device.cpp
+++ b/intern/cycles/device/device.cpp
@@ -157,6 +157,8 @@ Device *Device::create(DeviceType type, bool background, int threads)
                        return NULL;
        }
 
                        return NULL;
        }
 

+       device->device_type = type;
+

        return device;
 }
 
        return device;
 }
 

diff --git a/intern/cycles/device/device.h b/intern/cycles/device/device.h
index 0b3c66dbb2c13f5ef9e2d6a851aef45b62f2cccc..360478cfd75c5f3785b4121a3c87de8b2ad24014 100644 (file)
--- a/intern/cycles/device/device.h
+++ b/intern/cycles/device/device.h
@@ -75,12 +75,14 @@ class Device {
 protected:
        Device() {}
 
 protected:
        Device() {}
 

-       DeviceType type;
+       DeviceType device_type;

        bool background;
 
 public:
        virtual ~Device() {}
 
        bool background;
 
 public:
        virtual ~Device() {}
 

+       DeviceType type() { return device_type; }
+

        /* info */
        virtual string description() = 0;
 
        /* info */
        virtual string description() = 0;
 

diff --git a/intern/cycles/kernel/kernel_types.h b/intern/cycles/kernel/kernel_types.h
index bda037e88cc6a2401e73b1843e4d9c86bd5dfaa2..c789601cd276bffe0c9bec5f37a615ee635932d6 100644 (file)
--- a/intern/cycles/kernel/kernel_types.h
+++ b/intern/cycles/kernel/kernel_types.h
@@ -44,7 +44,7 @@ CCL_NAMESPACE_BEGIN
 #define __EMISSION__
 #define __TEXTURES__
 #define __HOLDOUT__
 #define __EMISSION__
 #define __TEXTURES__
 #define __HOLDOUT__

-//#define __MULTI_CLOSURE__
+#define __MULTI_CLOSURE__

 //#define __TRANSPARENT_SHADOWS__
 //#define __MULTI_LIGHT__
 #endif
 //#define __TRANSPARENT_SHADOWS__
 //#define __MULTI_LIGHT__
 #endif

diff --git a/intern/cycles/render/scene.h b/intern/cycles/render/scene.h
index a969dd66c4c2fe21c392d24749556b489b571368..c191f8a56e5f58046c7ee4f5b07e9788abd16b64 100644 (file)
--- a/intern/cycles/render/scene.h
+++ b/intern/cycles/render/scene.h
@@ -109,7 +109,7 @@ public:
        SceneParams()
        {
                shadingsystem = SVM;
        SceneParams()
        {
                shadingsystem = SVM;

-               use_multi_closure = false;
+               use_multi_closure = true;

                bvh_type = BVH_DYNAMIC;
                use_bvh_cache = false;
                use_bvh_spatial_split = false;
                bvh_type = BVH_DYNAMIC;
                use_bvh_cache = false;
                use_bvh_spatial_split = false;

diff --git a/intern/cycles/render/svm.cpp b/intern/cycles/render/svm.cpp
index a1687ae5e290b809f5e975a1ad7c5550b96b5710..b9c4219f85201181ad9d0214780866ef43cd3381 100644 (file)
--- a/intern/cycles/render/svm.cpp
+++ b/intern/cycles/render/svm.cpp
@@ -58,6 +58,7 @@ void SVMShaderManager::device_update(Device *device, DeviceScene *dscene, Scene
        }
        
        bool sunsky_done = false;
        }
        
        bool sunsky_done = false;

+       bool use_multi_closure = (scene->params.use_multi_closure && device->type() != DEVICE_OPENCL);

 
        for(i = 0; i < scene->shaders.size(); i++) {
                Shader *shader = scene->shaders[i];
 
        for(i = 0; i < scene->shaders.size(); i++) {
                Shader *shader = scene->shaders[i];


@@ -70,7 +71,7 @@ void SVMShaderManager::device_update(Device *device, DeviceScene *dscene, Scene
                        scene->light_manager->need_update = true;
 
                SVMCompiler compiler(scene->shader_manager, scene->image_manager,
                        scene->light_manager->need_update = true;
 
                SVMCompiler compiler(scene->shader_manager, scene->image_manager,

-                       scene->params.use_multi_closure);
+                       use_multi_closure);

                compiler.sunsky = (sunsky_done)? NULL: &dscene->data.sunsky;
                compiler.background = ((int)i == scene->default_background);
                compiler.compile(shader, svm_nodes, i);
                compiler.sunsky = (sunsky_done)? NULL: &dscene->data.sunsky;
                compiler.background = ((int)i == scene->default_background);
                compiler.compile(shader, svm_nodes, i);