@@ -0,0 +1,293 @@
+//===-- RegisterContextUnifiedCore.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: Ap
https://github.com/ashgti updated
https://github.com/llvm/llvm-project/pull/145621
>From 954f107c85f766ee75370b349eb039819f456db5 Mon Sep 17 00:00:00 2001
From: John Harrison
Date: Wed, 25 Jun 2025 18:03:39 -0700
Subject: [PATCH] [lldb] Adding file and pipe support to
lldb_private::MainLoopWin
https://github.com/ashgti updated
https://github.com/llvm/llvm-project/pull/145621
>From 3db6bada43cb11c5fb9fe82634b939d6884eb074 Mon Sep 17 00:00:00 2001
From: John Harrison
Date: Wed, 25 Jun 2025 18:03:39 -0700
Subject: [PATCH] [lldb] Adding file and pipe support to
lldb_private::MainLoopWin
https://github.com/ashgti updated
https://github.com/llvm/llvm-project/pull/145621
>From 0427c242228ecb4034931ff77d16cedd4b1bacaf Mon Sep 17 00:00:00 2001
From: John Harrison
Date: Wed, 25 Jun 2025 18:03:39 -0700
Subject: [PATCH] [lldb] Adding file and pipe support to
lldb_private::MainLoopWin
https://github.com/JDevlieghere updated
https://github.com/llvm/llvm-project/pull/145823
>From 0c722624e71814eee9a4c493de0a75f7392cfebe Mon Sep 17 00:00:00 2001
From: Jonas Devlieghere
Date: Wed, 25 Jun 2025 17:18:22 -0700
Subject: [PATCH] [lldb] Correctly restore the cursor column after resizi
https://github.com/ashgti updated
https://github.com/llvm/llvm-project/pull/145621
>From 97b8cf6986847bb73fa9ceb6f0c182d8c79737d4 Mon Sep 17 00:00:00 2001
From: John Harrison
Date: Wed, 25 Jun 2025 18:03:39 -0700
Subject: [PATCH] [lldb] Adding file and pipe support to
lldb_private::MainLoopWin
jimingham wrote:
We don't have SB API's currently for the stop-hooks yet, so we're free to
invent them.
But my notion was something like this:
class ScriptedHookDefinition {
void SetScriptingLanguage(lldb::ScriptLanguage lang);
void SetScriptClassName(const char *script_class_name);
voi
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: None (tedwoodward)
Changes
RISC-V supports proprietary extensions, where the TD files don't know about
certain instructions, and the disassembler can't disassemble them. Internal
users want to be able to disassemble these instructions in L
https://github.com/tedwoodward created
https://github.com/llvm/llvm-project/pull/145793
RISC-V supports proprietary extensions, where the TD files don't know about
certain instructions, and the disassembler can't disassemble them. Internal
users want to be able to disassemble these instruction
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
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Pavel Labath (labath)
Changes
It's not necessary on posix platforms as of #126935 and it's ignored on
windows as of #138896. For both platforms, we have a better way of
inheriting FDs/HANDLEs.
---
Patch is 22.46 KiB, truncated to 20.00 K
@@ -1998,6 +1998,23 @@ threads (live system debug) / cores (JTAG) in your
program have
stopped and allows LLDB to display and control your program
correctly.
+## qWatchpointSupportInfo
+
+Get the number of hardware watchpoints available on the remote target.
+
+```
+send pack
https://github.com/JDevlieghere closed
https://github.com/llvm/llvm-project/pull/145616
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: Jonas Devlieghere
Date: 2025-06-25T13:46:33-07:00
New Revision: e8abdfc88ffed632750fe0fd7deafb577e902bd6
URL:
https://github.com/llvm/llvm-project/commit/e8abdfc88ffed632750fe0fd7deafb577e902bd6
DIFF:
https://github.com/llvm/llvm-project/commit/e8abdfc88ffed632750fe0fd7deafb577e902bd6.d
@@ -552,10 +552,9 @@ bool ClassDescriptorV2::Describe(
} else {
std::optional base_method_list =
GetMethodList(process, class_ro->m_baseMethods_ptr);
- if (!base_method_list)
-return false;
- if (!ProcessMethodList(instance_method_func, *ba
https://github.com/charles-zablit created
https://github.com/llvm/llvm-project/pull/145744
This patch is part of an effort to remove the `ResolveSDKPathFromDebugInfo`
method, and more specifically the variant which takes a `Module` as argument.
This PR should be merged after https://github.com
igorkudrin wrote:
> But we do support scalar persistent expression results in core files
> currently:
>
> (lldb) expr int $my_int = 5
(lldb) expr $my_int * 2
(int) $0 = 10
>
> and that should still work after your change. I see no reason why it
> wouldn't, but it would be good to add some use
royitaqi wrote:
Hi @JDevlieghere and @clayborg ,
Could you kindly review when you have the chance? I think the PR is close to
finish, except the "how to make sure hello.cpp builds in Windows" question.
TL;DR updates:
1. Called `xcrun simctl spawn -s ` to launch
`a.out` and verified that its
royitaqi wrote:
> The Windows-conditional parts will be dead code.
Hi @JDevlieghere, I have moved the Windows related code. Should be good to go
(re-running tests on the side).
FWIW, my bad, I overlooked the `@skipUnlessDarwin` in the test.
https://github.com/llvm/llvm-project/pull/139174
@@ -53,6 +54,29 @@ bool DWARFExpressionList::ContainsAddress(lldb::addr_t
func_load_addr,
return GetExpressionAtAddress(func_load_addr, addr) != nullptr;
}
+llvm::Expected
+DWARFExpressionList::GetExpressionEntryAtAddress(lldb::addr_t func_load_addr,
+
https://github.com/royitaqi updated
https://github.com/llvm/llvm-project/pull/139174
>From d98210b81f7b49f5384e968b1a18e00e432aaf2c Mon Sep 17 00:00:00 2001
From: Roy Shi
Date: Thu, 8 May 2025 16:21:53 -0700
Subject: [PATCH 1/7] Fix macOS FindProcessImpl()
---
lldb/source/Host/macosx/objcxx/H
athierry-oct wrote:
Thanks for the feedback and the good discussion!
> I think I understand most of what you're saying, and I think the proposed
> solution may solve the problem you have in mind -- but I don't think it can
> solve the problem I have in mind (and I'm not sure which of those is
https://github.com/slydiman approved this pull request.
LGTM.
https://github.com/llvm/llvm-project/pull/145706
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/Jlalond closed
https://github.com/llvm/llvm-project/pull/144800
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
cmtice wrote:
I've tried to move most of the tests around as you suggested. I have kept the
dil smart pointer tests separate from the existing stl tests (although I moved
them over to the data-formattter-stl/libcxx tree). They do test functionality
that the
@@ -421,6 +422,10 @@ class AppleObjCRuntimeV2 : public AppleObjCRuntime {
lldb::addr_t GetISAHashTablePointer();
+ using ValueObjectSet = llvm::SmallSet;
adrian-prantl wrote:
`llvm::SmallPtrSet`?
https://github.com/llvm/llvm-project/pull/145396
_
https://github.com/JDevlieghere approved this pull request.
https://github.com/llvm/llvm-project/pull/145021
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
JDevlieghere wrote:
+1 on testing and Pavel's suggestion to use assembly/yaml for it so it can run
everywhere.
https://github.com/llvm/llvm-project/pull/145645
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/ma
https://github.com/DavidSpickett created
https://github.com/llvm/llvm-project/pull/145709
None
>From f6411a087299e570db9ef4a460c2ed2048160c39 Mon Sep 17 00:00:00 2001
From: David Spickett
Date: Wed, 25 Jun 2025 14:12:37 +
Subject: [PATCH] [lldb][docs] Document qWatchpointSupportInfo
---
https://github.com/ashgti approved this pull request.
LGTM!
https://github.com/llvm/llvm-project/pull/145616
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: David Spickett
Date: 2025-06-25T10:31:38+01:00
New Revision: 3a3d1bf4a39bbbe1cfdc56ab9eeeb13cb438348a
URL:
https://github.com/llvm/llvm-project/commit/3a3d1bf4a39bbbe1cfdc56ab9eeeb13cb438348a
DIFF:
https://github.com/llvm/llvm-project/commit/3a3d1bf4a39bbbe1cfdc56ab9eeeb13cb438348a.diff
https://github.com/adrian-prantl closed
https://github.com/llvm/llvm-project/pull/144913
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/ashgti edited
https://github.com/llvm/llvm-project/pull/145616
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -1130,14 +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) {
-
andrurogerz wrote:
> do you need us to merge this for you?
@charles-zablit yes, I would appreciate it! Thank you!
https://github.com/llvm/llvm-project/pull/145606
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin
https://github.com/Nerixyz updated
https://github.com/llvm/llvm-project/pull/145175
>From a7e33a16dd280250df2ff2d01648bd4d5bfe000b Mon Sep 17 00:00:00 2001
From: Nerixyz
Date: Sat, 21 Jun 2025 17:36:58 +0200
Subject: [PATCH] [LLDB] Warn about truncated DWARF section names on Windows
---
.../O
https://github.com/DavidSpickett closed
https://github.com/llvm/llvm-project/pull/145338
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
jimingham wrote:
It isn't terribly important. We are switching to returning expected so if you
think that looks nicer, feel free to do that. But otherwise, this is fine as
it.
https://github.com/llvm/llvm-project/pull/145599
___
lldb-commits maili
chelcassanova wrote:
> I'd be alright with it going in a follow-up. That being said, it would be
> easy to kick this can down the road. Let's figure out a way to make this
> happen without blocking this PR.
I think that this should be addressed in a follow-up PR, mainly because I think
change
https://github.com/cs01 edited https://github.com/llvm/llvm-project/pull/145382
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -0,0 +1,24 @@
+#include
+
+int main(int argc, char **argv) {
+
+ struct NodeS {
+std::shared_ptr next;
+int value;
+ };
+ auto ptr_node = std::shared_ptr(new NodeS{nullptr, 2});
+ ptr_node = std::shared_ptr(new NodeS{std::move(ptr_node), 1});
+
+ std::shared_ptr p
charles-zablit wrote:
Confirmed that this builds fine with the changes 👍
https://github.com/llvm/llvm-project/pull/145606
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/labath edited
https://github.com/llvm/llvm-project/pull/143786
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/qxy11 updated
https://github.com/llvm/llvm-project/pull/145436
>From 1323a237d4cbe554e7f373316a041ca172168afe Mon Sep 17 00:00:00 2001
From: Janet Yang
Date: Mon, 23 Jun 2025 14:25:12 -0700
Subject: [PATCH 1/4] Disable "transcript" in "statistics dump" by default
---
lldb/i
https://github.com/cs01 edited https://github.com/llvm/llvm-project/pull/145382
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/DavidSpickett commented:
Also wondering about testing this. What tools can produce DWARF64?
Have you ran `check-lldb` with DWARF64 enabled and what results did you get?
I doubt anyone has spare capacity to add a whole new bot for this, but perhaps
we would be ok with a few n
https://github.com/charles-zablit updated
https://github.com/llvm/llvm-project/pull/145744
>From e28a9e6249077c9ffca878cbf4c933b6f4f9eab8 Mon Sep 17 00:00:00 2001
From: Charles Zablit
Date: Thu, 19 Jun 2025 16:17:33 +0100
Subject: [PATCH 1/4] [lldb][darwin] force BuiltinHeadersInSystemModules t
https://github.com/qxy11 updated
https://github.com/llvm/llvm-project/pull/145436
>From b7889c1f7b0be25f71c6c893bfabe0ebb26f9f52 Mon Sep 17 00:00:00 2001
From: Janet Yang
Date: Mon, 23 Jun 2025 14:25:12 -0700
Subject: [PATCH 1/5] Disable "transcript" in "statistics dump" by default
---
lldb/i
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/commands/statistics/basic/TestStats.py
``
View th
qxy11 wrote:
Added a unit test around the warning message as well. I'll probably need some
help with landing this.
https://github.com/llvm/llvm-project/pull/145436
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bi
@@ -321,7 +321,8 @@ IRMemoryMap::Allocation::Allocation(lldb::addr_t
process_alloc,
lldb::addr_t IRMemoryMap::Malloc(size_t size, uint8_t alignment,
uint32_t permissions, AllocationPolicy policy,
- bool zero_mem
https://github.com/charles-zablit closed
https://github.com/llvm/llvm-project/pull/145606
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: Andrew Rogers
Date: 2025-06-25T19:30:50+01:00
New Revision: 369cbcc1a21ab07ea6ca142a3b87c75d0a1b2014
URL:
https://github.com/llvm/llvm-project/commit/369cbcc1a21ab07ea6ca142a3b87c75d0a1b2014
DIFF:
https://github.com/llvm/llvm-project/commit/369cbcc1a21ab07ea6ca142a3b87c75d0a1b2014.diff
https://github.com/bulbazord approved this pull request.
https://github.com/llvm/llvm-project/pull/145606
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
jimingham wrote:
That seems like a sensible strategy.
I made one trivial comment about the ordering of parameters, but otherwise this
LGTM.
https://github.com/llvm/llvm-project/pull/145599
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
htt
Author: Charles Zablit
Date: 2025-06-25T11:18:07-07:00
New Revision: 487581b826a5339410aaf306eafedc7b806b25e3
URL:
https://github.com/llvm/llvm-project/commit/487581b826a5339410aaf306eafedc7b806b25e3
DIFF:
https://github.com/llvm/llvm-project/commit/487581b826a5339410aaf306eafedc7b806b25e3.diff
@@ -321,7 +321,8 @@ IRMemoryMap::Allocation::Allocation(lldb::addr_t
process_alloc,
lldb::addr_t IRMemoryMap::Malloc(size_t size, uint8_t alignment,
uint32_t permissions, AllocationPolicy policy,
- bool zero_mem
charles-zablit wrote:
@andrurogerz do you need us to merge this for you?
https://github.com/llvm/llvm-project/pull/145606
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -1130,14 +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) {
-
https://github.com/charles-zablit updated
https://github.com/llvm/llvm-project/pull/145744
>From e28a9e6249077c9ffca878cbf4c933b6f4f9eab8 Mon Sep 17 00:00:00 2001
From: Charles Zablit
Date: Thu, 19 Jun 2025 16:17:33 +0100
Subject: [PATCH 1/5] [lldb][darwin] force BuiltinHeadersInSystemModules t
https://github.com/JDevlieghere updated
https://github.com/llvm/llvm-project/pull/145616
>From 5ed60a3aa5022694a593e2885ad6e563df6ffa37 Mon Sep 17 00:00:00 2001
From: Jonas Devlieghere
Date: Tue, 24 Jun 2025 16:22:46 -0700
Subject: [PATCH 1/4] [lldb] Make MCP server instance global
Rather than
@@ -66,16 +87,62 @@ LLDBCommandTool::Call(const llvm::json::Value &args) {
output += err_str;
}
- mcp::protocol::TextResult text_result;
- text_result.content.emplace_back(mcp::protocol::TextContent{{output}});
- text_result.isError = !result.Succeeded();
- return te
https://github.com/adrian-prantl approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/145744
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -0,0 +1,22 @@
+include(CheckCXXCompilerFlag)
chelcassanova wrote:
Looking at this again, I realize that on its own this is probably unnecessary,
but this include is actually here for a check of a compiler flag:
```
check_cxx_compiler_flag("-Wno-gnu-zero-vari
https://github.com/adrian-prantl approved this pull request.
This LGTM. There is no good reason to support debugging with modules against an
older SDK, since you can't actually install one easily either.
https://github.com/llvm/llvm-project/pull/144913
__
@@ -66,16 +87,62 @@ LLDBCommandTool::Call(const llvm::json::Value &args) {
output += err_str;
}
- mcp::protocol::TextResult text_result;
- text_result.content.emplace_back(mcp::protocol::TextContent{{output}});
- text_result.isError = !result.Succeeded();
- return te
@@ -0,0 +1,22 @@
+include(CheckCXXCompilerFlag)
bulbazord wrote:
If you remove it, does it add new compiler warnings? If not, let's take it out.
https://github.com/llvm/llvm-project/pull/138031
___
lldb-commits mailing
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Charles Zablit (charles-zablit)
Changes
This patch is part of an effort to remove the `ResolveSDKPathFromDebugInfo`
method, and more specifically the variant which takes a `Module` as argument.
This PR should be merged after https://github
https://github.com/JDevlieghere updated
https://github.com/llvm/llvm-project/pull/145616
>From 5ed60a3aa5022694a593e2885ad6e563df6ffa37 Mon Sep 17 00:00:00 2001
From: Jonas Devlieghere
Date: Tue, 24 Jun 2025 16:22:46 -0700
Subject: [PATCH 1/3] [lldb] Make MCP server instance global
Rather than
da-viper wrote:
> if we don't have a symbol, it would be nice to have a source reference for a
> given range (say the next 64 or 128 instructions)
I was able to find a scenario there can have an executable address without an
associated symbol. from one of the test cases.
https://github.com/
https://github.com/JDevlieghere edited
https://github.com/llvm/llvm-project/pull/145616
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -58,6 +59,19 @@ class DWARFExpressionList {
}
lldb::addr_t GetFuncFileAddress() { return m_func_file_addr; }
+
+ /// Represents an entry in the DWARFExpressionList with all needed metadata.
+ struct DWARFExpressionEntry {
+AddressRange file_range; /// Represents
https://github.com/JDevlieghere updated
https://github.com/llvm/llvm-project/pull/145616
>From 5ed60a3aa5022694a593e2885ad6e563df6ffa37 Mon Sep 17 00:00:00 2001
From: Jonas Devlieghere
Date: Tue, 24 Jun 2025 16:22:46 -0700
Subject: [PATCH 1/2] [lldb] Make MCP server instance global
Rather than
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: None (eleviant)
Changes
This fixes issues found in e066f35c6981c720e3a7e5883efc40c861b3b7, which was
later reverted. The problem was with "k" message which was sent with
sync_on_timeout flag set to true, so lldb was waiting for response, w
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: David Spickett (DavidSpickett)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/145709.diff
1 Files Affected:
- (modified) lldb/docs/resources/lldbgdbremote.md (+17)
``diff
diff --git a/lldb/docs/resources/lld
JDevlieghere wrote:
> I think the PR is close to finish, except the "how to make sure hello.cpp
> builds in Windows" question.
Does this test need to build on Windows? That test should only run on Darwin,
so if it's properly skipped, we shouldn't even try to build the inferior. The
Windows-co
@@ -72,10 +80,46 @@ LLDBCommandTool::Call(const llvm::json::Value &args) {
return text_result;
}
-std::optional LLDBCommandTool::GetSchema() const {
+std::optional CommandTool::GetSchema() const {
+ llvm::json::Object id_type{{"type", "number"}};
llvm::json::Object str_t
@@ -37,20 +37,26 @@ class Tool {
std::string m_description;
};
-class LLDBCommandTool : public mcp::Tool {
+class CommandTool : public mcp::Tool {
public:
- LLDBCommandTool(std::string name, std::string description,
- Debugger &debugger);
- ~LLDBCommandTo
@@ -72,10 +80,46 @@ LLDBCommandTool::Call(const llvm::json::Value &args) {
return text_result;
}
-std::optional LLDBCommandTool::GetSchema() const {
+std::optional CommandTool::GetSchema() const {
+ llvm::json::Object id_type{{"type", "number"}};
llvm::json::Object str_t
DavidSpickett wrote:
Clang only supports DWARF64 for ELF:
```
def gdwarf64 : Flag<["-"], "gdwarf64">, Group,
Visibility<[ClangOption, CC1Option, CC1AsOption]>,
HelpText<"Enables DWARF64 format for ELF binaries, if debug information
emission is enabled.">,
MarshallingInfoFlag>;
```
So the e
@@ -72,10 +80,46 @@ LLDBCommandTool::Call(const llvm::json::Value &args) {
return text_result;
}
-std::optional LLDBCommandTool::GetSchema() const {
+std::optional CommandTool::GetSchema() const {
+ llvm::json::Object id_type{{"type", "number"}};
llvm::json::Object str_t
@@ -37,20 +37,26 @@ class Tool {
std::string m_description;
};
-class LLDBCommandTool : public mcp::Tool {
+class CommandTool : public mcp::Tool {
public:
- LLDBCommandTool(std::string name, std::string description,
- Debugger &debugger);
- ~LLDBCommandTo
@@ -72,10 +80,46 @@ LLDBCommandTool::Call(const llvm::json::Value &args) {
return text_result;
}
-std::optional LLDBCommandTool::GetSchema() const {
+std::optional CommandTool::GetSchema() const {
+ llvm::json::Object id_type{{"type", "number"}};
llvm::json::Object str_t
@@ -7,20 +7,23 @@
//===--===//
#include "Tool.h"
+#include "lldb/Core/Module.h"
#include "lldb/Interpreter/CommandInterpreter.h"
#include "lldb/Interpreter/CommandReturnObject.h"
using namespace lldb_pri
@@ -72,10 +80,46 @@ LLDBCommandTool::Call(const llvm::json::Value &args) {
return text_result;
}
-std::optional LLDBCommandTool::GetSchema() const {
+std::optional CommandTool::GetSchema() const {
+ llvm::json::Object id_type{{"type", "number"}};
llvm::json::Object str_t
https://github.com/da-viper approved this pull request.
https://github.com/llvm/llvm-project/pull/145231
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
DavidSpickett wrote:
As far as I can tell this is only used as a binary to tell if we have any
hardware breakpoints, or none. Even though the packet returns the exact number.
https://github.com/llvm/llvm-project/pull/145709
___
lldb-commits mailing li
https://github.com/labath created
https://github.com/llvm/llvm-project/pull/145706
.. from the guts of GDBRemoteCommunication to ~top level.
This is motivated by #131519 and by the fact that's impossible to guess whether
the author of a symlink intended it to be a "convenience shortcut" -- mea
@@ -121,8 +124,12 @@ bool DWARFFormValue::ExtractValue(const DWARFDataExtractor
&data,
assert(m_unit);
if (m_unit->GetVersion() <= 2)
ref_addr_size = m_unit->GetAddressByteSize();
- else
-ref_addr_size = 4;
+ else {
+if (m_unit->Ge
labath wrote:
@yuvald-sweet-security
https://github.com/llvm/llvm-project/pull/145706
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Pavel Labath (labath)
Changes
.. from the guts of GDBRemoteCommunication to ~top level.
This is motivated by #131519 and by the fact that's impossible to guess
whether the author of a symlink intended it to be a "convenience shortcut" --
https://github.com/jasonmolenda updated
https://github.com/llvm/llvm-project/pull/144627
>From 92348b28fb02901e9437b92c1ddf8cfed31c Mon Sep 17 00:00:00 2001
From: Jason Molenda
Date: Tue, 17 Jun 2025 18:57:11 -0700
Subject: [PATCH 01/10] [lldb][Mach-O] Allow "process metadata" LC_NOTE to
s
@@ -1084,55 +1052,30 @@ Status GDBRemoteCommunication::StartDebugserverProcess(
if (socket_pipe.CanWrite())
socket_pipe.CloseWriteFileDescriptor();
- if (socket_pipe.CanRead()) {
-// Read port from pipe with 10 second timeout.
-std::string port_str;
-while (
@@ -121,8 +124,12 @@ bool DWARFFormValue::ExtractValue(const DWARFDataExtractor
&data,
assert(m_unit);
if (m_unit->GetVersion() <= 2)
ref_addr_size = m_unit->GetAddressByteSize();
- else
-ref_addr_size = 4;
+ else {
+if (m_unit->Ge
https://github.com/charles-zablit approved this pull request.
https://github.com/llvm/llvm-project/pull/145606
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
labath wrote:
Also note that there are remnants of our previous attempt at dwarf64 support
(which was deleted many years ago) lying around the DWARFDataExtractor class.
However, since this isn't how *llvm*'s DWARFDataExtractor works, I think we
should not be using that (instead, we should dele
HemangGadhavi wrote:
> Also wondering about testing this. What tools can produce DWARF64?
>
@DavidSpickett answer to your question, we can produce `DWARF64` by enabling
-`gdwarf64` clang compilation flag. But that could be different for different
compiler
> Run `check-lldb` with DWARF64 ena
DavidSpickett wrote:
>Thats what I need suggestions and inputs what changes we can accommodate to
>better support the DWARF64.
Start by looking at the main differences DWARF64 has. Write DWARF64 specific
tests that cover those corner cases e.g. that some symbol can now be described
as a great
https://github.com/labath updated
https://github.com/llvm/llvm-project/pull/145516
>From 598b2f3f89fd3b09ab40f1ca0417e7158bae3488 Mon Sep 17 00:00:00 2001
From: Pavel Labath
Date: Sun, 20 Oct 2024 00:18:56 +0200
Subject: [PATCH] [lldb] Remove child_process_inherit argument from Pipe
It's not n
https://github.com/eleviant created
https://github.com/llvm/llvm-project/pull/145675
This fixes issues found in e066f35c6981c720e3a7e5883efc40c861b3b7, which was
later reverted. The problem was with "k" message which was sent with
sync_on_timeout flag set to true, so lldb was waiting for respo
1 - 100 of 119 matches
Mail list logo
