[Bf-blender-cvs] [b30fbac] cycles_kernel_split: Refactor : Remove un-necessary code/comment

varunsundar08 noreply at git.blender.org
Thu Apr 30 23:25:11 CEST 2015


Commit: b30fbac8de7f08e94295cc0a94ade01b149fa63d
Author: varunsundar08
Date:   Tue Apr 28 16:49:44 2015 +0530
Branches: cycles_kernel_split
https://developer.blender.org/rBb30fbac8de7f08e94295cc0a94ade01b149fa63d

Refactor : Remove un-necessary code/comment

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

M	intern/cycles/device/device_opencl.cpp

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

diff --git a/intern/cycles/device/device_opencl.cpp b/intern/cycles/device/device_opencl.cpp
index c86ef66..197d631 100644
--- a/intern/cycles/device/device_opencl.cpp
+++ b/intern/cycles/device/device_opencl.cpp
@@ -25,8 +25,6 @@
 #include "device.h"
 #include "device_intern.h"
 
-#define __SPLIT_KERNEL__
-
 #include "buffers.h"
 
 #include "util_foreach.h"
@@ -42,8 +40,7 @@ CCL_NAMESPACE_BEGIN
 
 #define CL_MEM_PTR(p) ((cl_mem)(uintptr_t)(p))
 
-#ifdef __SPLIT_KERNEL__
-
+/* Macro declarations used with split kernel */
 #define SPLIT_KERNEL_LOCAL_SIZE_X 64
 #define SPLIT_KERNEL_LOCAL_SIZE_Y 1
 
@@ -71,7 +68,6 @@ CCL_NAMESPACE_BEGIN
 #define SD_NUM_DIFFERENTIAL 2
 #endif
 #define SD_NUM_RAY_DP_DIFFERENTIAL3 1
-#endif
 
 static cl_device_type opencl_device_type()
 {
@@ -4186,7 +4182,6 @@ Device *device_opencl_create(DeviceInfo& info, Stats &stats, bool background)
 	if (get_platform_and_devicetype(info, platform_name, device_type)) {
 		if (platform_name == "AMD Accelerated Parallel Processing" && device_type == CL_DEVICE_TYPE_GPU) {
 			/* If the device is an AMD GPU, take split kernel path */
-			/* TO DO : Create a separate class for split kernel and return a pointer to that class */
 			return new OpenCLDeviceSplitKernel(info, stats, background);
 		} else {
 			/* For any other device, take megakernel path */




More information about the Bf-blender-cvs mailing list