From 47cca78efa07fcf94507e1b84b098f984493cadc Mon Sep 17 00:00:00 2001 From: Sean Callanan Date: Sat, 26 Mar 2016 00:30:40 +0000 Subject: [PATCH] Record all translation units with more than one function in them (e.g., blocks). Blocks and lambdas have their implementation functions stored in the IR for an expression. If we put the block/lambda into a result variable it needs to stay around. As a heuristic, remember any execution unit that has more than one function in it. llvm-svn: 264483 --- .../Plugins/ExpressionParser/Clang/ClangUserExpression.cpp | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/lldb/source/Plugins/ExpressionParser/Clang/ClangUserExpression.cpp b/lldb/source/Plugins/ExpressionParser/Clang/ClangUserExpression.cpp index 9bf8cd5..6da103e 100644 --- a/lldb/source/Plugins/ExpressionParser/Clang/ClangUserExpression.cpp +++ b/lldb/source/Plugins/ExpressionParser/Clang/ClangUserExpression.cpp @@ -577,12 +577,18 @@ ClangUserExpression::Parse(DiagnosticManager &diagnostic_manager, ExecutionConte { register_execution_unit = true; } + + // If there is more than one external function in the execution + // unit, it needs to keep living even if it's not top level, because + // the result could refer to that function. + + if (m_execution_unit_sp->GetJittedFunctions().size() > 1) + { + register_execution_unit = true; + } if (register_execution_unit) { - // We currently key off there being more than one external function in the execution - // unit to determine whether it needs to live in the process. - llvm::cast( exe_ctx.GetTargetPtr()->GetPersistentExpressionStateForLanguage(m_language)) ->RegisterExecutionUnit(m_execution_unit_sp); -- 2.7.4