@@ -1629,6 +1630,80 @@ static void
LoadLibStdcppFormatters(lldb::TypeCategoryImplSP cpp_category_sp) {
"^std::optional<.+>(( )?&)?$", stl_summary_flags, true);
}
+static void LoadCommonStlFormatters(lldb::TypeCategoryImplSP cpp_category_sp) {
@@ -1372,6 +1373,37 @@ static void
LoadLibCxxFormatters(lldb::TypeCategoryImplSP cpp_category_sp) {
"${var.__y_} ${var.__m_} ${var.__wdl_}")));
}
+static void RegisterStdStringSummaryProvider(
+const lldb::TypeCategoryImplSP &category_sp, llvm::StringRef string_
@@ -175,15 +174,16 @@ static FormSize g_form_sizes[] = {
{1, 8}, // 0x14 DW_FORM_ref8
{0, 0}, // 0x15 DW_FORM_ref_udata
{0, 0}, // 0x16 DW_FORM_indirect
-{1, 4}, // 0x17 DW_FORM_sec_offset
+{0,
+ 0}, // 0x17 DW_FORM_sec_offset (4 bytes for DWARF32, 8
@@ -175,15 +174,16 @@ static FormSize g_form_sizes[] = {
{1, 8}, // 0x14 DW_FORM_ref8
{0, 0}, // 0x15 DW_FORM_ref_udata
{0, 0}, // 0x16 DW_FORM_indirect
-{1, 4}, // 0x17 DW_FORM_sec_offset
+{0,
+ 0}, // 0x17 DW_FORM_sec_offset (4 bytes for DWARF32, 8
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/5] [lldb][DWARF64] Enable support for DWARF64 format
han
@@ -21,9 +26,15 @@ TEST(Support, getProcFile_Pid) {
}
#endif // #ifndef __APPLE__
-#if defined(_AIX) && defined(LLVM_ENABLE_THREADING)
+#if (defined(_AIX) || defined(__linux__)) && defined(LLVM_ENABLE_THREADING)
TEST(Support, getProcFile_Tid) {
- auto BufferOrError = getProc
https://github.com/DavidSpickett 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
DavidSpickett wrote:
ping!
Note that `Show MTE store only setting in mte_ctrl` is the commit with new
code, rest is the previous PR.
https://github.com/llvm/llvm-project/pull/145033
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://li
@@ -175,15 +174,16 @@ static FormSize g_form_sizes[] = {
{1, 8}, // 0x14 DW_FORM_ref8
{0, 0}, // 0x15 DW_FORM_ref_udata
{0, 0}, // 0x16 DW_FORM_indirect
-{1, 4}, // 0x17 DW_FORM_sec_offset
+{0,
+ 0}, // 0x17 DW_FORM_sec_offset (4 bytes for DWARF32, 8
https://github.com/DavidSpickett approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/146500
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
DavidSpickett wrote:
Please go ahead and merge this, or I can do so for you if you're happy with 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/listi
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/5] [lldb][DWARF64] Enable support for DWARF64 format
han
labath wrote:
Sorry for going off-topic, but what does this say about the support for
`i386-apple-*` as a whole? I find the prospect of dropping that exciting, as I
believe that is the only target where eh_ and debug_frame register numbers are
different.
https://github.com/llvm/llvm-project/p
@@ -0,0 +1,6 @@
+# All tests for the tool need lldb-rpc-gen to be built.
+# As far as I can see, you need to check all possible ways to
+# say "no" in a CMake option.
+negation_options = ["OFF", "NO", 0]
+if not config.lldb_has_lldbrpc or config.lldb_has_lldbrpc in negation_option
HemangGadhavi wrote:
> Assuming there is not a lot more work to be done for DWARF64, this needs a
> release note -
> https://github.com/llvm/llvm-project/blob/main/llvm/docs/ReleaseNotes.md#changes-to-lldb.
>
> You can do that in this PR or a follow up.
Updated with this PR only, Please revie
@@ -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(
+
@@ -124,7 +124,9 @@ TEST_F(MainLoopTest, NoSpuriousPipeReads) {
// pipe is empty, and we should not be called anymore.
char X;
size_t len = sizeof(X);
- EXPECT_THAT_EXPECTED(r->Read(&X, len).ToError(), llvm::HasValue(1));
+ ASSERT
https://github.com/labath approved this pull request.
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
DavidSpickett wrote:
Assuming there is not a lot more work to be done for DWARF64, this needs a
release note -
https://github.com/llvm/llvm-project/blob/main/llvm/docs/ReleaseNotes.md#changes-to-lldb.
You can do that in this PR or a follow up.
https://github.com/llvm/llvm-project/pull/145645
@@ -41,10 +41,26 @@ Statusline::Statusline(Debugger &debugger)
Statusline::~Statusline() { Disable(); }
void Statusline::TerminalSizeChanged() {
- UpdateTerminalProperties();
+ const uint64_t terminal_width = m_debugger.GetTerminalWidth();
+ const uint64_t terminal_height =
https://github.com/labath commented:
This messes up my terminal when reducing the height. It goes from
to
https://github.com/labath edited
https://github.com/llvm/llvm-project/pull/146435
___
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.
Thanks.
https://github.com/llvm/llvm-project/pull/145055
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
labath wrote:
Okay, I think I see the source of confusion. I was expecting the terminal to
keep the status line at the bottom of the screen after a resize (just like it
does with regular text), but my terminal (XFCE) at least simply discards it
when decreasing the window size. It makes kind of
labath wrote:
The way I understand this, this patch isn't trying to fix the step over problem
(https://github.com/llvm/llvm-project/issues/144061). What it does is fix a
different (but similar) problem in lldb-dap where it would show the wrong
source code for in cases where the step operations
@@ -21,9 +26,15 @@ TEST(Support, getProcFile_Pid) {
}
#endif // #ifndef __APPLE__
-#if defined(_AIX) && defined(LLVM_ENABLE_THREADING)
+#if (defined(_AIX) || defined(__linux__)) && defined(LLVM_ENABLE_THREADING)
TEST(Support, getProcFile_Tid) {
- auto BufferOrError = getProc
@@ -0,0 +1,120 @@
+# coding=utf8
+"""
+Test std::*string summaries with MSVC's STL.
+"""
labath wrote:
I think it's mainly historical. We have
`data-formatter/data-formatter-stl/generic/`, but it's a fairly recent
addition. I think it would be nice to move ever
@@ -175,15 +174,16 @@ static FormSize g_form_sizes[] = {
{1, 8}, // 0x14 DW_FORM_ref8
{0, 0}, // 0x15 DW_FORM_ref_udata
{0, 0}, // 0x16 DW_FORM_indirect
-{1, 4}, // 0x17 DW_FORM_sec_offset
+{0,
+ 0}, // 0x17 DW_FORM_sec_offset (4 bytes for DWARF32, 8
@@ -41,10 +41,26 @@ Statusline::Statusline(Debugger &debugger)
Statusline::~Statusline() { Disable(); }
void Statusline::TerminalSizeChanged() {
- UpdateTerminalProperties();
+ const uint64_t terminal_width = m_debugger.GetTerminalWidth();
+ const uint64_t terminal_height =
https://github.com/labath updated
https://github.com/llvm/llvm-project/pull/146500
>From 2b30e53ff0c1d03e3acfd10c06863463a99dec60 Mon Sep 17 00:00:00 2001
From: Pavel Labath
Date: Tue, 1 Jul 2025 12:06:23 +0200
Subject: [PATCH 1/2] [lldb] Add linux test for the three-arg version of
getProcFile
@@ -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 =
@@ -21,9 +26,15 @@ TEST(Support, getProcFile_Pid) {
}
#endif // #ifndef __APPLE__
-#if defined(_AIX) && defined(LLVM_ENABLE_THREADING)
+#if (defined(_AIX) || defined(__linux__)) && defined(LLVM_ENABLE_THREADING)
labath wrote:
Good question. I assumed that's b
DavidSpickett wrote:
The above result is a flaky text, the next build was green so you can ignore 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/list
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/7] [lldb][NFC] Inline ResolveSDKPathFromDebugInfo in on
https://github.com/Nerixyz updated
https://github.com/llvm/llvm-project/pull/143177
>From df589845dbccef086d966e21e8183a3dbada041a Mon Sep 17 00:00:00 2001
From: Nerixyz
Date: Wed, 18 Jun 2025 21:49:16 +0200
Subject: [PATCH] [LLDB] Add type summaries for MSVC STL strings
---
lldb/packages/Pyt
@@ -18,11 +18,11 @@ namespace lldb_private {
namespace formatters {
bool LibStdcppStringSummaryProvider(
ValueObject &valobj, Stream &stream,
-const TypeSummaryOptions &options); // libcstdc++ c++11 std::string
+const TypeSummaryOptions &options); // libcstdc++ std:
https://github.com/DavidSpickett 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
Author: Hemang Gadhavi
Date: 2025-07-01T18:35:40+05:30
New Revision: da0828b1e93c0fb6266225d8e6d41b2d9ef1f346
URL:
https://github.com/llvm/llvm-project/commit/da0828b1e93c0fb6266225d8e6d41b2d9ef1f346
DIFF:
https://github.com/llvm/llvm-project/commit/da0828b1e93c0fb6266225d8e6d41b2d9ef1f346.diff
https://github.com/DhruvSrivastavaX closed
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
101 - 139 of 139 matches
Mail list logo
