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

2025-10-10 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/9] [lldb][Expression] Emit a 'Note' diagnistc that indicates

[Lldb-commits] [lldb] [LLDB] Fix `GetIndexOfChildMemberWithName` to handle anonymous struct in base classes (PR #158256)

2025-10-10 Thread Michael Buch via lldb-commits
https://github.com/Michael137 commented: I have the suspicion that this could be done simpler. `CXXRecordDecl::lookupInBases` is already perfectly capable of finding field in anonymous structures in bases. That's why the expression evaluator is able to find them just fine. I think the actual

[Lldb-commits] [lldb] [LLDB] Fix `GetIndexOfChildMemberWithName` to handle anonymous struct in base classes (PR #158256)

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

[Lldb-commits] [lldb] [LLDB] Fix `GetIndexOfChildMemberWithName` to handle anonymous struct in base classes (PR #158256)

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

[Lldb-commits] [lldb] [LLDB, x86, FreeBSD] Fix Architecture parsing by reading the ELF header. (PR #162811)

2025-10-10 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett approved this pull request. LGTM, I see that NetBSD and Linux do this too. Though I doubt whether debugging works on those platforms either, and I don't think we have upstream tests for it so I won't demand them here. This change applies to AArch64 as well, rig

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

2025-10-10 Thread Martin Storsjö via lldb-commits
@@ -167,6 +167,12 @@ function(add_lldb_executable name) ) target_link_libraries(${name} PRIVATE ${ARG_LINK_LIBS}) + if(WIN32) +list(FIND ARG_LINK_LIBS liblldb LIBLLDB_INDEX) +if(NOT LIBLLDB_INDEX EQUAL -1) + target_link_options(${name} PRIVATE "/DELAYLOAD:$.

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

2025-10-10 Thread Martin Storsjö via lldb-commits
mstorsjo wrote: > This, combined with `liblldb.dll` being delay loaded, allows to package > `python.dll` with the `llvm` installer. Just for reference - for llvm-mingw, we've been bundling a python install in our toolchain as well, but we've solved this slightly differently. We've just copied

[Lldb-commits] [lldb] [lldb] [cmake] Fix delayloading liblldb.dll in mingw builds (PR #162831)

2025-10-10 Thread via lldb-commits
llvmbot wrote: @llvm/pr-subscribers-lldb Author: Martin Storsjö (mstorsjo) Changes ec28b95b7491bc2fbb6ec66cdbfd939e71255c42 made liblldb delayloaded, but the supplied command line parameter is only valid for MSVC style builds. For mingw builds using LLD, we can easily pass a similar opti

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

2025-10-10 Thread Sergei Druzhkov via lldb-commits
https://github.com/DrSergei updated https://github.com/llvm/llvm-project/pull/162517 >From 3d548533d38277eb2ba59c810f7634909bb7adbd Mon Sep 17 00:00:00 2001 From: Druzhkov Sergei Date: Wed, 8 Oct 2025 11:38:55 +0300 Subject: [PATCH 1/2] [lldb-dap] Allow empty memory reference in disassemble ar

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

2025-10-10 Thread David Spickett via lldb-commits
@@ -789,6 +789,10 @@ def canRunLibcxxTests(): return True, "libc++ always present" if platform == "linux": +if not configuration.libcxx_include_dir or not configuration.libcxx_library_dir: +return False, "API tests require a locally built libc+

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

2025-10-10 Thread David Spickett via lldb-commits
DavidSpickett wrote: > --category libc++ It shows 68 skipped, did you paste the wrong output? https://github.com/llvm/llvm-project/pull/162657 ___ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/ll

[Lldb-commits] [lldb] [LLDB, x86, FreeBSD] Fix Architecture parsing by reading the ELF header. (PR #162811)

2025-10-10 Thread via lldb-commits
aokblast wrote: > I can merge this if you need me to. Sure. Thanks for your help! https://github.com/llvm/llvm-project/pull/162811 ___ 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-10 Thread David Spickett via lldb-commits
DavidSpickett wrote: > As in, I wanted to show that the simluators aren't part of the libc++ > category :p Ok I see now. You enabled only that category, to show that no simulator test falls into it. Rather than disabling the category, to show that all the simulator tests ran (because how woul

[Lldb-commits] [libunwind] [lldb] [llvm] [openmp] Fix typos: 'auxilliary' to 'auxiliary' and 'unit64_t' to 'uint64_t' (PR #161955)

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

[Lldb-commits] [lldb] [lldb] Add a missing include. NFC. (PR #162809)

2025-10-10 Thread Martin Storsjö via lldb-commits
https://github.com/mstorsjo closed https://github.com/llvm/llvm-project/pull/162809 ___ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [LLDB, x86, FreeBSD] Fix Architecture parsing by reading the ELF header. (PR #162811)

2025-10-10 Thread David Spickett via lldb-commits
DavidSpickett wrote: My impression is that 32-bit executables are much more common on x86. To the point where there are standard flags like `-m32`, which we do not have for AArch64/AArch32. https://github.com/llvm/llvm-project/pull/162811 ___ lldb-co

[Lldb-commits] [libunwind] [lldb] [llvm] [openmp] Fix typos: 'auxilliary' to 'auxiliary' and 'unit64_t' to 'uint64_t' (PR #161955)

2025-10-10 Thread via lldb-commits
smallzhong wrote: Sorry, I accidentally created a pull request for all commits. I'll close this one and create a new one. https://github.com/llvm/llvm-project/pull/161955 ___ lldb-commits mailing list [email protected] https://lists.llvm.org

[Lldb-commits] [lldb] [lldb] [cmake] Fix delayloading liblldb.dll in mingw builds (PR #162831)

2025-10-10 Thread Martin Storsjö via lldb-commits
https://github.com/mstorsjo closed https://github.com/llvm/llvm-project/pull/162831 ___ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Introduce ScriptedFrameProvider for real threads (PR #161870)

2025-10-10 Thread Felipe de Azevedo Piovezan via lldb-commits
felipepiovezan wrote: > the need for coroutine stack traces. I need to splice in additional stack > frames in the middle of real, physical stack frames. @vogelsgesang Have you considered writing a custom unwinder for this? If you want stack frames from which you can perform step operations, I

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

2025-10-10 Thread David Spickett via lldb-commits
@@ -0,0 +1,72 @@ +""" +Tests the exit code/description coming from the debugserver. +""" + +import lldb +from lldbsuite.test.decorators import * +from lldbsuite.test.lldbtest import * +from lldbsuite.test import lldbutil + + +@skipUnlessDarwin +@skipIfOutOfTreeDebugserver +class T

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

2025-10-10 Thread David Spickett via lldb-commits
@@ -0,0 +1,72 @@ +""" +Tests the exit code/description coming from the debugserver. DavidSpickett wrote: "...when handling MultiMemRead packets." https://github.com/llvm/llvm-project/pull/162670 ___ lldb-commits mailin

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

2025-10-10 Thread Alexandre Perez via lldb-commits
https://github.com/aperez closed https://github.com/llvm/llvm-project/pull/162752 ___ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] Fix empty register set when trying to get size of register (PR #162890)

2025-10-10 Thread via lldb-commits
https://github.com/aokblast updated https://github.com/llvm/llvm-project/pull/162890 >From 02a3bd3f786925b22f29a8c9f7c9a1db2c9dbec4 Mon Sep 17 00:00:00 2001 From: ShengYi Hung Date: Sat, 11 Oct 2025 01:07:36 +0800 Subject: [PATCH] [LLDB, FreeBSD, x86] Fix empty register set when trying to get

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

2025-10-10 Thread via lldb-commits
Author: Michael Buch Date: 2025-10-10T19:23:02+01:00 New Revision: e3620fe0685c656915977d55f822a82090041965 URL: https://github.com/llvm/llvm-project/commit/e3620fe0685c656915977d55f822a82090041965 DIFF: https://github.com/llvm/llvm-project/commit/e3620fe0685c656915977d55f822a82090041965.diff

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

2025-10-10 Thread Adrian Prantl via lldb-commits
https://github.com/adrian-prantl approved this pull request. 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][test] Don't run libc++ API tests without a locally built libc++ (PR #162657)

2025-10-10 Thread David Spickett via lldb-commits
@@ -789,6 +789,10 @@ def canRunLibcxxTests(): return True, "libc++ always present" if platform == "linux": +if not configuration.libcxx_include_dir or not configuration.libcxx_library_dir: +return False, "API tests require a locally built libc+

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

2025-10-10 Thread Adrian Prantl via lldb-commits
@@ -426,6 +433,47 @@ SBError Driver::ProcessArgs(const opt::InputArgList &args, bool &exiting) { return error; } +#ifdef _WIN32 +// Returns the full path to the lldb.exe executable +inline std::wstring GetPathToExecutableW() { + // Iterate until we reach the Windows max pa

[Lldb-commits] [lldb] [LLDB][NativePDB] Use typedef compiler type for typedef types (PR #156250)

2025-10-10 Thread Michael Buch via lldb-commits
Michael137 wrote: > Ping - I'd like to use the added test from this PR as the "current state" and > fix the missing things from above in other PRs. Will some time today. Sorry for the delay! https://github.com/llvm/llvm-project/pull/156250 ___ lldb-

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

2025-10-10 Thread Walter Erquinigo via lldb-commits
@@ -9,6 +10,16 @@ import { LogFilePathProvider, LogType } from "./logging"; const exec = util.promisify(child_process.execFile); +/** walter-erquinigo wrote: better use this package https://www.npmjs.com/package/untildify there are some interesting cases rel