[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