diff options
author | Sergey Sharybin <sergey.vfx@gmail.com> | 2017-05-19 13:33:28 +0300 |
---|---|---|
committer | Sergey Sharybin <sergey.vfx@gmail.com> | 2017-05-19 13:41:03 +0300 |
commit | 803337f3f64fed240e9adc6f286d5f9d13a5026a (patch) | |
tree | a262b427bd53873be9d18e952f193d66801edac8 /intern/cycles/kernel/kernel_compat_opencl.h | |
parent | 8e655446d1ec667a08a6d351d1e452fc51f1428a (diff) |
\0;115;0cCycles: Cleanup, use ccl_restrict instead of ccl_restrict_ptr
There were following issues with ccl_restrict_ptr:
- We already had ccl_restrict for all platforms.
- It was secretly adding `const` qualifier to the declaration,
which is quite weird since non-const pointer can also be
declared as restricted.
- We never in Blender are using foo_ptr or FooPtr type definitions,
so not sure why we should introduce such a thing here.
- It is absolutely wrong from semantic point of view to put pointer
into the restrict macro -- const is a part of type, not part of
hint for compiler that some pointer is never aliased.
Diffstat (limited to 'intern/cycles/kernel/kernel_compat_opencl.h')
-rw-r--r-- | intern/cycles/kernel/kernel_compat_opencl.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/intern/cycles/kernel/kernel_compat_opencl.h b/intern/cycles/kernel/kernel_compat_opencl.h index 15cf4b81b21..c2263ac0d49 100644 --- a/intern/cycles/kernel/kernel_compat_opencl.h +++ b/intern/cycles/kernel/kernel_compat_opencl.h @@ -50,8 +50,6 @@ # define ccl_addr_space #endif -#define ccl_restrict_ptr const * __restrict__ - #define ccl_local_id(d) get_local_id(d) #define ccl_global_id(d) get_global_id(d) |