From 1be16466e7983edbe8794127141267a8099d54be Mon Sep 17 00:00:00 2001
From: Brecht Van Lommel <brechtvanlommel@gmail.com>
Date: Wed, 13 Mar 2019 18:26:11 +0100
Subject: Fix T60434: crash with OSL and viewport + preview render at the same
 time.

Don't free LLVM JIT memory until process exit, there might be multiple OSL
instances using it.
---
 intern/cycles/render/osl.cpp    | 4 ++++
 intern/cycles/render/osl.h      | 2 ++
 intern/cycles/render/shader.cpp | 4 ++++
 3 files changed, 10 insertions(+)

(limited to 'intern')

diff --git a/intern/cycles/render/osl.cpp b/intern/cycles/render/osl.cpp
index 3fbc7d33a74..c603dc69a16 100644
--- a/intern/cycles/render/osl.cpp
+++ b/intern/cycles/render/osl.cpp
@@ -66,6 +66,10 @@ OSLShaderManager::~OSLShaderManager()
 {
 	shading_system_free();
 	texture_system_free();
+}
+
+void OSLShaderManager::free_memory()
+{
 #ifdef OSL_HAS_BLENDER_CLEANUP_FIX
 	/* There is a problem with llvm+osl: The order global destructors across
 	 * different compilation units run cannot be guaranteed, on windows this means
diff --git a/intern/cycles/render/osl.h b/intern/cycles/render/osl.h
index e196e0be787..93cc3139608 100644
--- a/intern/cycles/render/osl.h
+++ b/intern/cycles/render/osl.h
@@ -70,6 +70,8 @@ public:
 	OSLShaderManager();
 	~OSLShaderManager();
 
+	static void free_memory();
+
 	void reset(Scene *scene);
 
 	bool use_osl() { return true; }
diff --git a/intern/cycles/render/shader.cpp b/intern/cycles/render/shader.cpp
index 3ee63e91e8d..9f256a665cd 100644
--- a/intern/cycles/render/shader.cpp
+++ b/intern/cycles/render/shader.cpp
@@ -692,6 +692,10 @@ void ShaderManager::get_requested_features(Scene *scene,
 void ShaderManager::free_memory()
 {
 	beckmann_table.free_memory();
+
+#ifdef WITH_OSL
+	OSLShaderManager::free_memory();
+#endif
 }
 
 float ShaderManager::linear_rgb_to_gray(float3 c)
-- 
cgit v1.2.3