[Bf-blender-cvs] [1f5f3b55884] master: Cycles: Make compatible with C++17 on 10.9 deployment target

Sergey Sharybin noreply at git.blender.org
Fri Jun 19 12:03:02 CEST 2020


Commit: 1f5f3b558848e84abcd5703353fc1a3328099aed
Author: Sergey Sharybin
Date:   Thu Jun 18 16:17:20 2020 +0200
Branches: master
https://developer.blender.org/rB1f5f3b558848e84abcd5703353fc1a3328099aed

Cycles: Make compatible with C++17 on 10.9 deployment target

OSLRenderServices contains a member which is 64 bytes aligned and this
is onlys supported by default allocators in SDK 10.14. Since we take
care of allocation ourselves ignoring the diagnostic is an acceptable
way dealign with it.

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

M	intern/cycles/kernel/osl/CMakeLists.txt

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

diff --git a/intern/cycles/kernel/osl/CMakeLists.txt b/intern/cycles/kernel/osl/CMakeLists.txt
index fc0c845fd4f..d7ab778181e 100644
--- a/intern/cycles/kernel/osl/CMakeLists.txt
+++ b/intern/cycles/kernel/osl/CMakeLists.txt
@@ -36,6 +36,15 @@ set(LIB
 # OSL and LLVM are built without RTTI
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${RTTI_DISABLE_FLAGS}")
 
+if(APPLE)
+  # Disable allocation warning on macOS prior to 10.14: the OSLRenderServices
+  # contains member which is 64 bytes aligned (cache inside of OIIO's
+  # unordered_map_concurrent). This is not something what the SDK supportsm, but
+  # since we take care of allocations ourselves is is OK to ignore the
+  # diagnostic message.
+  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -faligned-allocation")
+endif()
+
 include_directories(${INC})
 include_directories(SYSTEM ${INC_SYS})



More information about the Bf-blender-cvs mailing list