================ @@ -669,15 +670,8 @@ bool ClangUserExpression::Parse(DiagnosticManager &diagnostic_manager, // Parse the expression // - Process *process = exe_ctx.GetProcessPtr(); - ExecutionContextScope *exe_scope = process; - - if (!exe_scope) - exe_scope = exe_ctx.GetTargetPtr(); - - bool parse_success = TryParse(diagnostic_manager, exe_scope, exe_ctx, - execution_policy, keep_result_in_memory, - generate_debug_info); + bool parse_success = TryParse(diagnostic_manager, exe_ctx, execution_policy, + keep_result_in_memory, generate_debug_info); ---------------- Michael137 wrote:
Removing `exe_ctx` in favour of `exe_scope` is trickier because we store away the `exe_ctx` for use in `ClangExpressionDeclMap`. I'd prefer sticking with this approach unless people are strongly opposed to it https://github.com/llvm/llvm-project/pull/87657 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits