AlexeyMerzlyakov wrote:
@JDevlieghere, to get coredump file, I've used
[make-core.sh](https://github.com/llvm/llvm-project/blob/main/lldb/test/API/functionalities/postmortem/elf-core/make-core.sh)
script on the target H/W. Generated file should be minimal, so you can set
`coredump_filter` (htt
dyung wrote:
Maybe I'm missing something, but was either this change or the original one
approved for submit by someone? I don't see any review/approval in this commit,
and the original PR doesn't seem to have been approved.
https://github.com/llvm/llvm-project/pull/136121
https://github.com/ashgti approved this pull request.
https://github.com/llvm/llvm-project/pull/136486
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
DavidSpickett wrote:
Likely won't find the fix today, but I suspect that the Windows process class
just doesn't know that it doesn't support reverse execution:
```
(lldb) process continue --reverse
Process 3108 resuming
(lldb) process status
Process 5360 stopped
```
The process being stopped is
@@ -0,0 +1,64 @@
+"""
+Test lldb-dap stack trace containing x86 assembly
+"""
+
+import lldbdap_testcase
+from lldbsuite.test.decorators import skipUnlessArch, skipUnlessPlatform
+from lldbsuite.test.lldbtest import line_number
+
+
+class TestDAP_stacktrace_x86(lldbdap_testcase.DA
luporl wrote:
It seems this broke lldb-aarch64-windows bot:
https://lab.llvm.org/buildbot/#/builders/141/builds/8149
https://github.com/llvm/llvm-project/pull/132783
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff HEAD~1 HEAD --extensions h,cpp --
lldb/include/lldb/DataFormatters/TypeSynthetic.h
l
labath wrote:
We already have one piece of "status" parsing code in
`source/Host/linux/Host.cpp`. I think it'd be better to reuse that one. I'm
slightly torn as to whether reuse Host::GetProcessInfo for this (and add a new
field to ProcessInstanceInfo -- or possibly expand on IsZombie), or whe
Author: David Spickett
Date: 2025-04-24T12:16:29Z
New Revision: bcdafc107e1fb68893036cfba14e761c7b37b0e9
URL:
https://github.com/llvm/llvm-project/commit/bcdafc107e1fb68893036cfba14e761c7b37b0e9
DIFF:
https://github.com/llvm/llvm-project/commit/bcdafc107e1fb68893036cfba14e761c7b37b0e9.diff
LOG
https://github.com/oontvoo updated
https://github.com/llvm/llvm-project/pull/129728
>From 21103adacdf9c08cee4065f8a6b90ff812fefbb3 Mon Sep 17 00:00:00 2001
From: Vy Nguyen
Date: Tue, 4 Mar 2025 11:01:46 -0500
Subject: [PATCH 01/23] [LLDB][Telemetry] Collect telemetry from client when
allowed.
https://github.com/hapee updated
https://github.com/llvm/llvm-project/pull/136682
>From 2b8cce2ca76287d4620801878079f5dad453cc9a Mon Sep 17 00:00:00 2001
From: hapee <623151...@qq.com>
Date: Tue, 22 Apr 2025 11:34:38 +0800
Subject: [PATCH] [lldb] Fix crash after second run when set a pre
https://github.com/hapee updated
https://github.com/llvm/llvm-project/pull/136682
>From 563cbddfe9b7da394c818a03e05924c0f7d39d5f Mon Sep 17 00:00:00 2001
From: hapee <623151...@qq.com>
Date: Tue, 22 Apr 2025 11:34:38 +0800
Subject: [PATCH 1/3] [lldb] Fix crash after second run when set a
@@ -0,0 +1,64 @@
+"""
+Test lldb-dap stack trace containing x86 assembly
+"""
+
+import lldbdap_testcase
+from lldbsuite.test.decorators import skipUnlessArch, skipUnlessPlatform
+from lldbsuite.test.lldbtest import line_number
+
+
+class TestDAP_stacktrace_x86(lldbdap_testcase.DA
https://github.com/AaronBallman commented:
I think the changes LG, but please wait for our CMake experts (@petrhosek and
@Ericson2314) to review before landing.
https://github.com/llvm/llvm-project/pull/136843
___
lldb-commits mailing list
lldb-commit
@@ -48,14 +48,6 @@ if (LLVM_ENABLE_DOXYGEN)
set(bolt_doxygen_qhp_cust_filter_attrs "")
endif()
- option(LLVM_DOXYGEN_SVG
AaronBallman wrote:
Awesome, thanks!
https://github.com/llvm/llvm-project/pull/136843
@@ -1202,13 +1225,16 @@ lldb_private::formatters::Foundation1100::
m_data_64 = nullptr;
}
-size_t
-lldb_private::formatters::Foundation1100::
- NSDictionaryMSyntheticFrontEnd::GetIndexOfChildWithName(ConstString name) {
+llvm::Expected lldb_private::formatters::Foundation11
labath wrote:
> The problem I encountered is that Dereference has 2 separate checks: the
> first one checks if the type is a pointer or a reference, the second is done
> if the first one succeeds and checks if a child with a type is actually found
> by GetChildCompilerTypeAtIndex
Why does it
https://github.com/labath created
https://github.com/llvm/llvm-project/pull/137155
AddressFunctionScope was always returning the first address range of the
function (assuming it was the only one). This doesn't work for
RegisterContextUnwind (it's only caller), when the function doesn't
start at
@@ -28,7 +28,7 @@ class GenericBitsetFrontEnd : public
SyntheticChildrenFrontEnd {
GenericBitsetFrontEnd(ValueObject &valobj, StdLib stdlib);
- size_t GetIndexOfChildWithName(ConstString name) override {
+ llvm::Expected GetIndexOfChildWithName(ConstString name) override
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Pavel Labath (labath)
Changes
AddressFunctionScope was always returning the first address range of the
function (assuming it was the only one). This doesn't work for
RegisterContextUnwind (it's only caller), when the function doesn't
start a
https://github.com/charles-zablit edited
https://github.com/llvm/llvm-project/pull/136693
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/labath approved this pull request.
https://github.com/llvm/llvm-project/pull/134354
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -319,40 +319,49 @@
ValueObjectSynthetic::GetChildMemberWithName(llvm::StringRef name,
bool can_create) {
UpdateValueIfNeeded();
- uint32_t index = GetIndexOfChildWithName(name);
+ auto index_or_err = GetIndexOfChildWithName(
@@ -319,40 +319,49 @@
ValueObjectSynthetic::GetChildMemberWithName(llvm::StringRef name,
bool can_create) {
UpdateValueIfNeeded();
- uint32_t index = GetIndexOfChildWithName(name);
+ auto index_or_err = GetIndexOfChildWithName(
@@ -316,8 +316,10 @@
lldb_private::formatters::LibcxxSharedPtrSyntheticFrontEnd::
return 0;
if (name == "$$dereference$$")
return 1;
- return llvm::createStringError("Cannot find index of child '%s'",
- name.AsCString());
+ return ll
@@ -103,9 +103,20 @@
lldb_private::formatters::LibcxxInitializerListSyntheticFrontEnd::Update() {
llvm::Expected
lldb_private::formatters::LibcxxInitializerListSyntheticFrontEnd::
GetIndexOfChildWithName(ConstString name) {
- if (!m_start)
-return UINT32_MAX;
- retur
https://github.com/Michael137 approved this pull request.
LGTM
I left some remaining comments. Should be good to go once those are addressed
https://github.com/llvm/llvm-project/pull/136693
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
htt
@@ -418,8 +420,10 @@
lldb_private::formatters::LibcxxUniquePtrSyntheticFrontEnd::
return 1;
if (name == "$$dereference$$")
return 2;
- return llvm::createStringError("Cannot find index of child '%s'",
- name.AsCString());
+ return ll
@@ -107,7 +107,14 @@ class ListIterator {
class AbstractListFrontEnd : public SyntheticChildrenFrontEnd {
public:
llvm::Expected GetIndexOfChildWithName(ConstString name) override {
-return ExtractIndexFromString(name.GetCString());
+size_t idx = ExtractIndexFromStrin
@@ -1202,13 +1225,16 @@ lldb_private::formatters::Foundation1100::
m_data_64 = nullptr;
}
-size_t
-lldb_private::formatters::Foundation1100::
- NSDictionaryMSyntheticFrontEnd::GetIndexOfChildWithName(ConstString name) {
+llvm::Expected lldb_private::formatters::Foundation11
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/136693
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: Michael Buch
Date: 2025-04-24T12:45:19+01:00
New Revision: e35cc2d387e170d0e1f6ef647f17423262feb1ea
URL:
https://github.com/llvm/llvm-project/commit/e35cc2d387e170d0e1f6ef647f17423262feb1ea
DIFF:
https://github.com/llvm/llvm-project/commit/e35cc2d387e170d0e1f6ef647f17423262feb1ea.diff
@@ -123,11 +123,22 @@
lldb_private::formatters::LibcxxStdValarraySyntheticFrontEnd::Update() {
return ChildCacheState::eRefetch;
}
-size_t lldb_private::formatters::LibcxxStdValarraySyntheticFrontEnd::
+llvm::Expected
+lldb_private::formatters::LibcxxStdValarraySyntheticFro
@@ -259,14 +272,23 @@
lldb_private::formatters::LibcxxVectorBoolSyntheticFrontEnd::Update() {
return lldb::ChildCacheState::eRefetch;
}
-size_t lldb_private::formatters::LibcxxVectorBoolSyntheticFrontEnd::
+llvm::Expected
+lldb_private::formatters::LibcxxVectorBoolSynthetic
@@ -163,11 +163,24 @@
lldb_private::formatters::LibcxxStdVectorSyntheticFrontEnd::Update() {
return lldb::ChildCacheState::eRefetch;
}
-size_t lldb_private::formatters::LibcxxStdVectorSyntheticFrontEnd::
+llvm::Expected
+lldb_private::formatters::LibcxxStdVectorSyntheticFro
tromey wrote:
> `DW_AT_bit_size` will be ignored by lldb if `DW_AT_byte_size` is available
> and non-zero. Unfortunately, gdb crashes when `DW_AT_byte_size` does not
> exist or is set to zero. So I decided to change lldb to match gdb's behavior
> :(
Can you file a gdb bug report with an execu
anjenner wrote:
The original PR (https://github.com/llvm/llvm-project/pull/115331) was approved
and merged, but later reverted due an unstable test. The only difference in
this version of the PR was a fix to make the test stable on Windows, so I
thought it would be ok to merge by the approval
dyung wrote:
> The original PR (#115331) was approved and merged, but later reverted due an
> unstable test. The only difference in this version of the PR was a fix to
> make the test stable on Windows, so I thought it would be ok to merge by the
> approval of the original PR. Apologies if I w
anjenner wrote:
Hmm... perhaps I have misunderstood the LLVM contribution process here. I had
assumed that once all the review feedback had been addressed, some time had
passed to allow reviewers to comment on the modifications resulting from review
feedback, and the "merge" button was availab
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff HEAD~1 HEAD --extensions cpp --
lldb/source/Host/posix/ProcessLauncherPosixFork.cpp
DavidSpickett wrote:
Sod's law that the supposedly simple test would be the problem.
I'll figure this out, I've disabled it on Windows for now.
https://github.com/llvm/llvm-project/pull/132783
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
dtcxzyw wrote:
> > `DW_AT_bit_size` will be ignored by lldb if `DW_AT_byte_size` is available
> > and non-zero. Unfortunately, gdb crashes when `DW_AT_byte_size` does not
> > exist or is set to zero. So I decided to change lldb to match gdb's
> > behavior :(
>
> Can you file a gdb bug report
DavidSpickett wrote:
I have got Arm's Foundation Model to boot with SME only, and the cpuinfo is:
```
Features: fp asimd evtstrm crc32 atomics fphp asimdhp cpuid asimdrdm
jscvt fcma lrcpc dcpop asimddp asimdfhm dit uscat ilrcpc flagm sb paca pacg gcs
dcpodp flagm2 frint i8mm bf16 dgh rn
DavidSpickett wrote:
> Therefore, this situation should only be an issue with older kernels, so
> perhaps it not something that regular user mode applications should need to
> worry about (unless specifically wanting to run with older kernels).
I will test this PR with an older kernel as well.
https://github.com/hapee updated
https://github.com/llvm/llvm-project/pull/136682
>From 563cbddfe9b7da394c818a03e05924c0f7d39d5f Mon Sep 17 00:00:00 2001
From: hapee <623151...@qq.com>
Date: Tue, 22 Apr 2025 11:34:38 +0800
Subject: [PATCH 1/2] [lldb] Fix crash after second run when set a
@@ -174,6 +175,9 @@ int main(int argc, char **argv) {
//% self.expect("expr (new struct C[1]); side_effect", endstr=" = 4\n")
//% self.expect("expr delete c2; side_effect", endstr=" = 1\n")
//% self.expect("expr delete[] c3; side_effect", endstr=" = 2\n")
+ //% self.expe
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/137072
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Michael137 wrote:
> Yeah, getting some numbers would be nice. Maybe just repeating the benchmark
> in that diff. It might be a good idea to do it with and without the
> accelerator tables, just to rule out any effect from those.
Will do
On Darwin (without accelerator tables and statically lin
@@ -339,6 +342,48 @@ TEST_F(SocketTest, DomainGetConnectURI) {
}
#endif
+#if LLDB_ENABLE_POSIX
+TEST_F(SocketTest, DomainSocketFromBoundNativeSocket) {
+ // Generate a name for the domain socket.
+ llvm::SmallString<64> name;
+ std::error_code EC = llvm::sys::fs::createUniq
@@ -455,37 +455,33 @@ int main_platform(int argc, char *argv[]) {
lldb_private::Args inferior_arguments;
inferior_arguments.SetArguments(argc, const_cast(argv));
- Socket::SocketProtocol protocol = Socket::ProtocolUnixDomain;
-
+ Log *log = GetLog(LLDBLog::Platform);
@@ -339,6 +342,48 @@ TEST_F(SocketTest, DomainGetConnectURI) {
}
#endif
+#if LLDB_ENABLE_POSIX
+TEST_F(SocketTest, DomainSocketFromBoundNativeSocket) {
+ // Generate a name for the domain socket.
+ llvm::SmallString<64> name;
+ std::error_code EC = llvm::sys::fs::createUniq
@@ -339,6 +342,48 @@ TEST_F(SocketTest, DomainGetConnectURI) {
}
#endif
+#if LLDB_ENABLE_POSIX
+TEST_F(SocketTest, DomainSocketFromBoundNativeSocket) {
+ // Generate a name for the domain socket.
+ llvm::SmallString<64> name;
+ std::error_code EC = llvm::sys::fs::createUniq
@@ -182,3 +191,20 @@ std::vector
DomainSocket::GetListeningConnectionURI() const {
return {llvm::formatv("unix-connect://{0}", addr.sun_path)};
}
+
+llvm::Expected>
+DomainSocket::FromBoundNativeSocket(NativeSocket sockfd, bool should_close) {
+#ifdef __linux__
+ // Check
@@ -339,6 +342,48 @@ TEST_F(SocketTest, DomainGetConnectURI) {
}
#endif
+#if LLDB_ENABLE_POSIX
labath wrote:
```suggestion
```
https://github.com/llvm/llvm-project/pull/136466
___
lldb-commits mailing list
lldb-com
https://github.com/labath approved this pull request.
https://github.com/llvm/llvm-project/pull/136466
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -0,0 +1,195 @@
+"""
+Test lldb-dap stack trace when some of the source paths are missing
+"""
+
+from lldbsuite.test.decorators import skipIfWindows
+from lldbsuite.test.lldbtest import line_number
+import lldbdap_testcase
+from contextlib import contextmanager
+import os
+
+
+
@@ -0,0 +1,195 @@
+"""
+Test lldb-dap stack trace when some of the source paths are missing
+"""
+
+from lldbsuite.test.decorators import skipIfWindows
+from lldbsuite.test.lldbtest import line_number
+import lldbdap_testcase
+from contextlib import contextmanager
+import os
+
+
+
@@ -0,0 +1,195 @@
+"""
+Test lldb-dap stack trace when some of the source paths are missing
+"""
+
+from lldbsuite.test.decorators import skipIfWindows
+from lldbsuite.test.lldbtest import line_number
+import lldbdap_testcase
+from contextlib import contextmanager
+import os
+
+
+
@@ -0,0 +1,195 @@
+"""
+Test lldb-dap stack trace when some of the source paths are missing
+"""
+
+from lldbsuite.test.decorators import skipIfWindows
+from lldbsuite.test.lldbtest import line_number
+import lldbdap_testcase
+from contextlib import contextmanager
+import os
+
+
+
@@ -0,0 +1,10 @@
+#include
+#include
da-viper wrote:
we could remove the includes as it is not really used in tests. as adding them
increases the test times.
https://github.com/llvm/llvm-project/pull/136494
___
lld
@@ -1383,6 +1383,14 @@ enum CommandReturnObjectCallbackResult {
eCommandReturnObjectPrintCallbackHandled = 1,
};
+// Used to determine when to show disassembly
eronnen wrote:
:100:
https://github.com/llvm/llvm-project/pull/136494
_
@@ -0,0 +1,76 @@
+//===-- RegisterInfoPOSIX_riscv32.h -*- C++
-*-===//
+//
+// 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
@@ -0,0 +1,76 @@
+//===-- RegisterInfoPOSIX_riscv32.h -*- C++
-*-===//
+//
+// 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
https://github.com/eronnen updated
https://github.com/llvm/llvm-project/pull/136494
>From 05d605e3add0461b12ddfbb24349e20be4259060 Mon Sep 17 00:00:00 2001
From: Ely Ronnen
Date: Sun, 20 Apr 2025 17:07:09 +0200
Subject: [PATCH 1/6] fallback to assembly when source code is not available
---
ll
@@ -48,14 +48,6 @@ if (LLVM_ENABLE_DOXYGEN)
set(bolt_doxygen_qhp_cust_filter_attrs "")
endif()
- option(LLVM_DOXYGEN_SVG
cor3ntin wrote:
The couple of projects I found on Github (which sets it to "on" might get
```
CMake Warning:
Manually-specified
eronnen wrote:
@JDevlieghere Fixed comments and conflicts with `main`, from my part it's
ready to merge
https://github.com/llvm/llvm-project/pull/136494
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/
@@ -0,0 +1,195 @@
+"""
+Test lldb-dap stack trace when some of the source paths are missing
+"""
+
+from lldbsuite.test.decorators import skipIfWindows
+from lldbsuite.test.lldbtest import line_number
+import lldbdap_testcase
+from contextlib import contextmanager
+import os
+
+
+
https://github.com/da-viper edited
https://github.com/llvm/llvm-project/pull/136494
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -0,0 +1,195 @@
+"""
+Test lldb-dap stack trace when some of the source paths are missing
+"""
+
+from lldbsuite.test.decorators import skipIfWindows
+from lldbsuite.test.lldbtest import line_number
+import lldbdap_testcase
+from contextlib import contextmanager
+import os
+
+
+
@@ -2074,6 +2076,64 @@ static const Definition *FindEntry(const llvm::StringRef
&format_str,
return parent;
}
+/// Parses a single highlighting format specifier.
+///
+/// Example syntax for such specifier:
+/// \code
+/// ${function.name-with-args:%highlight_basename(ansi.
@@ -0,0 +1,195 @@
+"""
+Test lldb-dap stack trace when some of the source paths are missing
+"""
+
+from lldbsuite.test.decorators import skipIfWindows
+from lldbsuite.test.lldbtest import line_number
+import lldbdap_testcase
+from contextlib import contextmanager
+import os
+
+
+
@@ -0,0 +1,195 @@
+"""
+Test lldb-dap stack trace when some of the source paths are missing
+"""
+
+from lldbsuite.test.decorators import skipIfWindows
+from lldbsuite.test.lldbtest import line_number
+import lldbdap_testcase
+from contextlib import contextmanager
+import os
+
+
+
https://github.com/kastiglione created
https://github.com/llvm/llvm-project/pull/137262
None
>From 83d011739ba3a03058474922428c341b9333056f Mon Sep 17 00:00:00 2001
From: Dave Lee
Date: Thu, 24 Apr 2025 15:53:00 -0700
Subject: [PATCH] [lldb] Rename
reverse-continue/TestReverseContinueNotSuppo
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Dave Lee (kastiglione)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/137262.diff
1 Files Affected:
- (renamed)
lldb/test/API/commands/process/reverse-continue/TestReverseContinueCommandNotSupported.py
()
https://github.com/kastiglione edited
https://github.com/llvm/llvm-project/pull/137262
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
kastiglione wrote:
cc @rocallahan (for some reason I can't add you as a reviewer)
https://github.com/llvm/llvm-project/pull/137262
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/kastiglione edited
https://github.com/llvm/llvm-project/pull/137262
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/kastiglione edited
https://github.com/llvm/llvm-project/pull/137262
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/kastiglione edited
https://github.com/llvm/llvm-project/pull/137262
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/kastiglione edited
https://github.com/llvm/llvm-project/pull/137262
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/JDevlieghere updated
https://github.com/llvm/llvm-project/pull/137113
>From 21681616560eceb9b46ef4c370817099fe149701 Mon Sep 17 00:00:00 2001
From: Jonas Devlieghere
Date: Wed, 23 Apr 2025 20:05:19 -0700
Subject: [PATCH 1/4] [lldb-dap] Support StackFrameFormat
The debug adap
https://github.com/petrhosek approved this pull request.
https://github.com/llvm/llvm-project/pull/136843
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/Jlalond edited
https://github.com/llvm/llvm-project/pull/137041
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -67,7 +67,9 @@ TypeFilterImpl::FrontEnd::GetIndexOfChildWithName(ConstString
name) {
}
}
}
- return UINT32_MAX;
+ return llvm::createStringError(
+ "'SyntheticChildrenFrontEnd::FrontEnd' cannot find index of child '%s'",
adrian-prantl wro
Author: royitaqi
Date: 2025-04-24T17:23:41-07:00
New Revision: 967434aa3275842637937e9ac17614a10f81bae7
URL:
https://github.com/llvm/llvm-project/commit/967434aa3275842637937e9ac17614a10f81bae7
DIFF:
https://github.com/llvm/llvm-project/commit/967434aa3275842637937e9ac17614a10f81bae7.diff
LOG:
https://github.com/royitaqi closed
https://github.com/llvm/llvm-project/pull/136795
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/royitaqi updated
https://github.com/llvm/llvm-project/pull/136795
>From 85cb8cca4fe41cf4080b3dbf7ce4f98c4b15a3c7 Mon Sep 17 00:00:00 2001
From: Roy Shi
Date: Thu, 17 Apr 2025 15:03:00 -0700
Subject: [PATCH 01/10] [lldb] Do not parse symtab during statistics dump
Summary:
Te
royitaqi wrote:
Ran `ninja check-lldb` on both macOS and Linux and compared test counts
before/after this patch. No regression observed. Merging.
https://github.com/llvm/llvm-project/pull/136795
___
lldb-commits mailing list
lldb-commits@lists.llvm.or
@@ -672,21 +670,17 @@ DAP::CreateTargetFromArguments(const llvm::json::Object
&arguments,
// enough information to determine correct arch and platform (or ELF can be
// omitted at all), so it is good to leave the user an apportunity to specify
// those. Any of those thre
royitaqi wrote:
Checked `bin/llvm-lit -sv
../llvm-project/lldb/test/API/commands/statistics/basic/TestStats.py` again on
both macOS and Linux after merging with `main`.
https://github.com/llvm/llvm-project/pull/136795
___
lldb-commits mailing list
ll
@@ -9,127 +9,69 @@
#include "DAP.h"
#include "EventHelper.h"
#include "JSONUtils.h"
+#include "Protocol/ProtocolRequests.h"
#include "RequestHandler.h"
+#include "llvm/Support/Error.h"
#include "llvm/Support/FileSystem.h"
+using namespace llvm;
+using namespace lldb_dap::pr
@@ -9,127 +9,69 @@
#include "DAP.h"
#include "EventHelper.h"
#include "JSONUtils.h"
+#include "Protocol/ProtocolRequests.h"
#include "RequestHandler.h"
+#include "llvm/Support/Error.h"
#include "llvm/Support/FileSystem.h"
+using namespace llvm;
+using namespace lldb_dap::pr
@@ -37,14 +39,12 @@ MakeArgv(const llvm::ArrayRef &strs) {
return argv;
}
-static uint32_t SetLaunchFlag(uint32_t flags, const llvm::json::Object *obj,
- llvm::StringRef key, lldb::LaunchFlags mask) {
- if (const auto opt_value = GetBoolean(obj,
https://github.com/kusmour created
https://github.com/llvm/llvm-project/pull/137278
This patch makes the `request_attach` wait for events `process` and
`initialized` just like `request_launch`. This ensure the DAP session can move
forward somewhat correctly.
Recently `TestDap_attach.test_term
https://github.com/JDevlieghere created
https://github.com/llvm/llvm-project/pull/137280
Handle diagnostics events from the debugger (i.e. asynchronous errors and
warnings) in the lldb-dap event handler and emit them as "important" output.
>From f426ab064fff05cbda63dacc006a763e14fd1afd Mon Sep
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Jonas Devlieghere (JDevlieghere)
Changes
Handle diagnostics events from the debugger (i.e. asynchronous errors and
warnings) in the lldb-dap event handler and emit them as "important" output.
---
Full diff: https://github.com/llvm/llvm-pro
https://github.com/JDevlieghere updated
https://github.com/llvm/llvm-project/pull/137280
>From c2eab3f2f09fd771cbd8c18f141de1fa7664af49 Mon Sep 17 00:00:00 2001
From: Jonas Devlieghere
Date: Thu, 24 Apr 2025 17:59:29 -0700
Subject: [PATCH] [lldb] Emit diagnostics as "important" output
Handle d
@@ -159,6 +159,9 @@ GetEnvironmentFromArguments(const llvm::json::Object
&arguments);
/// Take ownership of the stored error.
llvm::Error ToError(const lldb::SBError &error);
+/// Provides the string value if this data structure is a string type.
+std::string GetStringValue(c
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Wanyi (kusmour)
Changes
This patch makes the `request_attach` wait for events `process` and
`initialized` just like `request_launch`. This ensure the DAP session can move
forward somewhat correctly.
Recently `TestDap_attach.test_terminate
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/packages/Python/lldbsuite/test/tools/lldb-dap/lldbdap_testcase.py
lldb
1 - 100 of 159 matches
Mail list logo
