[Lldb-commits] [lldb] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in one of its call site (PR #146062)

2025-06-30 Thread Michael Buch via lldb-commits
@@ -1130,13 +1130,33 @@ void PlatformDarwin::AddClangModuleCompilationOptionsForSDKType( if (target) { if (ModuleSP exe_module_sp = target->GetExecutableModule()) { - auto path_or_err = ResolveSDKPathFromDebugInfo(*exe_module_sp); - if (path_or_err) { -

[Lldb-commits] [lldb] [lldb][DWARF64] Enable support for DWARF64 format handling (PR #145645)

2025-06-30 Thread Hemang Gadhavi via lldb-commits
https://github.com/HemangGadhavi updated https://github.com/llvm/llvm-project/pull/145645 >From a85d648ce62b69b870dda306dbdc2d412ed89d33 Mon Sep 17 00:00:00 2001 From: HemangGadhavi Date: Wed, 25 Jun 2025 02:06:05 -0400 Subject: [PATCH 1/3] [lldb][DWARF64] Enable support for DWARF64 format han

[Lldb-commits] [lldb] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in one of its call site (PR #146062)

2025-06-30 Thread Charles Zablit via lldb-commits
https://github.com/charles-zablit updated https://github.com/llvm/llvm-project/pull/146062 >From 6ce75177264d47ffedbf0e6ee44831eac1d3ea55 Mon Sep 17 00:00:00 2001 From: Charles Zablit Date: Fri, 27 Jun 2025 12:49:53 +0100 Subject: [PATCH 1/2] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in on

[Lldb-commits] [lldb] [lldb][DWARF64] Enable support for DWARF64 format handling (PR #145645)

2025-06-30 Thread Hemang Gadhavi via lldb-commits
@@ -0,0 +1,96 @@ +//===-- DWARF64UnitTest.cpp=---===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier: Apa

[Lldb-commits] [lldb] [LLDB] Add type summaries for MSVC STL strings (PR #143177)

2025-06-30 Thread Pavel Labath via lldb-commits
https://github.com/labath edited https://github.com/llvm/llvm-project/pull/143177 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [LLDB] Add type summaries for MSVC STL strings (PR #143177)

2025-06-30 Thread Pavel Labath via lldb-commits
https://github.com/labath commented: I think the tests are fine. @Michael137, @jimingham, do you have anything to say about the implementation? https://github.com/llvm/llvm-project/pull/143177 ___ lldb-commits mailing list lldb-commits@lists.llvm.org

[Lldb-commits] [lldb] [LLDB] Add type summaries for MSVC STL strings (PR #143177)

2025-06-30 Thread Pavel Labath via lldb-commits
@@ -299,6 +299,8 @@ def parseOptionsAndInitTestdirs(): configuration.libcxx_library_dir = args.libcxx_library_dir configuration.cmake_build_type = args.cmake_build_type.lower() +configuration.target_triple = args.target_triple + labath wrote: Yeah

[Lldb-commits] [lldb] [LLDB] Optimize identifier lookup in DIL (PR #146094)

2025-06-30 Thread Pavel Labath via lldb-commits
labath wrote: > Well, my logic was that it's better to return something than nothing, For a simple variable query, it might be okay since you kind of see the full name in the command output. However, for expressions like `a+b`, it's kind of important to know which `a`s and `b`s are you adding

[Lldb-commits] [lldb] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in one of its call site (PR #146062)

2025-06-30 Thread Charles Zablit via lldb-commits
@@ -1130,13 +1130,33 @@ void PlatformDarwin::AddClangModuleCompilationOptionsForSDKType( if (target) { if (ModuleSP exe_module_sp = target->GetExecutableModule()) { - auto path_or_err = ResolveSDKPathFromDebugInfo(*exe_module_sp); - if (path_or_err) { -

[Lldb-commits] [lldb] [lldb][DWARF64] Enable support for DWARF64 format handling (PR #145645)

2025-06-30 Thread Hemang Gadhavi via lldb-commits
@@ -119,10 +121,7 @@ bool DWARFFormValue::ExtractValue(const DWARFDataExtractor &data, break; case DW_FORM_ref_addr: assert(m_unit); - if (m_unit->GetVersion() <= 2) -ref_addr_size = m_unit->GetAddressByteSize(); - else -ref_addr_size

[Lldb-commits] [lldb] [lldb][DWARF64] Enable support for DWARF64 format handling (PR #145645)

2025-06-30 Thread Hemang Gadhavi via lldb-commits
https://github.com/HemangGadhavi edited https://github.com/llvm/llvm-project/pull/145645 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][DWARF64] Enable support for DWARF64 format handling (PR #145645)

2025-06-30 Thread Michael Buch via lldb-commits
@@ -119,10 +121,7 @@ bool DWARFFormValue::ExtractValue(const DWARFDataExtractor &data, break; case DW_FORM_ref_addr: assert(m_unit); - if (m_unit->GetVersion() <= 2) -ref_addr_size = m_unit->GetAddressByteSize(); - else -ref_addr_size

[Lldb-commits] [lldb] [lldb][DWARF64] Enable support for DWARF64 format handling (PR #145645)

2025-06-30 Thread Michael Buch via lldb-commits
@@ -119,10 +121,7 @@ bool DWARFFormValue::ExtractValue(const DWARFDataExtractor &data, break; case DW_FORM_ref_addr: assert(m_unit); - if (m_unit->GetVersion() <= 2) -ref_addr_size = m_unit->GetAddressByteSize(); - else -ref_addr_size

[Lldb-commits] [lldb] [lldb][DWARF64] Enable support for DWARF64 format handling (PR #145645)

2025-06-30 Thread Hemang Gadhavi via lldb-commits
@@ -119,10 +121,7 @@ bool DWARFFormValue::ExtractValue(const DWARFDataExtractor &data, break; case DW_FORM_ref_addr: assert(m_unit); - if (m_unit->GetVersion() <= 2) -ref_addr_size = m_unit->GetAddressByteSize(); - else -ref_addr_size

[Lldb-commits] [lldb] [LLDB] Optimize identifier lookup in DIL (PR #146094)

2025-06-30 Thread Pavel Labath via lldb-commits
@@ -129,28 +71,22 @@ lldb::ValueObjectSP LookupGlobalIdentifier( target_sp->GetImages().FindGlobalVariables( ConstString(name_ref), std::numeric_limits::max(), modules_var_list); - if (modules_var_list.Empty()) -return nullptr; - for (const lldb::VariableS

[Lldb-commits] [lldb] [lldb][DWARF64] Enable support for DWARF64 format handling (PR #145645)

2025-06-30 Thread Hemang Gadhavi via lldb-commits
https://github.com/HemangGadhavi edited https://github.com/llvm/llvm-project/pull/145645 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][DWARF64] Enable support for DWARF64 format handling (PR #145645)

2025-06-30 Thread Hemang Gadhavi via lldb-commits
https://github.com/HemangGadhavi updated https://github.com/llvm/llvm-project/pull/145645 >From a85d648ce62b69b870dda306dbdc2d412ed89d33 Mon Sep 17 00:00:00 2001 From: HemangGadhavi Date: Wed, 25 Jun 2025 02:06:05 -0400 Subject: [PATCH 1/3] [lldb][DWARF64] Enable support for DWARF64 format han

[Lldb-commits] [lldb] [lldb][DWARF64] Enable support for DWARF64 format handling (PR #145645)

2025-06-30 Thread Hemang Gadhavi via lldb-commits
@@ -288,7 +284,9 @@ bool DWARFFormValue::SkipValue(dw_form_t form, case DW_FORM_sec_offset: case DW_FORM_strp: case DW_FORM_line_strp: - *offset_ptr += 4; + assert(unit); // Unit must be valid HemangGadhavi wrote: Thanks @Michael137 Done

[Lldb-commits] [lldb] [lldb][DWARF64] Enable support for DWARF64 format handling (PR #145645)

2025-06-30 Thread Hemang Gadhavi via lldb-commits
@@ -77,7 +77,9 @@ bool DWARFFormValue::ExtractValue(const DWARFDataExtractor &data, case DW_FORM_strp: case DW_FORM_line_strp: case DW_FORM_sec_offset: - m_value.uval = data.GetMaxU64(offset_ptr, 4); + assert(m_unit); // Unit must be valid ---

[Lldb-commits] [lldb] [LLDB] Optimize identifier lookup in DIL (PR #146094)

2025-06-30 Thread Pavel Labath via lldb-commits
https://github.com/labath approved this pull request. https://github.com/llvm/llvm-project/pull/146094 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][DWARF64] Enable support for DWARF64 format handling (PR #145645)

2025-06-30 Thread Pavel Labath via lldb-commits
@@ -0,0 +1,96 @@ +//===-- DWARF64UnitTest.cpp=---===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier: Apa

[Lldb-commits] [lldb] [lldb][DWARF64] Enable support for DWARF64 format handling (PR #145645)

2025-06-30 Thread Pavel Labath via lldb-commits
https://github.com/labath approved this pull request. https://github.com/llvm/llvm-project/pull/145645 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [LLDB] Add type summaries for MSVC STL strings (PR #143177)

2025-06-30 Thread Pavel Labath via lldb-commits
@@ -239,122 +240,24 @@ VectorIteratorSyntheticFrontEnd::GetIndexOfChildWithName(ConstString name) { bool lldb_private::formatters::LibStdcppStringSummaryProvider( ValueObject &valobj, Stream &stream, const TypeSummaryOptions &options) { - const bool scalar_is_load_addr =

[Lldb-commits] [lldb] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in one of its call site (PR #146062)

2025-06-30 Thread Michael Buch via lldb-commits
@@ -1128,17 +1128,8 @@ void PlatformDarwin::AddClangModuleCompilationOptionsForSDKType( FileSpec sysroot_spec; - if (target) { -if (ModuleSP exe_module_sp = target->GetExecutableModule()) { - auto path_or_err = ResolveSDKPathFromDebugInfo(*exe_module_sp); -

[Lldb-commits] [lldb] [lldb][DWARF64] Enable support for DWARF64 format handling (PR #145645)

2025-06-30 Thread Hemang Gadhavi via lldb-commits
HemangGadhavi wrote: @DhruvSrivastavaX can you please merge it https://github.com/llvm/llvm-project/pull/145645 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][DWARF64] Enable support for DWARF64 format handling (PR #145645)

2025-06-30 Thread Michael Buch via lldb-commits
@@ -77,7 +77,9 @@ bool DWARFFormValue::ExtractValue(const DWARFDataExtractor &data, case DW_FORM_strp: case DW_FORM_line_strp: case DW_FORM_sec_offset: - m_value.uval = data.GetMaxU64(offset_ptr, 4); + assert(m_unit); // Unit must be valid ---

[Lldb-commits] [lldb] [lldb][DWARF64] Enable support for DWARF64 format handling (PR #145645)

2025-06-30 Thread Michael Buch via lldb-commits
@@ -288,7 +284,9 @@ bool DWARFFormValue::SkipValue(dw_form_t form, case DW_FORM_sec_offset: case DW_FORM_strp: case DW_FORM_line_strp: - *offset_ptr += 4; + assert(unit); // Unit must be valid Michael137 wrote: ```suggestion assert(

[Lldb-commits] [lldb] [lldb][DWARF64] Enable support for DWARF64 format handling (PR #145645)

2025-06-30 Thread Dhruv Srivastava via lldb-commits
DhruvSrivastavaX wrote: @HemangGadhavi , Please check the code format failure and lets me have a check before merging. https://github.com/llvm/llvm-project/pull/145645 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cg

[Lldb-commits] [lldb] [lldb][DWARF64] Enable support for DWARF64 format handling (PR #145645)

2025-06-30 Thread Michael Buch via lldb-commits
@@ -119,10 +121,7 @@ bool DWARFFormValue::ExtractValue(const DWARFDataExtractor &data, break; case DW_FORM_ref_addr: assert(m_unit); - if (m_unit->GetVersion() <= 2) -ref_addr_size = m_unit->GetAddressByteSize(); - else -ref_addr_size

[Lldb-commits] [lldb] [lldb][DWARF64] Enable support for DWARF64 format handling (PR #145645)

2025-06-30 Thread Michael Buch via lldb-commits
@@ -288,7 +284,9 @@ bool DWARFFormValue::SkipValue(dw_form_t form, case DW_FORM_sec_offset: case DW_FORM_strp: case DW_FORM_line_strp: - *offset_ptr += 4; + assert(unit); // Unit must be valid + ref_addr_size = unit->GetFormParams().getDwarfOffsetByte

[Lldb-commits] [lldb] [lldb] Adding pipe support to lldb_private::MainLoopWindows. (PR #145621)

2025-06-30 Thread Pavel Labath via lldb-commits
https://github.com/labath edited https://github.com/llvm/llvm-project/pull/145621 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Adding pipe support to lldb_private::MainLoopWindows. (PR #145621)

2025-06-30 Thread Pavel Labath via lldb-commits
https://github.com/labath approved this pull request. > I made a sample binary > (https://gist.github.com/ashgti/1649fde8ef28783ace1e414f87a6b680) and it > seems stdin is opened as an anonymous pipe. Got it. Thanks for checking it out. I guess we have no choice, but to create the extra thread

[Lldb-commits] [lldb] [lldb] Adding pipe support to lldb_private::MainLoopWindows. (PR #145621)

2025-06-30 Thread Pavel Labath via lldb-commits
@@ -31,6 +34,122 @@ static DWORD ToTimeout(std::optional point) { return ceil(dur).count(); } +namespace { + +class PipeEvent : public MainLoopWindows::IOEvent { +public: + explicit PipeEvent(HANDLE handle) + : IOEvent((IOObject::WaitableHandle)CreateEventW( +

[Lldb-commits] [lldb] [lldb] Adding pipe support to lldb_private::MainLoopWindows. (PR #145621)

2025-06-30 Thread Pavel Labath via lldb-commits
@@ -79,7 +79,68 @@ TEST_F(MainLoopTest, ReadObject) { ASSERT_EQ(1u, callback_count); } -TEST_F(MainLoopTest, NoSpuriousReads) { +TEST_F(MainLoopTest, ReadPipeObject) { + Pipe pipe; + + ASSERT_TRUE(pipe.CreateNew().Success()); + + MainLoop loop; + + char X = 'X'; + size_

[Lldb-commits] [lldb] [lldb] Adding pipe support to lldb_private::MainLoopWindows. (PR #145621)

2025-06-30 Thread Pavel Labath via lldb-commits
@@ -31,6 +34,122 @@ static DWORD ToTimeout(std::optional point) { return ceil(dur).count(); } +namespace { + +class PipeEvent : public MainLoopWindows::IOEvent { +public: + explicit PipeEvent(HANDLE handle) + : IOEvent((IOObject::WaitableHandle)CreateEventW( +

[Lldb-commits] [lldb] 24fd343 - [lldb][test] TestMetadataRegisters.py: add missing include

2025-06-30 Thread Michael Buch via lldb-commits
Author: Michael Buch Date: 2025-06-30T10:46:17+01:00 New Revision: 24fd3437c311e42891d117a9c4cf993032561bab URL: https://github.com/llvm/llvm-project/commit/24fd3437c311e42891d117a9c4cf993032561bab DIFF: https://github.com/llvm/llvm-project/commit/24fd3437c311e42891d117a9c4cf993032561bab.diff

[Lldb-commits] [lldb] [lldb] Adding pipe support to lldb_private::MainLoopWindows. (PR #145621)

2025-06-30 Thread Pavel Labath via lldb-commits
@@ -44,25 +163,20 @@ MainLoopWindows::~MainLoopWindows() { } llvm::Expected MainLoopWindows::Poll() { - std::vector events; + std::vector events; events.reserve(m_read_fds.size() + 1); - for (auto &[fd, info] : m_read_fds) { -int result = WSAEventSelect(fd, info.even

[Lldb-commits] [lldb] [lldb] Adding pipe support to lldb_private::MainLoopWindows. (PR #145621)

2025-06-30 Thread Pavel Labath via lldb-commits
@@ -79,7 +79,68 @@ TEST_F(MainLoopTest, ReadObject) { ASSERT_EQ(1u, callback_count); } -TEST_F(MainLoopTest, NoSpuriousReads) { +TEST_F(MainLoopTest, ReadPipeObject) { + Pipe pipe; + + ASSERT_TRUE(pipe.CreateNew().Success()); + + MainLoop loop; + + char X = 'X'; + size_

[Lldb-commits] [lldb] [LLDB] Add type summaries for MSVC STL strings (PR #143177)

2025-06-30 Thread David Spickett via lldb-commits
@@ -299,6 +299,8 @@ def parseOptionsAndInitTestdirs(): configuration.libcxx_library_dir = args.libcxx_library_dir configuration.cmake_build_type = args.cmake_build_type.lower() +configuration.target_triple = args.target_triple + DavidSpickett wrote

[Lldb-commits] [lldb] [lldb] Document MCP support in LLDB (PR #145935)

2025-06-30 Thread David Spickett via lldb-commits
@@ -0,0 +1,68 @@ +# Model Context Protocol (MCP) + +LLDB supports for the [Model Context Protocol][1] (MCP). This structured, +machine-friendly protocol allows AI models to access and interact with external +tools, such as the debugger. Using MCP, an AI agent can execute LLDB comm

[Lldb-commits] [lldb] [lldb-dap] Prevent using an implicit `step-in`. (PR #143644)

2025-06-30 Thread Pavel Labath via lldb-commits
labath wrote: Cool. The test case looks good. Someone might want to look at the DAP part. https://github.com/llvm/llvm-project/pull/143644 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-co

[Lldb-commits] [lldb] [LLDB] Optimize identifier lookup in DIL (PR #146094)

2025-06-30 Thread Ilia Kuklin via lldb-commits
kuilpd wrote: > I'm not sure that's the right thing to do -- if there are multiple matches, > how can we know we picked the one that the user wanted to see? > What might matter for performance is, if returning false/nullptr here causes > the implementation to perform the lookup at a larger (mor

[Lldb-commits] [lldb] [lldb][RPC] Upstream lldb-rpc-gen tool (PR #138031)

2025-06-30 Thread David Spickett via lldb-commits
DavidSpickett wrote: On the same page you view the job results, there is a button top right that says "Re-run Jobs". I just used that here and it is re-running the job, I don't have special permissions that I know of, so it should work for you. https://github.com/llvm/llvm-project/pull/138031

[Lldb-commits] [lldb] Fix a bug in the breakpoint ID verifier in CommandObjectBreakpoint. (PR #145994)

2025-06-30 Thread David Spickett via lldb-commits
DavidSpickett wrote: > We can't remove the program file while lldb has it open. ``` path_to_exe = self.getBuildArtifact() (target, process, thread, bkpt) = lldbutil.run_to_name_breakpoint(self, "main") # Let the process continue to exit: process.Continue()

[Lldb-commits] [lldb] [lldb] Document MCP support in LLDB (PR #145935)

2025-06-30 Thread Jonas Devlieghere via lldb-commits
https://github.com/JDevlieghere updated https://github.com/llvm/llvm-project/pull/145935 >From 2b6a7e22481761b244e98dff00f905cf32a7d696 Mon Sep 17 00:00:00 2001 From: Jonas Devlieghere Date: Thu, 26 Jun 2025 10:31:09 -0700 Subject: [PATCH 1/3] [lldb] Document MCP support in LLDB --- lldb/docs

[Lldb-commits] [lldb] [lldb] Document MCP support in LLDB (PR #145935)

2025-06-30 Thread Jonas Devlieghere via lldb-commits
https://github.com/JDevlieghere closed https://github.com/llvm/llvm-project/pull/145935 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][RISCV] fix LR/SC atomic sequence handling in lldb-server (PR #146072)

2025-06-30 Thread via lldb-commits
https://github.com/dlav-sc closed https://github.com/llvm/llvm-project/pull/146072 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][DWARF64] Enable support for DWARF64 format handling (PR #145645)

2025-06-30 Thread Jonas Devlieghere via lldb-commits
https://github.com/JDevlieghere approved this pull request. https://github.com/llvm/llvm-project/pull/145645 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][DWARF64] Enable support for DWARF64 format handling (PR #145645)

2025-06-30 Thread Jonas Devlieghere via lldb-commits
@@ -0,0 +1,86 @@ +//===-- DWARF64UnitTest.cpp=---===// JDevlieghere wrote: ```suggestion //===-- DWARF64UnitTest.cpp===// ``` https://github.com/llvm/llvm-project/pull/1

[Lldb-commits] [lldb] [lldb][RISCV] fix LR/SC atomic sequence handling in lldb-server (PR #146072)

2025-06-30 Thread via lldb-commits
https://github.com/dlav-sc edited https://github.com/llvm/llvm-project/pull/146072 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Delete unused DWARFDataExtractor methods (PR #146356)

2025-06-30 Thread Pavel Labath via lldb-commits
@@ -7,20 +7,10 @@ //===--===// #include "DWARFDataExtractor.h" -#include "llvm/ADT/StringRef.h" labath wrote: yeah, a bit of drive-by header gardening :P https://github.com/llvm/llvm-projec

[Lldb-commits] [lldb] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in one of its call site (PR #146062)

2025-06-30 Thread Charles Zablit via lldb-commits
@@ -1128,17 +1128,8 @@ void PlatformDarwin::AddClangModuleCompilationOptionsForSDKType( FileSpec sysroot_spec; - if (target) { -if (ModuleSP exe_module_sp = target->GetExecutableModule()) { - auto path_or_err = ResolveSDKPathFromDebugInfo(*exe_module_sp); -

[Lldb-commits] [lldb] [LLDB][NFC] Refactor code extracting timestamp from StructuredData (PR #145954)

2025-06-30 Thread Vy Nguyen via lldb-commits
https://github.com/oontvoo updated https://github.com/llvm/llvm-project/pull/145954 >From f4092ed5fda4214bfd2c72f288a6c686315bdc3b Mon Sep 17 00:00:00 2001 From: Vy Nguyen Date: Thu, 26 Jun 2025 15:18:44 -0400 Subject: [PATCH 1/3] [LLDB][NFC] Refactor code extracting timestamp from StructuredD

[Lldb-commits] [lldb] [LLDB][NFC] Refactor code extracting timestamp from StructuredData (PR #145954)

2025-06-30 Thread Vy Nguyen via lldb-commits
@@ -148,23 +161,14 @@ void TelemetryManager::DispatchClientTelemetry( LLDB_LOG(GetLog(LLDBLog::Object), "Cannot determine client_data from client-telemetry entry"); - int64_t start_time; - if (dict->GetValueForKeyAsInteger("start_time", start_time)) { -c

[Lldb-commits] [lldb] 5548f4d - [LLDB][NFC] Refactor code extracting timestamp from StructuredData (#145954)

2025-06-30 Thread via lldb-commits
Author: Vy Nguyen Date: 2025-06-30T14:25:11-04:00 New Revision: 5548f4d5ef6ab7192fe454a497af6f2cfe403339 URL: https://github.com/llvm/llvm-project/commit/5548f4d5ef6ab7192fe454a497af6f2cfe403339 DIFF: https://github.com/llvm/llvm-project/commit/5548f4d5ef6ab7192fe454a497af6f2cfe403339.diff LOG

[Lldb-commits] [lldb] [LLDB][NFC] Refactor code extracting timestamp from StructuredData (PR #145954)

2025-06-30 Thread Vy Nguyen via lldb-commits
https://github.com/oontvoo closed https://github.com/llvm/llvm-project/pull/145954 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Document MCP support in LLDB (PR #145935)

2025-06-30 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/145935 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] 25e5010 - [lldb] Delete unused variables in TestLoongArchEmulator.cpp.

2025-06-30 Thread James Y Knight via lldb-commits
Author: James Y Knight Date: 2025-06-30T11:54:36-04:00 New Revision: 25e5010a302f2c93397c56d7c07168e075a15a44 URL: https://github.com/llvm/llvm-project/commit/25e5010a302f2c93397c56d7c07168e075a15a44 DIFF: https://github.com/llvm/llvm-project/commit/25e5010a302f2c93397c56d7c07168e075a15a44.diff

[Lldb-commits] [lldb] 16dc6f6 - [lldb] Document MCP support in LLDB (#145935)

2025-06-30 Thread via lldb-commits
Author: Jonas Devlieghere Date: 2025-06-30T09:01:15-07:00 New Revision: 16dc6f64ded3f47a6c03098e021ae943714a7663 URL: https://github.com/llvm/llvm-project/commit/16dc6f64ded3f47a6c03098e021ae943714a7663 DIFF: https://github.com/llvm/llvm-project/commit/16dc6f64ded3f47a6c03098e021ae943714a7663.d

[Lldb-commits] [lldb] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in one of its call site (PR #146062)

2025-06-30 Thread Michael Buch via lldb-commits
@@ -1152,6 +1150,46 @@ void PlatformDarwin::AddClangModuleCompilationOptionsForSDKType( } } +llvm::Expected +lldb_private::PlatformDarwin::ResolveSDKPathFromDebugInfo( +lldb_private::Target *target) { + + ModuleSP exe_module_sp = target->GetExecutableModule(); + if (!

[Lldb-commits] [lldb] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in one of its call site (PR #146062)

2025-06-30 Thread Michael Buch via lldb-commits
@@ -1152,6 +1150,46 @@ void PlatformDarwin::AddClangModuleCompilationOptionsForSDKType( } } +llvm::Expected +lldb_private::PlatformDarwin::ResolveSDKPathFromDebugInfo( +lldb_private::Target *target) { + + ModuleSP exe_module_sp = target->GetExecutableModule(); + if (!

[Lldb-commits] [lldb] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in one of its call site (PR #146062)

2025-06-30 Thread Michael Buch via lldb-commits
@@ -187,6 +187,9 @@ class PlatformDarwin : public PlatformPOSIX { std::vector &options, XcodeSDK::Type sdk_type); + static llvm::Expected + ResolveSDKPathFromDebugInfo(lldb_private::Tar

[Lldb-commits] [lldb] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in one of its call site (PR #146062)

2025-06-30 Thread Michael Buch via lldb-commits
https://github.com/Michael137 edited https://github.com/llvm/llvm-project/pull/146062 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in one of its call site (PR #146062)

2025-06-30 Thread Michael Buch via lldb-commits
https://github.com/Michael137 approved this pull request. LGTM, just some final stylistic comments https://github.com/llvm/llvm-project/pull/146062 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinf

[Lldb-commits] [lldb] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in one of its call site (PR #146062)

2025-06-30 Thread Michael Buch via lldb-commits
@@ -1152,6 +1150,46 @@ void PlatformDarwin::AddClangModuleCompilationOptionsForSDKType( } } +llvm::Expected +lldb_private::PlatformDarwin::ResolveSDKPathFromDebugInfo( +lldb_private::Target *target) { + + ModuleSP exe_module_sp = target->GetExecutableModule(); + if (!

[Lldb-commits] [lldb] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in one of its call site (PR #146062)

2025-06-30 Thread Michael Buch via lldb-commits
@@ -1152,6 +1150,46 @@ void PlatformDarwin::AddClangModuleCompilationOptionsForSDKType( } } +llvm::Expected +lldb_private::PlatformDarwin::ResolveSDKPathFromDebugInfo( +lldb_private::Target *target) { + + ModuleSP exe_module_sp = target->GetExecutableModule(); + if (!

[Lldb-commits] [lldb] [lldb] Delete unused DWARFDataExtractor methods (PR #146356)

2025-06-30 Thread Michael Buch via lldb-commits
https://github.com/Michael137 approved this pull request. https://github.com/llvm/llvm-project/pull/146356 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Delete unused DWARFDataExtractor methods (PR #146356)

2025-06-30 Thread Michael Buch via lldb-commits
@@ -7,20 +7,10 @@ //===--===// #include "DWARFDataExtractor.h" -#include "llvm/ADT/StringRef.h" Michael137 wrote: Makes sense to include `ArrayRef` though since it's used in this source file

[Lldb-commits] [lldb] [lldb] Delete unused DWARFDataExtractor methods (PR #146356)

2025-06-30 Thread Michael Buch via lldb-commits
@@ -7,20 +7,10 @@ //===--===// #include "DWARFDataExtractor.h" -#include "llvm/ADT/StringRef.h" Michael137 wrote: How come this changed? Is this `clangd`? https://github.com/llvm/llvm-proje

[Lldb-commits] [lldb] [lldb] Correctly restore the cursor column after resizing the statusline (PR #145823)

2025-06-30 Thread Jonas Devlieghere via lldb-commits
JDevlieghere wrote: > Yeah, that would definitely be bad, but I feel like it should be avoidable. I > take it that's a side effect of implementing resize by disabling and > re-enabling the status line. However, does it have to be that way? What if > resize was a separate operation that just in

[Lldb-commits] [lldb] [lldb] Correctly restore the cursor column after resizing the statusline (PR #146132)

2025-06-30 Thread Jonas Devlieghere via lldb-commits
JDevlieghere wrote: > This looks good. Any chance of a test case? This sounds like something > pexpect should manage to handle. I'm somewhat skeptical of the value, but I can add a test case to check for the escape codes to make sure the new Editline code is called. https://github.com/llvm/l

[Lldb-commits] [lldb] [lldb] Adding pipe support to lldb_private::MainLoopWindows. (PR #145621)

2025-06-30 Thread John Harrison via lldb-commits
https://github.com/ashgti updated https://github.com/llvm/llvm-project/pull/145621 >From 79c10b6f6630b3ceaa85af45956aeafdf372b5be Mon Sep 17 00:00:00 2001 From: John Harrison Date: Wed, 25 Jun 2025 18:03:39 -0700 Subject: [PATCH 1/7] [lldb] Adding file and pipe support to lldb_private::MainLoo

[Lldb-commits] [lldb] [lldb] Adding pipe support to lldb_private::MainLoopWindows. (PR #145621)

2025-06-30 Thread John Harrison via lldb-commits
@@ -31,6 +34,122 @@ static DWORD ToTimeout(std::optional point) { return ceil(dur).count(); } +namespace { + +class PipeEvent : public MainLoopWindows::IOEvent { +public: + explicit PipeEvent(HANDLE handle) + : IOEvent((IOObject::WaitableHandle)CreateEventW( +

[Lldb-commits] [lldb] [lldb] Adding pipe support to lldb_private::MainLoopWindows. (PR #145621)

2025-06-30 Thread John Harrison via lldb-commits
https://github.com/ashgti updated https://github.com/llvm/llvm-project/pull/145621 >From 79c10b6f6630b3ceaa85af45956aeafdf372b5be Mon Sep 17 00:00:00 2001 From: John Harrison Date: Wed, 25 Jun 2025 18:03:39 -0700 Subject: [PATCH 1/6] [lldb] Adding file and pipe support to lldb_private::MainLoo

[Lldb-commits] [lldb] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in one of its call site (PR #146062)

2025-06-30 Thread Charles Zablit via lldb-commits
https://github.com/charles-zablit updated https://github.com/llvm/llvm-project/pull/146062 >From 6ce75177264d47ffedbf0e6ee44831eac1d3ea55 Mon Sep 17 00:00:00 2001 From: Charles Zablit Date: Fri, 27 Jun 2025 12:49:53 +0100 Subject: [PATCH 1/3] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in on

[Lldb-commits] [lldb] [lldb] Delete unused DWARFDataExtractor methods (PR #146356)

2025-06-30 Thread Pavel Labath via lldb-commits
https://github.com/labath created https://github.com/llvm/llvm-project/pull/146356 They are left over from our previous attempt at DWARF64. The new attempt is not using them, and they also don't have equivalents in the llvm DWARFDataExtractor class. >From b2ad70820e53be64e5908c84d8b828369dce9

[Lldb-commits] [lldb] [lldb] Delete unused DWARFDataExtractor methods (PR #146356)

2025-06-30 Thread via lldb-commits
llvmbot wrote: @llvm/pr-subscribers-lldb Author: Pavel Labath (labath) Changes They are left over from our previous attempt at DWARF64. The new attempt is not using them, and they also don't have equivalents in the llvm DWARFDataExtractor class. --- Full diff: https://github.com/llvm/ll

[Lldb-commits] [lldb] [lldb] Correctly restore the cursor column after resizing the statusline (PR #146132)

2025-06-30 Thread Jonas Devlieghere via lldb-commits
https://github.com/JDevlieghere updated https://github.com/llvm/llvm-project/pull/146132 >From e601025c14fb609635dc6a9adea5b1ae8b8e4540 Mon Sep 17 00:00:00 2001 From: Jonas Devlieghere Date: Fri, 27 Jun 2025 11:08:08 -0700 Subject: [PATCH 1/2] [lldb] Correctly restore the cursor column after re

[Lldb-commits] [lldb] [lldb] Correctly restore the cursor column after resizing the statusline (PR #146132)

2025-06-30 Thread via lldb-commits
github-actions[bot] wrote: :warning: Python code formatter, darker found issues in your code. :warning: You can test this locally with the following command: ``bash darker --check --diff -r HEAD~1...HEAD lldb/test/API/functionalities/statusline/TestStatusline.py `` V

[Lldb-commits] [lldb] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in one of its call site (PR #146062)

2025-06-30 Thread Charles Zablit via lldb-commits
https://github.com/charles-zablit updated https://github.com/llvm/llvm-project/pull/146062 >From 6ce75177264d47ffedbf0e6ee44831eac1d3ea55 Mon Sep 17 00:00:00 2001 From: Charles Zablit Date: Fri, 27 Jun 2025 12:49:53 +0100 Subject: [PATCH 1/6] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in on

[Lldb-commits] [lldb] d7e23be - [lldb][RISCV] fix LR/SC atomic sequence handling in lldb-server (#146072)

2025-06-30 Thread via lldb-commits
Author: dlav-sc Date: 2025-06-30T16:27:44+03:00 New Revision: d7e23bef6ad497cd5a100126957c381e053dda9b URL: https://github.com/llvm/llvm-project/commit/d7e23bef6ad497cd5a100126957c381e053dda9b DIFF: https://github.com/llvm/llvm-project/commit/d7e23bef6ad497cd5a100126957c381e053dda9b.diff LOG:

[Lldb-commits] [lldb] [lldb] Delete unused DWARFDataExtractor methods (PR #146356)

2025-06-30 Thread Jonas Devlieghere via lldb-commits
https://github.com/JDevlieghere approved this pull request. https://github.com/llvm/llvm-project/pull/146356 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in one of its call site (PR #146062)

2025-06-30 Thread Charles Zablit via lldb-commits
https://github.com/charles-zablit updated https://github.com/llvm/llvm-project/pull/146062 >From b7700480aa7150237f73aa37b0cf39f592ac512d Mon Sep 17 00:00:00 2001 From: Charles Zablit Date: Fri, 27 Jun 2025 12:49:53 +0100 Subject: [PATCH 1/6] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in on

[Lldb-commits] [lldb] [lldb] Adding pipe support to lldb_private::MainLoopWindows. (PR #145621)

2025-06-30 Thread John Harrison via lldb-commits
https://github.com/ashgti updated https://github.com/llvm/llvm-project/pull/145621 >From 79c10b6f6630b3ceaa85af45956aeafdf372b5be Mon Sep 17 00:00:00 2001 From: John Harrison Date: Wed, 25 Jun 2025 18:03:39 -0700 Subject: [PATCH 1/8] [lldb] Adding file and pipe support to lldb_private::MainLoo

[Lldb-commits] [lldb] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in one of its call site (PR #146062)

2025-06-30 Thread Charles Zablit via lldb-commits
https://github.com/charles-zablit updated https://github.com/llvm/llvm-project/pull/146062 >From 6ce75177264d47ffedbf0e6ee44831eac1d3ea55 Mon Sep 17 00:00:00 2001 From: Charles Zablit Date: Fri, 27 Jun 2025 12:49:53 +0100 Subject: [PATCH 1/4] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in on

[Lldb-commits] [lldb] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in one of its call site (PR #146062)

2025-06-30 Thread Charles Zablit via lldb-commits
https://github.com/charles-zablit updated https://github.com/llvm/llvm-project/pull/146062 >From 6ce75177264d47ffedbf0e6ee44831eac1d3ea55 Mon Sep 17 00:00:00 2001 From: Charles Zablit Date: Fri, 27 Jun 2025 12:49:53 +0100 Subject: [PATCH 1/5] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in on

[Lldb-commits] [lldb] [lldb] Add DWARFExpressionEntry and GetExpressionEntryAtAddress() to … (PR #144238)

2025-06-30 Thread Adrian Prantl via lldb-commits
@@ -53,6 +54,30 @@ bool DWARFExpressionList::ContainsAddress(lldb::addr_t func_load_addr, return GetExpressionAtAddress(func_load_addr, addr) != nullptr; } +std::optional +DWARFExpressionList::GetExpressionEntryAtAddress(lldb::addr_t func_load_addr, +

[Lldb-commits] [lldb] [lldb] Add DWARFExpressionEntry and GetExpressionEntryAtAddress() to … (PR #144238)

2025-06-30 Thread Adrian Prantl via lldb-commits
@@ -53,6 +54,30 @@ bool DWARFExpressionList::ContainsAddress(lldb::addr_t func_load_addr, return GetExpressionAtAddress(func_load_addr, addr) != nullptr; } +std::optional +DWARFExpressionList::GetExpressionEntryAtAddress(lldb::addr_t func_load_addr, +

[Lldb-commits] [lldb] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in one of its call site (PR #146062)

2025-06-30 Thread Adrian Prantl via lldb-commits
https://github.com/adrian-prantl approved this pull request. LGTM with Michael's comments addressed. https://github.com/llvm/llvm-project/pull/146062 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listi

[Lldb-commits] [lldb] [lldb] Add DWARFExpressionEntry and GetExpressionEntryAtAddress() to … (PR #144238)

2025-06-30 Thread Adrian Prantl via lldb-commits
https://github.com/adrian-prantl commented: LGTM with under/overflow checks added. https://github.com/llvm/llvm-project/pull/144238 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][RPC] Upstream lldb-rpc-gen tool (PR #138031)

2025-06-30 Thread Chelsea Cassanova via lldb-commits
chelcassanova wrote: `home/gha/actions-runner/_work/llvm-project/llvm-project/build/tools/lldb/test/Shell/RPC/Generator/Tests/Output/CheckRPCGenToolByproducts.test.script: line 1: fg: no job control` This means that the test does not recognize the `lldb-rpc-gen` binary. It was working before an

[Lldb-commits] [lldb] [lldb] Correctly restore the cursor column after resizing the statusline (PR #146132)

2025-06-30 Thread Jonas Devlieghere via lldb-commits
https://github.com/JDevlieghere updated https://github.com/llvm/llvm-project/pull/146132 >From e601025c14fb609635dc6a9adea5b1ae8b8e4540 Mon Sep 17 00:00:00 2001 From: Jonas Devlieghere Date: Fri, 27 Jun 2025 11:08:08 -0700 Subject: [PATCH 1/3] [lldb] Correctly restore the cursor column after re

[Lldb-commits] [lldb] [lldb] Delete unused DWARFDataExtractor methods (PR #146356)

2025-06-30 Thread Alex Langford via lldb-commits
https://github.com/bulbazord approved this pull request. https://github.com/llvm/llvm-project/pull/146356 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [LLDB][NFC] Refactor code extracting timestamp from StructuredData (PR #145954)

2025-06-30 Thread Alex Langford via lldb-commits
https://github.com/bulbazord approved this pull request. https://github.com/llvm/llvm-project/pull/145954 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [LLDB][NFC] Refactor code extracting timestamp from StructuredData (PR #145954)

2025-06-30 Thread Alex Langford via lldb-commits
@@ -148,23 +161,12 @@ void TelemetryManager::DispatchClientTelemetry( LLDB_LOG(GetLog(LLDBLog::Object), "Cannot determine client_data from client-telemetry entry"); - int64_t start_time; - if (dict->GetValueForKeyAsInteger("start_time", start_time)) { -c

[Lldb-commits] [lldb] [LLDB][NFC] Refactor code extracting timestamp from StructuredData (PR #145954)

2025-06-30 Thread Vy Nguyen via lldb-commits
oontvoo wrote: Thanks! https://github.com/llvm/llvm-project/pull/145954 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [LLDB][NFC] Refactor code extracting timestamp from StructuredData (PR #145954)

2025-06-30 Thread Vy Nguyen via lldb-commits
https://github.com/oontvoo updated https://github.com/llvm/llvm-project/pull/145954 >From f4092ed5fda4214bfd2c72f288a6c686315bdc3b Mon Sep 17 00:00:00 2001 From: Vy Nguyen Date: Thu, 26 Jun 2025 15:18:44 -0400 Subject: [PATCH 1/4] [LLDB][NFC] Refactor code extracting timestamp from StructuredD

[Lldb-commits] [lldb] eb904e8 - [lldb] Fix link syntax in docs/use/mcp.md

2025-06-30 Thread Jonas Devlieghere via lldb-commits
Author: Jonas Devlieghere Date: 2025-06-30T13:50:32-07:00 New Revision: eb904e857af19abc2ea32ea3be455d96af6902f1 URL: https://github.com/llvm/llvm-project/commit/eb904e857af19abc2ea32ea3be455d96af6902f1 DIFF: https://github.com/llvm/llvm-project/commit/eb904e857af19abc2ea32ea3be455d96af6902f1.d

[Lldb-commits] [lldb] [lldb][Mach-O] Fix several bugs in x86_64 Mach-O corefile (PR #146460)

2025-06-30 Thread Jason Molenda via lldb-commits
https://github.com/jasonmolenda updated https://github.com/llvm/llvm-project/pull/146460 >From a9b23cc16ca7b3a20ccc7c63e64f840fcbf7da9d Mon Sep 17 00:00:00 2001 From: Jason Molenda Date: Mon, 30 Jun 2025 20:26:40 -0700 Subject: [PATCH 1/2] [lldb][Mach-O] Fix several bugs in x86_64 Mach-O corefi

[Lldb-commits] [lldb] [lldb][Mach-O] Fix several bugs in x86_64 Mach-O corefile (PR #146460)

2025-06-30 Thread Jason Molenda via lldb-commits
https://github.com/jasonmolenda closed https://github.com/llvm/llvm-project/pull/146460 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

  1   2   >