jimingham wrote:
Sorry to come in a little late on this, but I think we need a setting to turn
this off as well. If you aren't planning to use the transcript, you shouldn't
have to pay the cost for it.
https://github.com/llvm/llvm-project/pull/90703
___
@@ -766,6 +768,12 @@ class CommandInterpreter : public Broadcaster,
CommandUsageMap m_command_usages;
StreamString m_transcript_stream;
+
+ /// Contains a list of handled commands, output and error. Each element in
+ /// the list is a dictionary with three keys: "command
@@ -2044,6 +2052,15 @@ bool CommandInterpreter::HandleCommand(const char
*command_line,
m_transcript_stream << result.GetOutputData();
m_transcript_stream << result.GetErrorData();
+ // Add output and error to the transcript item after splitting lines. In the
+ // futur
https://github.com/slydiman edited
https://github.com/llvm/llvm-project/pull/91146
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/slydiman updated
https://github.com/llvm/llvm-project/pull/91146
>From c7cef45aa369014650cc99d404913c17cb3c1d45 Mon Sep 17 00:00:00 2001
From: Dmitry Vasilyev
Date: Mon, 6 May 2024 01:10:03 +0400
Subject: [PATCH] [lldb] Fixed SyntaxWarning: invalid escape sequence \[ \d \s
R
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: None (jeffreytan81)
Changes
This PR introduces a new `ThreadPlanSingleThreadTimeout` that will be used to
address potential deadlock during single-thread stepping.
While debugging a target with a non-trivial number of threads (around 5000
https://github.com/jeffreytan81 ready_for_review
https://github.com/llvm/llvm-project/pull/90930
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/jeffreytan81 edited
https://github.com/llvm/llvm-project/pull/90930
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/JDevlieghere closed
https://github.com/llvm/llvm-project/pull/91254
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: Jonas Devlieghere
Date: 2024-05-06T14:43:47-07:00
New Revision: 3809e20afc68d7d03821f0ec59b928dcf9befbf4
URL:
https://github.com/llvm/llvm-project/commit/3809e20afc68d7d03821f0ec59b928dcf9befbf4
DIFF:
https://github.com/llvm/llvm-project/commit/3809e20afc68d7d03821f0ec59b928dcf9befbf4.d
JDevlieghere wrote:
Good point. I can't imagine anything needing the SB header that doesn't need
the rest of the framework, but if that's the case we go through a custom
target.
https://github.com/llvm/llvm-project/pull/91254
___
lldb-commits mailin
JDevlieghere wrote:
The change itself looks fine, but can you please add a bit more info to the
commit message, such as the Python version you're seeing this with.
https://github.com/llvm/llvm-project/pull/91146
___
lldb-commits mailing list
lldb-com
https://github.com/royitaqi edited
https://github.com/llvm/llvm-project/pull/90703
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/royitaqi edited
https://github.com/llvm/llvm-project/pull/90703
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/royitaqi edited
https://github.com/llvm/llvm-project/pull/90703
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/bulbazord approved this pull request.
https://github.com/llvm/llvm-project/pull/91254
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
bulbazord wrote:
This should work if there is exactly one thing that depends on the output of
this custom command. If multiple things start depending on the generated files,
you may end up with weird results. See:
https://cmake.org/cmake/help/latest/command/add_custom_command.html
> Do not lis
https://github.com/JDevlieghere approved this pull request.
https://github.com/llvm/llvm-project/pull/91162
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
smurfd wrote:
Thanks @adrian-prantl, simple is better :)
https://github.com/llvm/llvm-project/pull/91128
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/smurfd updated
https://github.com/llvm/llvm-project/pull/91128
>From 9a90c71f6fa086cdaf3400bdc5142835a12239bd Mon Sep 17 00:00:00 2001
From: Nicklas Boman
Date: Sun, 5 May 2024 17:40:51 +0200
Subject: [PATCH] lldb create API folder if it does not exist, before creating
SBLan
@@ -20,9 +20,19 @@ if(LLDB_ENABLE_LUA)
set(lldb_lua_wrapper ${lua_bindings_dir}/LLDBWrapLua.cpp)
endif()
-lldb_tablegen(../../include/lldb/API/SBLanguages.h -gen-lldb-sbapi-dwarf-enum
- SOURCE ${LLVM_MAIN_INCLUDE_DIR}/llvm/BinaryFormat/Dwarf.def
- TARGET lldb-sbapi-dwarf-e
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Jonas Devlieghere (JDevlieghere)
Changes
Use add_custom_command instead of add_custom_target to generate SBLanguages.h.
---
Full diff: https://github.com/llvm/llvm-project/pull/91254.diff
1 Files Affected:
- (modified) lldb/source/API/CM
https://github.com/JDevlieghere created
https://github.com/llvm/llvm-project/pull/91254
Use add_custom_command instead of add_custom_target to generate SBLanguages.h.
>From 7fa130d2bb2e45f350ca8c3851d32bbec3332f8c Mon Sep 17 00:00:00 2001
From: Jonas Devlieghere
Date: Mon, 6 May 2024 11:48:24
@@ -37,6 +38,10 @@ def emit_enum(input, output):
with open(input, "r") as f:
lines = f.readlines()
+# Create output folder if it does not exist
+if not os.path.isdir(os.path.dirname(output)):
adrian-prantl wrote:
This can be simplified: ht
smurfd wrote:
Thanks @JDevlieghere! i do not have commit access, will you please merge this?
https://github.com/llvm/llvm-project/pull/91128
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-
https://github.com/jeffreytan81 updated
https://github.com/llvm/llvm-project/pull/90930
>From b72df8cf2a047ed731913609b58bdb4a3601e111 Mon Sep 17 00:00:00 2001
From: jeffreytan81
Date: Thu, 2 May 2024 18:12:04 -0700
Subject: [PATCH 1/3] Single thread timeout feature
---
lldb/include/lldb/Targ
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 cf3c714e4bd7b8a68793f2827080fe3388ae8bb1
d125c5a761a70b024afb16d22b4326df4071e04b --
https://github.com/jeffreytan81 updated
https://github.com/llvm/llvm-project/pull/90930
>From b72df8cf2a047ed731913609b58bdb4a3601e111 Mon Sep 17 00:00:00 2001
From: jeffreytan81
Date: Thu, 2 May 2024 18:12:04 -0700
Subject: [PATCH 1/2] Single thread timeout feature
---
lldb/include/lldb/Targ
medismailben wrote:
> Since we're already using `argparse`, can we use the `BooleanOptionalAction`
> to make this work automatically with the affirmative option?
> https://docs.python.org/3.9/library/argparse.html#action
I did that originally but I had to comment it out in 446abb5 because some
royitaqi wrote:
I believe I have addressed all your comments, @bulbazord .
I appreciate your comments so far - they all make sense and are great comments
to help me improve this PR. Thank you for spending time on this.
https://github.com/llvm/llvm-project/pull/90703
___
https://github.com/royitaqi edited
https://github.com/llvm/llvm-project/pull/90703
___
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/90703
>From 0fd67e2de7e702ce6f7353845454ea7ff9f980d6 Mon Sep 17 00:00:00 2001
From: Roy Shi
Date: Tue, 30 Apr 2024 21:35:49 -0700
Subject: [PATCH 01/11] Add SBCommandInterpreter::GetTranscript()
---
lldb/include/lld
https://github.com/JDevlieghere approved this pull request.
https://github.com/llvm/llvm-project/pull/91128
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/royitaqi edited
https://github.com/llvm/llvm-project/pull/90703
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/royitaqi edited
https://github.com/llvm/llvm-project/pull/90703
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -85,3 +86,84 @@ def test_command_output(self):
self.assertEqual(res.GetOutput(), "")
self.assertIsNotNone(res.GetError())
self.assertEqual(res.GetError(), "")
+
+def test_structured_transcript(self):
+"""Test structured transcript generati
https://github.com/JDevlieghere commented:
Since we're already using `argparse`, can we use the `BooleanOptionalAction` to
make this work automatically with the affirmative option?
https://docs.python.org/3.9/library/argparse.html#action
https://github.com/llvm/llvm-project/pull/91162
@@ -100,6 +111,10 @@ def get_register_context(self) -> str:
def __lldb_init_module(debugger, dict):
if not "SKIP_SCRIPTED_PROCESS_LAUNCH" in os.environ:
+debugger.HandleCommand(
JDevlieghere wrote:
This seems like it would be worth documenting in
@@ -85,3 +86,84 @@ def test_command_output(self):
self.assertEqual(res.GetOutput(), "")
self.assertIsNotNone(res.GetError())
self.assertEqual(res.GetError(), "")
+
+def test_structured_transcript(self):
+"""Test structured transcript generati
https://github.com/bulbazord closed
https://github.com/llvm/llvm-project/pull/91010
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: Alex Langford
Date: 2024-05-06T10:11:03-07:00
New Revision: cf6d79711d8a423cade5a555db650cb46a74e565
URL:
https://github.com/llvm/llvm-project/commit/cf6d79711d8a423cade5a555db650cb46a74e565
DIFF:
https://github.com/llvm/llvm-project/commit/cf6d79711d8a423cade5a555db650cb46a74e565.diff
@@ -100,6 +111,10 @@ def get_register_context(self) -> str:
def __lldb_init_module(debugger, dict):
if not "SKIP_SCRIPTED_PROCESS_LAUNCH" in os.environ:
+debugger.HandleCommand(
medismailben wrote:
I added this for when loading the file it by hand
@@ -100,6 +111,10 @@ def get_register_context(self) -> str:
def __lldb_init_module(debugger, dict):
if not "SKIP_SCRIPTED_PROCESS_LAUNCH" in os.environ:
+debugger.HandleCommand(
jimingham wrote:
Why do you have to add the stop hook here when the m
@@ -801,7 +801,7 @@ static OutputDesc
*addInputSec(StringMap> &map,
auto *firstIsec = cast(
cast(sec->commands[0])->sectionBases[0]);
OutputSection *firstIsecOut =
- firstIsec->flags & SHF_LINK_ORDER
luolent wrote:
Hi @MaskRay ,
ZequanWu wrote:
> You could enable logging and check for specific logging after steps. In the
> test I described above if you just print the "Foo *foo" variable, it won't
> need to complete the definition, you could check for logging, and then if you
> print "*foo", then it should complete the
@@ -1667,13 +1791,40 @@ DWARFASTParserClang::ParseStructureLikeDIE(const
SymbolContext &sc,
}
if (dwarf->GetUniqueDWARFASTTypeMap().Find(
-unique_typename, die, unique_decl, attrs.byte_size.value_or(-1),
-*unique_ast_entry_up)) {
+u
https://github.com/ZequanWu updated
https://github.com/llvm/llvm-project/pull/90663
>From 4e83099b593e66f12dc21be5fbac5279e03e Mon Sep 17 00:00:00 2001
From: Zequan Wu
Date: Tue, 30 Apr 2024 16:23:11 -0400
Subject: [PATCH 1/6] [lldb][DWARF] Delay struct/class/union definition DIE
searching
https://github.com/ZequanWu closed
https://github.com/llvm/llvm-project/pull/87649
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: Zequan Wu
Date: 2024-05-06T09:55:23-04:00
New Revision: 7b040d01c59505d62d4700c75b0438269a0f7bb4
URL:
https://github.com/llvm/llvm-project/commit/7b040d01c59505d62d4700c75b0438269a0f7bb4
DIFF:
https://github.com/llvm/llvm-project/commit/7b040d01c59505d62d4700c75b0438269a0f7bb4.diff
LOG
https://github.com/labath updated
https://github.com/llvm/llvm-project/pull/91189
>From 7de5cfcc3016a9e49e289e1009bcf4cc68aa445e Mon Sep 17 00:00:00 2001
From: Pavel Labath
Date: Mon, 6 May 2024 11:30:59 +
Subject: [PATCH] [lldb] Make SBType::GetDirectNestedType (mostly) work with
typedefs
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Pavel Labath (labath)
Changes
The implementation is straight-forward, but comes with a big disclaimer. See
#91186 for details.
---
Full diff: https://github.com/llvm/llvm-project/pull/91189.diff
3 Files Affected:
- (modified) lldb/sourc
https://github.com/labath created
https://github.com/llvm/llvm-project/pull/91189
The implementation is straight-forward, but comes with a big disclaimer. See
#91186 for details.
>From 62f1917a75b4db82470cf5b2721097e6757d5dfb Mon Sep 17 00:00:00 2001
From: Pavel Labath
Date: Mon, 6 May 2024 1
https://github.com/zyn0217 closed
https://github.com/llvm/llvm-project/pull/91132
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
zyn0217 wrote:
This had been fixed by 41574f5a6. Thanks @dwblaikie!
https://github.com/llvm/llvm-project/pull/91132
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -20,9 +20,19 @@ if(LLDB_ENABLE_LUA)
set(lldb_lua_wrapper ${lua_bindings_dir}/LLDBWrapLua.cpp)
endif()
-lldb_tablegen(../../include/lldb/API/SBLanguages.h -gen-lldb-sbapi-dwarf-enum
- SOURCE ${LLVM_MAIN_INCLUDE_DIR}/llvm/BinaryFormat/Dwarf.def
- TARGET lldb-sbapi-dwarf-e
https://github.com/santhoshe447 updated
https://github.com/llvm/llvm-project/pull/91170
>From 960351c9abf51f42d92604ac6297aa5b76ddfba5 Mon Sep 17 00:00:00 2001
From: Santhosh Kumar Ellendula
Date: Fri, 17 Nov 2023 15:09:10 +0530
Subject: [PATCH 1/6] [lldb][test] Add the ability to extract the v
@@ -1,3 +1,10 @@
+# Checkout as native, commit as LF except in specific circumstances
+* text=auto
+*.bat text eol=crlf
+*.rc text eol=crlf
+*.sln text eol=crlf
ldrumm wrote:
I think it makes sense to put the well-known-extension attributes in the root.
I don't
https://github.com/santhoshe447 updated
https://github.com/llvm/llvm-project/pull/91170
>From 960351c9abf51f42d92604ac6297aa5b76ddfba5 Mon Sep 17 00:00:00 2001
From: Santhosh Kumar Ellendula
Date: Fri, 17 Nov 2023 15:09:10 +0530
Subject: [PATCH 1/5] [lldb][test] Add the ability to extract the v
https://github.com/santhoshe447 updated
https://github.com/llvm/llvm-project/pull/91170
>From 960351c9abf51f42d92604ac6297aa5b76ddfba5 Mon Sep 17 00:00:00 2001
From: Santhosh Kumar Ellendula
Date: Fri, 17 Nov 2023 15:09:10 +0530
Subject: [PATCH 1/4] [lldb][test] Add the ability to extract the v
https://github.com/santhoshe447 updated
https://github.com/llvm/llvm-project/pull/91170
>From 960351c9abf51f42d92604ac6297aa5b76ddfba5 Mon Sep 17 00:00:00 2001
From: Santhosh Kumar Ellendula
Date: Fri, 17 Nov 2023 15:09:10 +0530
Subject: [PATCH 1/4] [lldb][test] Add the ability to extract the v
https://github.com/santhoshe447 updated
https://github.com/llvm/llvm-project/pull/91170
>From 960351c9abf51f42d92604ac6297aa5b76ddfba5 Mon Sep 17 00:00:00 2001
From: Santhosh Kumar Ellendula
Date: Fri, 17 Nov 2023 15:09:10 +0530
Subject: [PATCH 1/3] [lldb][test] Add the ability to extract the v
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
e2c89254e12e844214d02f1d12bf29ba2ca322c7...efffdf33f5f290c318ce4dd15250be5adcbed252
lldb/
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 e2c89254e12e844214d02f1d12bf29ba2ca322c7
efffdf33f5f290c318ce4dd15250be5adcbed252 --
https://github.com/labath closed https://github.com/llvm/llvm-project/pull/90960
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: Pavel Labath
Date: 2024-05-06T10:06:51+02:00
New Revision: 30367cb5982dfdab2655401f020711311e7d78b9
URL:
https://github.com/llvm/llvm-project/commit/30367cb5982dfdab2655401f020711311e7d78b9
DIFF:
https://github.com/llvm/llvm-project/commit/30367cb5982dfdab2655401f020711311e7d78b9.diff
https://github.com/santhoshe447 created
https://github.com/llvm/llvm-project/pull/91170
Adding a "port" property to the VsCode "attach" command likely extends the
functionality of the debugger configuration to allow attaching to a process
using PID or PORT number.
Currently, the "Attach" confi
@@ -1667,13 +1791,40 @@ DWARFASTParserClang::ParseStructureLikeDIE(const
SymbolContext &sc,
}
if (dwarf->GetUniqueDWARFASTTypeMap().Find(
-unique_typename, die, unique_decl, attrs.byte_size.value_or(-1),
-*unique_ast_entry_up)) {
+u
@@ -154,6 +154,27 @@ static bool TagIsRecordType(dw_tag_t tag) {
}
}
+static bool
+IsForwardDeclaration(const lldb_private::plugin::dwarf::DWARFDIE &die,
labath wrote:
This shouldn't be necessary here, as this file has `using namespace
lldb_private::plugin
https://github.com/labath approved this pull request.
https://github.com/llvm/llvm-project/pull/90783
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
69 matches
Mail list logo
