[Lldb-commits] [lldb] [lldb][windows] add support for out of PATH python.dll resolution (PR #162509)

2025-10-09 Thread Charles Zablit via lldb-commits
https://github.com/charles-zablit updated https://github.com/llvm/llvm-project/pull/162509 >From 37cf46ea187afd50ce8b1eae58239aafe9b115db Mon Sep 17 00:00:00 2001 From: Charles Zablit Date: Wed, 8 Oct 2025 17:18:51 +0100 Subject: [PATCH 1/4] [lldb][windows] add support for out of PATH python.dl

[Lldb-commits] [lldb] [lldb-dap] expand tilde in dap executable path (PR #162635)

2025-10-09 Thread Ebuka Ezike via lldb-commits
https://github.com/da-viper created https://github.com/llvm/llvm-project/pull/162635 Users may have multiple devices and would like to resolve the homepath based on the machine they are on. >From c62a3532785cc0b5a72475a965ba4a99404432ba Mon Sep 17 00:00:00 2001 From: Ebuka Ezike Date: Thu, 9

[Lldb-commits] [lldb] [debugserver][NFC] Add helper function for escaping special characters (PR #162297)

2025-10-09 Thread Felipe de Azevedo Piovezan via lldb-commits
felipepiovezan wrote: > Assuming this is part of `lldbDebugserverCommon`, can we add a unit test for > this? I don't believe that exists today, but it shouldn't be hard to add. We > have some prior art with the unit tests that I added for the library code of > lldb-dap. it's not, these are in

[Lldb-commits] [lldb] 36bce68 - Add a scripted way to re-present a stop location (#158128)

2025-10-09 Thread via lldb-commits
Author: jimingham Date: 2025-10-09T08:37:21-07:00 New Revision: 36bce68b97316363085ae3681e8dde33a62fc9b1 URL: https://github.com/llvm/llvm-project/commit/36bce68b97316363085ae3681e8dde33a62fc9b1 DIFF: https://github.com/llvm/llvm-project/commit/36bce68b97316363085ae3681e8dde33a62fc9b1.diff LOG

[Lldb-commits] [lldb] [lldb][test] Don't run libc++ API tests without a locally built libc++ (PR #162657)

2025-10-09 Thread Michael Buch via lldb-commits
https://github.com/Michael137 updated https://github.com/llvm/llvm-project/pull/162657 >From 0b23bd05541354a493acea8fbb2c89b771a3e774 Mon Sep 17 00:00:00 2001 From: Michael Buch Date: Thu, 9 Oct 2025 13:06:40 +0100 Subject: [PATCH 1/2] [lldb][test] Don't run libc++ API tests without a locally

[Lldb-commits] [lldb] [lldb] Adding A new Binding helper for JSONTransport. (PR #159160)

2025-10-09 Thread John Harrison via lldb-commits
ashgti wrote: I can update the test names to not match the using definition. https://github.com/llvm/llvm-project/pull/159160 ___ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][test] Don't run libc++ API tests without a locally built libc++ (PR #162657)

2025-10-09 Thread Michael Buch via lldb-commits
https://github.com/Michael137 ready_for_review https://github.com/llvm/llvm-project/pull/162657 ___ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb-dap] expand tilde in dap executable path (PR #162635)

2025-10-09 Thread via lldb-commits
llvmbot wrote: @llvm/pr-subscribers-lldb Author: Ebuka Ezike (da-viper) Changes Users may have multiple devices and would like to resolve the homepath based on the machine they are on. --- Full diff: https://github.com/llvm/llvm-project/pull/162635.diff 1 Files Affected: - (modified)

[Lldb-commits] [lldb] Fix comment typo in `StructuredDataPlugin.cpp` (PR #162637)

2025-10-09 Thread Max Desiatov via lldb-commits
https://github.com/MaxDesiatov created https://github.com/llvm/llvm-project/pull/162637 `structured-data ommand` -> `structured-data command` >From 6eb937909ca641e949d4ea05b9dfb1f9be7121cd Mon Sep 17 00:00:00 2001 From: Max Desiatov Date: Thu, 9 Oct 2025 12:42:42 +0100 Subject: [PATCH] Fix com

[Lldb-commits] [lldb] Fix comment typo in `StructuredDataPlugin.cpp` (PR #162637)

2025-10-09 Thread via lldb-commits
github-actions[bot] wrote: Thank you for submitting a Pull Request (PR) to the LLVM Project! This PR will be automatically labeled and the relevant teams will be notified. If you wish to, you can add reviewers by using the "Reviewers" section on this page. If this is not working for you, it

[Lldb-commits] [lldb] [lldb] Fix assertion caused by invalid SupportFileSP (PR #162710)

2025-10-09 Thread via lldb-commits
llvmbot wrote: @llvm/pr-subscribers-lldb Author: Jonas Devlieghere (JDevlieghere) Changes SupportFileSP should never be null, and instead should use a default constructed SupportFile to represent an invalid instance. This is because the class used to be a value type before it became poly

[Lldb-commits] [lldb] 8523c6a - [lldb] Actually use new SharedModuleList class (#162574)

2025-10-09 Thread via lldb-commits
Author: Augusto Noronha Date: 2025-10-09T10:39:45-07:00 New Revision: 8523c6a448c3f01396b805aca30376072c469845 URL: https://github.com/llvm/llvm-project/commit/8523c6a448c3f01396b805aca30376072c469845 DIFF: https://github.com/llvm/llvm-project/commit/8523c6a448c3f01396b805aca30376072c469845.dif

[Lldb-commits] [lldb] [gdbremote] Document MultiMemRead packet in protocol extensions (PR #162675)

2025-10-09 Thread via lldb-commits
llvmbot wrote: @llvm/pr-subscribers-lldb Author: Felipe de Azevedo Piovezan (felipepiovezan) Changes This adds a specification for the new packet discussed in the RFC [1]. [1]: https://discourse.llvm.org/t/rfc-a-new-vectorized-memory-read-packet/88441/12 --- Full diff: https://github.co

[Lldb-commits] [lldb] [debugserver] Implement MultiMemRead packet (PR #162670)

2025-10-09 Thread Felipe de Azevedo Piovezan via lldb-commits
https://github.com/felipepiovezan ready_for_review https://github.com/llvm/llvm-project/pull/162670 ___ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][windows] fix undeclared identifier error (PR #162705)

2025-10-09 Thread Charles Zablit via lldb-commits
https://github.com/charles-zablit closed https://github.com/llvm/llvm-project/pull/162705 ___ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [debugserver] Implement MultiMemRead packet (PR #162670)

2025-10-09 Thread Dave Lee via lldb-commits
@@ -3160,6 +3165,140 @@ rnb_err_t RNBRemote::HandlePacket_m(const char *p) { return SendPacket(ostrm.str()); } +/// Returns true if `str` starts with `prefix`. +static bool starts_with(std::string_view str, std::string_view prefix) { + return str.size() >= prefix.size() &&

[Lldb-commits] [lldb] [debugserver] Implement MultiMemRead packet (PR #162670)

2025-10-09 Thread Felipe de Azevedo Piovezan via lldb-commits
@@ -3160,6 +3165,140 @@ rnb_err_t RNBRemote::HandlePacket_m(const char *p) { return SendPacket(ostrm.str()); } +/// Returns true if `str` starts with `prefix`. +static bool starts_with(std::string_view str, std::string_view prefix) { + return str.size() >= prefix.size() &&

[Lldb-commits] [lldb] [lldb] Adding A new Binding helper for JSONTransport. (PR #159160)

2025-10-09 Thread Carlos Alberto Enciso via lldb-commits
CarlosAlbertoEnciso wrote: > I'm seeing compilation errors with gcc 11: > > ``` > /work/kparzysz/git/llvm.org/lldb/unittests/Host/JSONTransportTest.cpp:190:9: > error: declaration of ‘using Req = struct {anonymous}::test_protocol::Req’ > changes meaning of ‘Req’ [-fpermissive] > 190 | usin

[Lldb-commits] [lldb] [gdbremote] Document MultiMemRead packet in protocol extensions (PR #162675)

2025-10-09 Thread Alex Langford via lldb-commits
@@ -2530,3 +2530,41 @@ read packet: $e0030100#b9 **Priority to Implement:** Only required for Wasm support. Necessary to show variables. + +### MultiMemRead + +Read memory from multiple memory addresses. + +There are two arguments to the request: + +* `ranges`: a list of base-

[Lldb-commits] [lldb] [gdbremote] Document MultiMemRead packet in protocol extensions (PR #162675)

2025-10-09 Thread Alex Langford via lldb-commits
@@ -2530,3 +2530,41 @@ read packet: $e0030100#b9 **Priority to Implement:** Only required for Wasm support. Necessary to show variables. + +### MultiMemRead + +Read memory from multiple memory addresses. + +There are two arguments to the request: + +* `ranges`: a list of base-

[Lldb-commits] [lldb] support attaching by name for platform android (PR #160931)

2025-10-09 Thread Jonas Devlieghere via lldb-commits
https://github.com/JDevlieghere closed https://github.com/llvm/llvm-project/pull/160931 ___ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [debugserver] Implement MultiMemRead packet (PR #162670)

2025-10-09 Thread Felipe de Azevedo Piovezan via lldb-commits
@@ -3160,6 +3165,140 @@ rnb_err_t RNBRemote::HandlePacket_m(const char *p) { return SendPacket(ostrm.str()); } +/// Returns true if `str` starts with `prefix`. +static bool starts_with(std::string_view str, std::string_view prefix) { + return str.size() >= prefix.size() &&

[Lldb-commits] [lldb] [lldb] Fix TypeSystemClang::GetBasicTypeEnumeration for 128-bit int types (PR #162278)

2025-10-09 Thread Greg Clayton via lldb-commits
@@ -849,8 +849,8 @@ lldb::BasicType TypeSystemClang::GetBasicTypeEnumeration(llvm::StringRef name) { {"unsigned long long int", eBasicTypeUnsignedLongLong}, // "int128" - {"__int128_t", eBasicTypeInt128}, - {"__uint128_t", eBasicTypeUnsignedInt128}, +

[Lldb-commits] [lldb] [lldb] Fix TypeSystemClang::GetBasicTypeEnumeration for 128-bit int types (PR #162278)

2025-10-09 Thread Greg Clayton via lldb-commits
https://github.com/clayborg commented: This looks ok, but I think we should leave the old typename mappings in to make sure we don't break anyone's scripts that might be using the `__int128_t` or `__uint128_t` https://github.com/llvm/llvm-project/pull/162278 ___

[Lldb-commits] [lldb] eb06c7e - [lldb] Fix assertion caused by invalid SupportFileSP (#162710)

2025-10-09 Thread via lldb-commits
Author: Jonas Devlieghere Date: 2025-10-09T12:05:00-07:00 New Revision: eb06c7e7d25da30dd611812a9bec56bf5c3f5ac3 URL: https://github.com/llvm/llvm-project/commit/eb06c7e7d25da30dd611812a9bec56bf5c3f5ac3 DIFF: https://github.com/llvm/llvm-project/commit/eb06c7e7d25da30dd611812a9bec56bf5c3f5ac3.d

[Lldb-commits] [lldb] [llvm] [lldb] Add type hints to gdbclientutils.py and use abstract base class (PR #162172)

2025-10-09 Thread Daniel Sanders via lldb-commits
@@ -242,7 +245,7 @@ def qProcessInfo(self): def qHostInfo(self): return "ptrsize:8;endian:little;" -def qEcho(self): +def qEcho(self, _: int): dsandersllvm wrote: Ok, I've added the name. This does cause tools like pyright to report it as

[Lldb-commits] [lldb] update lldb-server platform help text (PR #162730)

2025-10-09 Thread Chad Smith via lldb-commits
cs01 wrote: Added a few tests, one for help text, one for error messages, and one for successful startups. ``` llvm-lit -v lldb-server/TestPlatformHelp.test lldb-server/TestPlatformErrorMessages.test lldb-server/TestPlatformSuccessfulStartup.test ... -- Testing: 3 tests, 3 workers -- PASS: lld

[Lldb-commits] [lldb] [llvm] [lldb] Add type hints to gdbclientutils.py and use abstract base class (PR #162172)

2025-10-09 Thread Daniel Sanders via lldb-commits
dsandersllvm wrote: > And yes we should have something somewhere enforcing our minimums but let me > put it this way: we claim minimum compiler versions and those aren't tested > by anyone either. So I'm sorry you had to encounter this confusion but it is > what it is for the time being. No w

[Lldb-commits] [lldb] update lldb-server platform help parsing (PR #162730)

2025-10-09 Thread Jonas Devlieghere via lldb-commits
@@ -371,107 +424,107 @@ int main_platform(int argc, char *argv[]) { signal(SIGPIPE, SIG_IGN); signal(SIGHUP, signal_handler); #endif - int long_option_index = 0; + + // Special handling for 'help' as first argument + if (argc > 0 && strcmp(argv[0], "help") == 0) { +L

[Lldb-commits] [lldb] [lldb][Expression] Emit a 'Note' diagnostic that indicates the language used for expression evaluation (PR #161688)

2025-10-09 Thread Michael Buch via lldb-commits
https://github.com/Michael137 updated https://github.com/llvm/llvm-project/pull/161688 >From 7300143d42a2a496d5ce522c7f862cb458fa6d9d Mon Sep 17 00:00:00 2001 From: Michael Buch Date: Thu, 2 Oct 2025 16:24:12 +0100 Subject: [PATCH 1/7] [lldb][Expression] Emit a 'Note' diagnistc that indicates

[Lldb-commits] [lldb] [lldb][test] Don't run libc++ API tests without a locally built libc++ (PR #162657)

2025-10-09 Thread Michael Buch via lldb-commits
https://github.com/Michael137 created https://github.com/llvm/llvm-project/pull/162657 API tests in the `libc++` category will try their best to build against a locally built libc++. If none exists, the `Makefile.rules` currently fall back to using the system libc++. The issue with falling ba

[Lldb-commits] [lldb] [lldb][test] Don't run libc++ API tests without a locally built libc++ (PR #162657)

2025-10-09 Thread Michael Buch via lldb-commits
https://github.com/Michael137 edited https://github.com/llvm/llvm-project/pull/162657 ___ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] support attaching by name for platform android (PR #160931)

2025-10-09 Thread Walter Erquinigo via lldb-commits
https://github.com/walter-erquinigo approved this pull request. fancy https://github.com/llvm/llvm-project/pull/160931 ___ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] support attaching by name for platform android (PR #160931)

2025-10-09 Thread Walter Erquinigo via lldb-commits
@@ -477,6 +477,248 @@ std::string PlatformAndroid::GetRunAs() { } return run_as.str(); } + +// Helper function to populate process status information from +// /proc/[pid]/status +void PlatformAndroid::PopulateProcessStatusInfo( +lldb::pid_t pid, ProcessInstanceInfo &pro

[Lldb-commits] [lldb] [lldb-dap] Allow empty memory reference in disassemble arguments (PR #162517)

2025-10-09 Thread Walter Erquinigo via lldb-commits
https://github.com/walter-erquinigo edited https://github.com/llvm/llvm-project/pull/162517 ___ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] update lldb-server platform help text (PR #162730)

2025-10-09 Thread Chad Smith via lldb-commits
https://github.com/cs01 updated https://github.com/llvm/llvm-project/pull/162730 >From 6d6cabbca4280bed1041423096bb071487e19248 Mon Sep 17 00:00:00 2001 From: Chad Smith Date: Thu, 9 Oct 2025 11:52:01 -0700 Subject: [PATCH] update lldb-server platform help text --- lldb/tools/lldb-server/CMake

[Lldb-commits] [lldb] [debugserver] Implement MultiMemRead packet (PR #162670)

2025-10-09 Thread Jason Molenda via lldb-commits
@@ -3160,6 +3165,140 @@ rnb_err_t RNBRemote::HandlePacket_m(const char *p) { return SendPacket(ostrm.str()); } +/// Returns true if `str` starts with `prefix`. +static bool starts_with(std::string_view str, std::string_view prefix) { + return str.size() >= prefix.size() &&

[Lldb-commits] [lldb] 7a391e3 - [lldb] Adjusting the naming for gcc11. (#162693)

2025-10-09 Thread via lldb-commits
Author: John Harrison Date: 2025-10-09T17:15:19-07:00 New Revision: 7a391e32eaf2f2232bd7c81e651dd949f39cc9e1 URL: https://github.com/llvm/llvm-project/commit/7a391e32eaf2f2232bd7c81e651dd949f39cc9e1 DIFF: https://github.com/llvm/llvm-project/commit/7a391e32eaf2f2232bd7c81e651dd949f39cc9e1.diff

[Lldb-commits] [lldb] 47d3c5f - [lldb] Cortex-M exception unwind API test cleanup

2025-10-09 Thread Jason Molenda via lldb-commits
Author: Jason Molenda Date: 2025-10-09T19:10:42-07:00 New Revision: 47d3c5f3cb159fd261ab5c3efa55afae246c5e0d URL: https://github.com/llvm/llvm-project/commit/47d3c5f3cb159fd261ab5c3efa55afae246c5e0d DIFF: https://github.com/llvm/llvm-project/commit/47d3c5f3cb159fd261ab5c3efa55afae246c5e0d.diff

[Lldb-commits] [lldb] [lldb][mcp] Get the running MCP server connection information (PR #162752)

2025-10-09 Thread Alexandre Perez via lldb-commits
https://github.com/aperez updated https://github.com/llvm/llvm-project/pull/162752 >From cf719895cfd59ce55d349d8f27ae60ad13a3f96d Mon Sep 17 00:00:00 2001 From: Alexandre Perez Date: Thu, 9 Oct 2025 16:23:46 -0700 Subject: [PATCH 1/2] [lldb][mcp] Add protocol-server subcommand to get the runni

[Lldb-commits] [lldb] update lldb-server platform help parsing (PR #162730)

2025-10-09 Thread Jonas Devlieghere via lldb-commits
JDevlieghere wrote: > The one question I had was regarding > > ``` > if (!LLDBServerUtilities::SetupLogging(log_file, log_channels, 0)) > return -1; > ``` > > and whether I should replace the -1 with `EXIT_FAILURE` Personally I would keep the current value, as it seems to have been done

[Lldb-commits] [lldb] [debugserver] Implement MultiMemRead packet (PR #162670)

2025-10-09 Thread Jason Molenda via lldb-commits
@@ -3160,6 +3165,140 @@ rnb_err_t RNBRemote::HandlePacket_m(const char *p) { return SendPacket(ostrm.str()); } +/// Returns true if `str` starts with `prefix`. +static bool starts_with(std::string_view str, std::string_view prefix) { + return str.size() >= prefix.size() &&

[Lldb-commits] [lldb] [gdbremote] Document MultiMemRead packet in protocol extensions (PR #162675)

2025-10-09 Thread Jason Molenda via lldb-commits
@@ -2530,3 +2530,41 @@ read packet: $e0030100#b9 **Priority to Implement:** Only required for Wasm support. Necessary to show variables. + +### MultiMemRead + +Read memory from multiple memory addresses. + +There are two arguments to the request: + +* `ranges`: a list of base-

[Lldb-commits] [lldb] 446d950 - [libc++] Use std::__{scope, exception}_guard throughout the code base (#161322)

2025-10-09 Thread via lldb-commits
Author: Nikolas Klauser Date: 2025-10-09T09:32:40+02:00 New Revision: 446d9505246227b514aa6359f17106e1a9c65897 URL: https://github.com/llvm/llvm-project/commit/446d9505246227b514aa6359f17106e1a9c65897 DIFF: https://github.com/llvm/llvm-project/commit/446d9505246227b514aa6359f17106e1a9c65897.dif