[Bf-blender-cvs] [9ddc556] opensubdiv-modifier: Code cleanup: use Blender code style
Sergey Sharybin
noreply at git.blender.org
Wed Jul 2 19:46:53 CEST 2014
Commit: 9ddc556574ad8dd59882d734c2a1071691ab48a1
Author: Sergey Sharybin
Date: Wed Jul 2 22:11:35 2014 +0600
https://developer.blender.org/rB9ddc556574ad8dd59882d734c2a1071691ab48a1
Code cleanup: use Blender code style
===================================================================
M intern/opensubdiv/clInit.h
M intern/opensubdiv/cudaInit.h
===================================================================
diff --git a/intern/opensubdiv/clInit.h b/intern/opensubdiv/clInit.h
index 2fc2f9f..2a4e131 100644
--- a/intern/opensubdiv/clInit.h
+++ b/intern/opensubdiv/clInit.h
@@ -26,31 +26,31 @@
#define OSD_EXAMPLE_CL_INIT_H
#if defined(_WIN32)
- #include <windows.h>
+# include <windows.h>
#elif defined(__APPLE__)
- #include <OpenGL/OpenGL.h>
+# include <OpenGL/OpenGL.h>
#else
- #include <GL/glx.h>
+# include <GL/glx.h>
#endif
#include <opensubdiv/osd/opencl.h>
#include <cstdio>
-static bool HAS_CL_VERSION_1_1 () {
+static bool HAS_CL_VERSION_1_1() {
#ifdef OPENSUBDIV_HAS_OPENCL
- #ifdef OPENSUBDIV_HAS_CLEW
- static bool clewInitialized = false;
- static bool clewLoadSuccess;
- if (not clewInitialized) {
- clewInitialized = true;
- clewLoadSuccess = clewInit() == CLEW_SUCCESS;
- if (not clewLoadSuccess) {
- fprintf(stderr, "Loading OpenCL failed.\n");
- }
- }
- return clewLoadSuccess;
- #endif
+# ifdef OPENSUBDIV_HAS_CLEW
+ static bool clewInitialized = false;
+ static bool clewLoadSuccess;
+ if (!clewInitialized) {
+ clewInitialized = true;
+ clewLoadSuccess = clewInit() == CLEW_SUCCESS;
+ if (!clewLoadSuccess) {
+ fprintf(stderr, "Loading OpenCL failed.\n");
+ }
+ }
+ return clewLoadSuccess;
+# endif
return true;
#else
return false;
@@ -59,104 +59,128 @@ static bool HAS_CL_VERSION_1_1 () {
static bool initCL(cl_context *clContext, cl_command_queue *clQueue)
{
- cl_int ciErrNum;
-
- cl_platform_id cpPlatform = 0;
- cl_uint num_platforms;
- ciErrNum = clGetPlatformIDs(0, NULL, &num_platforms);
- if (ciErrNum != CL_SUCCESS) {
- printf("Error %d in clGetPlatformIDs call.\n", ciErrNum);
- return false;
- }
- if (num_platforms == 0) {
- printf("No OpenCL platform found.\n");
- return false;
- }
- cl_platform_id *clPlatformIDs = new cl_platform_id[num_platforms];
- ciErrNum = clGetPlatformIDs(num_platforms, clPlatformIDs, NULL);
- char chBuffer[1024];
- for (cl_uint i = 0; i < num_platforms; ++i) {
- ciErrNum = clGetPlatformInfo(clPlatformIDs[i], CL_PLATFORM_NAME, 1024, chBuffer,NULL);
- if (ciErrNum == CL_SUCCESS) {
- cpPlatform = clPlatformIDs[i];
- }
- }
+ cl_int ciErrNum;
+ cl_platform_id cpPlatform = 0;
+ cl_uint num_platforms;
+ ciErrNum = clGetPlatformIDs(0, NULL, &num_platforms);
+ if (ciErrNum != CL_SUCCESS) {
+ printf("Error %d in clGetPlatformIDs call.\n", ciErrNum);
+ return false;
+ }
+ if (num_platforms == 0) {
+ printf("No OpenCL platform found.\n");
+ return false;
+ }
+ cl_platform_id *clPlatformIDs = new cl_platform_id[num_platforms];
+ ciErrNum = clGetPlatformIDs(num_platforms, clPlatformIDs, NULL);
+ char chBuffer[1024];
+ for (cl_uint i = 0; i < num_platforms; ++i) {
+ ciErrNum = clGetPlatformInfo(clPlatformIDs[i],
+ CL_PLATFORM_NAME,
+ 1024,
+ chBuffer,
+ NULL);
+ if (ciErrNum == CL_SUCCESS) {
+ cpPlatform = clPlatformIDs[i];
+ }
+ }
#if defined(_WIN32)
- cl_context_properties props[] = {
- CL_GL_CONTEXT_KHR, (cl_context_properties)wglGetCurrentContext(),
- CL_WGL_HDC_KHR, (cl_context_properties)wglGetCurrentDC(),
- CL_CONTEXT_PLATFORM, (cl_context_properties)cpPlatform,
- 0
- };
+ cl_context_properties props[] = {
+ CL_GL_CONTEXT_KHR, (cl_context_properties)wglGetCurrentContext(),
+ CL_WGL_HDC_KHR, (cl_context_properties)wglGetCurrentDC(),
+ CL_CONTEXT_PLATFORM, (cl_context_properties)cpPlatform,
+ 0
+ };
#elif defined(__APPLE__)
- CGLContextObj kCGLContext = CGLGetCurrentContext();
- CGLShareGroupObj kCGLShareGroup = CGLGetShareGroup(kCGLContext);
- cl_context_properties props[] = {
- CL_CONTEXT_PROPERTY_USE_CGL_SHAREGROUP_APPLE, (cl_context_properties)kCGLShareGroup,
- 0
- };
+ CGLContextObj kCGLContext = CGLGetCurrentContext();
+ CGLShareGroupObj kCGLShareGroup = CGLGetShareGroup(kCGLContext);
+ cl_context_properties props[] = {
+ CL_CONTEXT_PROPERTY_USE_CGL_SHAREGROUP_APPLE,
+ (cl_context_properties)kCGLShareGroup,
+ 0
+ };
#else
- cl_context_properties props[] = {
- CL_GL_CONTEXT_KHR, (cl_context_properties)glXGetCurrentContext(),
- CL_GLX_DISPLAY_KHR, (cl_context_properties)glXGetCurrentDisplay(),
- CL_CONTEXT_PLATFORM, (cl_context_properties)cpPlatform,
- 0
- };
+ cl_context_properties props[] = {
+ CL_GL_CONTEXT_KHR, (cl_context_properties)glXGetCurrentContext(),
+ CL_GLX_DISPLAY_KHR, (cl_context_properties)glXGetCurrentDisplay(),
+ CL_CONTEXT_PLATFORM, (cl_context_properties)cpPlatform,
+ 0
+ };
#endif
- delete[] clPlatformIDs;
+ delete [] clPlatformIDs;
#if defined(__APPLE__)
- *clContext = clCreateContext(props, 0, NULL, clLogMessagesToStdoutAPPLE, NULL, &ciErrNum);
- if (ciErrNum != CL_SUCCESS) {
- printf("Error %d in clCreateContext\n", ciErrNum);
- return false;
- }
+ *clContext = clCreateContext(props,
+ 0,
+ NULL,
+ clLogMessagesToStdoutAPPLE,
+ NULL,
+ &ciErrNum);
+ if (ciErrNum != CL_SUCCESS) {
+ printf("Error %d in clCreateContext\n", ciErrNum);
+ return false;
+ }
- size_t devicesSize = 0;
- clGetGLContextInfoAPPLE(*clContext, kCGLContext, CL_CGL_DEVICES_FOR_SUPPORTED_VIRTUAL_SCREENS_APPLE, 0, NULL, &devicesSize);
- int numDevices = int(devicesSize / sizeof(cl_device_id));
- if (numDevices == 0) {
- printf("No sharable devices.\n");
- return false;
- }
- cl_device_id *clDevices = new cl_device_id[numDevices];
- clGetGLContextInfoAPPLE(*clContext, kCGLContext, CL_CGL_DEVICES_FOR_SUPPORTED_VIRTUAL_SCREENS_APPLE, numDevices * sizeof(cl_device_id), clDevices, NULL);
+ size_t devicesSize = 0;
+ clGetGLContextInfoAPPLE(*clContext,
+ kCGLContext,
+ CL_CGL_DEVICES_FOR_SUPPORTED_VIRTUAL_SCREENS_APPLE,
+ 0,
+ NULL,
+ &devicesSize);
+ int numDevices = int(devicesSize / sizeof(cl_device_id));
+ if (numDevices == 0) {
+ printf("No sharable devices.\n");
+ return false;
+ }
+ cl_device_id *clDevices = new cl_device_id[numDevices];
+ clGetGLContextInfoAPPLE(*clContext,
+ kCGLContext,
+ CL_CGL_DEVICES_FOR_SUPPORTED_VIRTUAL_SCREENS_APPLE,
+ numDevices * sizeof(cl_device_id),
+ clDevices, NULL);
#else
- cl_uint numDevices = 0;
- clGetDeviceIDs(cpPlatform, CL_DEVICE_TYPE_GPU, 0, NULL, &numDevices);
- if (numDevices == 0) {
- printf("No sharable devices.\n");
- return false;
- }
- cl_device_id *clDevices = new cl_device_id[numDevices];
- clGetDeviceIDs(cpPlatform, CL_DEVICE_TYPE_GPU, numDevices, clDevices, NULL);
+ cl_uint numDevices = 0;
+ clGetDeviceIDs(cpPlatform, CL_DEVICE_TYPE_GPU, 0, NULL, &numDevices);
+ if (numDevices == 0) {
+ printf("No sharable devices.\n");
+ return false;
+ }
+ cl_device_id *clDevices = new cl_device_id[numDevices];
+ clGetDeviceIDs(cpPlatform, CL_DEVICE_TYPE_GPU, numDevices, clDevices, NULL);
- *clContext = clCreateContext(props, numDevices, clDevices, NULL, NULL, &ciErrNum);
- if (ciErrNum != CL_SUCCESS) {
- printf("Error %d in clCreateContext\n", ciErrNum);
- delete[] clDevices;
- return false;
- }
+ *clContext = clCreateContext(props,
+ numDevices,
+ clDevices,
+ NULL,
+ NULL,
+ &ciErrNum);
+ if (ciErrNum != CL_SUCCESS) {
+ printf("Error %d in clCreateContext\n", ciErrNum);
+ delete [] clDevices;
+ return false;
+ }
#endif
- *clQueue = clCreateCommandQueue(*clContext, clDevices[0], 0, &ciErrNum);
- delete[] clDevices;
- if (ciErrNum != CL_SUCCESS) {
- printf("Error %d in clCreateCommandQueue\n", ciErrNum);
- return false;
- }
- return true;
+ *clQueue = clCreateCommandQueue(*clContext, clDevices[0], 0, &ciErrNum);
+ delete [] clDevices;
+ if (ciErrNum != CL_SUCCESS) {
+ printf("Error %d in clCreateCommandQueue\n", ciErrNum);
+ return false;
+ }
+ return true;
}
static void uninitCL(cl_context clContext, cl_command_queue clQueue)
{
- // It's possible that OpenCL wasn't used and hence wasn't initialized yet,
- // no need to cleanup in this case.
- if (clReleaseCommandQueue) {
- clReleaseCommandQueue(clQueue);
- clReleaseContext(clContext);
- }
+ /* It's possible that OpenCL wasn't used and hence wasn't initialized yet,
+ * no need to cleanup in this case.
+ */
+ if (clReleaseCommandQueue) {
+ clReleaseCommandQueue(clQueue);
+ clReleaseContext(clContext);
+ }
}
-#endif // OSD_EXAMPLE_CL_INIT_H
+#endif /* OSD_EXAMPLE_CL_INIT_H */
diff --git a/intern/opensubdiv/cudaInit.h b/intern/opensubdiv/cudaInit.h
index 9ae6a14..a5a77d4 100644
--- a/intern/opensubdiv/cudaInit.h
+++ b/intern/opensubdiv/cudaInit.h
@@ -29,125 +29,131 @@
#include <cstdio>
#include <opensubdiv/osd/cuda.h>
-// From "NVIDIA GPU Computing SDK 4.2/C/common/inc/cutil_inline_runtime.h":
+/* From "NVIDIA GPU Computing SDK 4.2/C/common/inc/cutil_inline_runtime.h": */
-// Beginning of GPU Architecture definitions
+/* Beginning of GPU Architecture definitions */
inline int _ConvertSMVer2Cores_local(int major, int minor)
{
- // Defines for GPU Architecture types (using the SM version to determine the # of cores per SM
- typedef struct {
- int SM; // 0xMm (hexidecimal notation), M = SM Major version, and m = SM minor version
- int Cores;
- } sSMtoCores;
+ /* Defines for GPU Architecture types (using the SM version to determine
+ * the # of cores per SM
+ */
+ typedef struct {
+ int SM; /* 0xMm (hexidecimal notation),
+ * M = SM Major version,
+ * and m = SM minor version
+ */
+ int Cores;
+ } sSMtoCores
@@ Diff output truncated at 10240 characters. @@
More information about the Bf-blender-cvs
mailing list