Author: Felipe de Azevedo Piovezan Date: 2025-08-04T09:43:21-07:00 New Revision: 80bd72b9b0d5d1f8fd54a4ee6f8af80734912d5d
URL: https://github.com/llvm/llvm-project/commit/80bd72b9b0d5d1f8fd54a4ee6f8af80734912d5d DIFF: https://github.com/llvm/llvm-project/commit/80bd72b9b0d5d1f8fd54a4ee6f8af80734912d5d.diff LOG: [lldb][nfc] Use delegating ctor for ExecutionContext (#151987) The ctor that takes a reference to ExecutionContextRef can be implemented in terms of the ctor that takes a pointer to that object, removing code duplication. Added: Modified: lldb/include/lldb/Target/ExecutionContext.h lldb/source/Target/ExecutionContext.cpp Removed: ################################################################################ diff --git a/lldb/include/lldb/Target/ExecutionContext.h b/lldb/include/lldb/Target/ExecutionContext.h index d8c233a666bb4..aebd0d5308e72 100644 --- a/lldb/include/lldb/Target/ExecutionContext.h +++ b/lldb/include/lldb/Target/ExecutionContext.h @@ -318,7 +318,9 @@ class ExecutionContext { // These two variants take in a locker, and grab the target, lock the API // mutex into locker, then fill in the rest of the shared pointers. ExecutionContext(const ExecutionContextRef &exe_ctx_ref, - std::unique_lock<std::recursive_mutex> &locker); + std::unique_lock<std::recursive_mutex> &locker) + : ExecutionContext(&exe_ctx_ref, locker) {} + ExecutionContext(const ExecutionContextRef *exe_ctx_ref, std::unique_lock<std::recursive_mutex> &locker); // Create execution contexts from execution context scopes diff --git a/lldb/source/Target/ExecutionContext.cpp b/lldb/source/Target/ExecutionContext.cpp index b1563d9ceb71b..3f54ea55c9c81 100644 --- a/lldb/source/Target/ExecutionContext.cpp +++ b/lldb/source/Target/ExecutionContext.cpp @@ -140,19 +140,6 @@ ExecutionContext::ExecutionContext(const ExecutionContextRef *exe_ctx_ref_ptr, } } -ExecutionContext::ExecutionContext(const ExecutionContextRef &exe_ctx_ref, - std::unique_lock<std::recursive_mutex> &lock) - : m_target_sp(exe_ctx_ref.GetTargetSP()), m_process_sp(), m_thread_sp(), - m_frame_sp() { - if (m_target_sp) { - lock = std::unique_lock<std::recursive_mutex>(m_target_sp->GetAPIMutex()); - - m_process_sp = exe_ctx_ref.GetProcessSP(); - m_thread_sp = exe_ctx_ref.GetThreadSP(); - m_frame_sp = exe_ctx_ref.GetFrameSP(); - } -} - ExecutionContext::ExecutionContext(ExecutionContextScope *exe_scope_ptr) : m_target_sp(), m_process_sp(), m_thread_sp(), m_frame_sp() { if (exe_scope_ptr) _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
