[Bf-blender-cvs] [61e1e9a] master: Cycles: Revert changes to inline flags enabled for release only

Sergey Sharybin noreply at git.blender.org
Sat Nov 15 00:52:18 CET 2014


Commit: 61e1e9adb8a2370321a13773d2bb4b762abfb036
Author: Sergey Sharybin
Date:   Sat Nov 15 04:42:33 2014 +0500
Branches: master
https://developer.blender.org/rB61e1e9adb8a2370321a13773d2bb4b762abfb036

Cycles: Revert changes to inline flags enabled for release only

It appears it's not really needed for convenient debugging when
using proper flags passed to the compiler. Basically, it is -g3
and set breakpoint to a function as if it's not in the namespace.

Not as if a code was any wrong, just it's possible to have more
clear solution for the issue i've tried to solve in the past.

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

M	intern/cycles/util/util_types.h

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

diff --git a/intern/cycles/util/util_types.h b/intern/cycles/util/util_types.h
index 2a199e5..ce84200 100644
--- a/intern/cycles/util/util_types.h
+++ b/intern/cycles/util/util_types.h
@@ -33,11 +33,7 @@
 
 #ifndef __KERNEL_GPU__
 
-#  ifdef NDEBUG
-#    define ccl_device static inline
-#  else
-#    define ccl_device static
-#  endif
+#define ccl_device static inline
 #define ccl_device_noinline static
 #define ccl_global
 #define ccl_constant
@@ -53,11 +49,7 @@
 #define ccl_try_align(...) /* not support for function arguments (error C2719) */
 #endif
 #define ccl_may_alias
-#  ifdef NDEBUG
-#    define ccl_always_inline __forceinline
-#  else
-#    define ccl_always_inline
-#  endif
+#define ccl_always_inline __forceinline
 #define ccl_maybe_unused
 
 #else




More information about the Bf-blender-cvs mailing list