https://github.com/DavidSpickett approved this pull request.
Identical code to Linux apart from `/lwp/` in place of `/task/`. Wondered if we
could share that but as long as it remains identical, we could always do it
later if we want.
It would be something like `getProcFile(::pid_t pid, ::pid_
https://github.com/DavidSpickett approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/142573
___
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: Hemang Gadhavi (HemangGadhavi)
Changes
This PR is in reference to porting LLDB on AIX.
Link to discussions on llvm discourse and github:
1. https://discourse.llvm.org/t/port-lldb-to-ibm-aix/80640
2. https://github.com/llvm/llvm-project/issu
https://github.com/labath created
https://github.com/llvm/llvm-project/pull/142583
If we're not touching them, we don't need to do anything special to pass them
along -- with one important caveat: due to how cmake arguments work, the
implicitly passed arguments need to be specified before argu
llvmbot wrote:
@llvm/pr-subscribers-backend-hexagon
Author: Pavel Labath (labath)
Changes
If we're not touching them, we don't need to do anything special to pass them
along -- with one important caveat: due to how cmake arguments work, the
implicitly passed arguments need to be specifie
Author: Michael Buch
Date: 2025-06-03T12:17:50+01:00
New Revision: c48c91a92e50e7e01ab9befa1ff7f3bc9662ecac
URL:
https://github.com/llvm/llvm-project/commit/c48c91a92e50e7e01ab9befa1ff7f3bc9662ecac
DIFF:
https://github.com/llvm/llvm-project/commit/c48c91a92e50e7e01ab9befa1ff7f3bc9662ecac.diff
Author: Michael Buch
Date: 2025-06-03T12:20:31+01:00
New Revision: 05547fc3ec803b779beefeb52d21ef76cf00413f
URL:
https://github.com/llvm/llvm-project/commit/05547fc3ec803b779beefeb52d21ef76cf00413f
DIFF:
https://github.com/llvm/llvm-project/commit/05547fc3ec803b779beefeb52d21ef76cf00413f.diff
llvmbot wrote:
@llvm/pr-subscribers-llvm-support
Author: Hemang Gadhavi (HemangGadhavi)
Changes
This PR is in reference to porting LLDB on AIX.
Link to discussions on llvm discourse and github:
1. https://discourse.llvm.org/t/port-lldb-to-ibm-aix/80640
2. https://github.com/llvm/llvm-proj
https://github.com/HemangGadhavi created
https://github.com/llvm/llvm-project/pull/142586
This PR is in reference to porting LLDB on AIX.
Link to discussions on llvm discourse and github:
1. https://discourse.llvm.org/t/port-lldb-to-ibm-aix/80640
2. https://github.com/llvm/llvm-project/issues/1
vogelsgesang wrote:
superseded by #140486
https://github.com/llvm/llvm-project/pull/105446
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/vogelsgesang closed
https://github.com/llvm/llvm-project/pull/105446
___
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-03T09:33:24Z
New Revision: 75ec944e38eedfc9357171697bceabe98dd9fadb
URL:
https://github.com/llvm/llvm-project/commit/75ec944e38eedfc9357171697bceabe98dd9fadb
DIFF:
https://github.com/llvm/llvm-project/commit/75ec944e38eedfc9357171697bceabe98dd9fadb.diff
LOG
https://github.com/HemangGadhavi edited
https://github.com/llvm/llvm-project/pull/142586
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/HemangGadhavi edited
https://github.com/llvm/llvm-project/pull/142586
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/slydiman created
https://github.com/llvm/llvm-project/pull/142573
See #142196 and https://github.com/llvm/llvm-zorg/pull/452 for details.
>From 7f770486b80d0f927f5e4ad16f5e31a771b2ad0a Mon Sep 17 00:00:00 2001
From: Dmitry Vasilyev
Date: Tue, 3 Jun 2025 14:28:58 +0400
Subjec
llvmbot wrote:
@llvm/pr-subscribers-lldb
@llvm/pr-subscribers-platform-windows
Author: Dmitry Vasilyev (slydiman)
Changes
See #142196 and https://github.com/llvm/llvm-zorg/pull/452 for details.
---
Full diff: https://github.com/llvm/llvm-project/pull/142573.diff
1 Files Affected:
- (mo
Author: Michael Buch
Date: 2025-06-03T13:16:03+01:00
New Revision: 3ddc1e1cf397bd495f5aa42b04630561a9e6bf47
URL:
https://github.com/llvm/llvm-project/commit/3ddc1e1cf397bd495f5aa42b04630561a9e6bf47
DIFF:
https://github.com/llvm/llvm-project/commit/3ddc1e1cf397bd495f5aa42b04630561a9e6bf47.diff
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Pavel Labath (labath)
Changes
Replace (questionable) header globs with an explicit argument supported by
llvm_add_library.
---
Full diff: https://github.com/llvm/llvm-project/pull/142587.diff
17 Files Affected:
- (modified) lldb/CMakeLi
https://github.com/labath created
https://github.com/llvm/llvm-project/pull/142587
Replace (questionable) header globs with an explicit argument supported by
llvm_add_library.
>From 450e714d9eb1a6f15a6b6065cf0c60f120948f65 Mon Sep 17 00:00:00 2001
From: Pavel Labath
Date: Tue, 3 Jun 2025 13:4
Author: Michael Buch
Date: 2025-06-03T12:10:15+01:00
New Revision: 9c52b177ea27778e904908c974e8113ed637dd69
URL:
https://github.com/llvm/llvm-project/commit/9c52b177ea27778e904908c974e8113ed637dd69
DIFF:
https://github.com/llvm/llvm-project/commit/9c52b177ea27778e904908c974e8113ed637dd69.diff
@@ -414,6 +415,34 @@ bool fromJSON(const llvm::json::Value &,
SteppingGranularity &,
llvm::json::Path);
llvm::json::Value toJSON(const SteppingGranularity &);
+/// A `StepInTarget` can be used in the `stepIn` request and determines into
+/// which single target
@@ -356,7 +356,21 @@ class StepInRequestHandler : public
RequestHandler> {
+public:
+ using RequestHandler::RequestHandler;
+ static llvm::StringLiteral GetCommand() { return "stepInTargets"; }
+ FeatureSet GetSupportedFeatures() const override {
+return {protocol::eAdapte
https://github.com/da-viper updated
https://github.com/llvm/llvm-project/pull/142439
>From 69efc48d722d18600018f25db0f9ea46b9fd1d97 Mon Sep 17 00:00:00 2001
From: Ebuka Ezike
Date: Wed, 28 May 2025 14:02:01 +0100
Subject: [PATCH 1/2] [lldb-dap] Use structured types for stepInTargets request
--
https://github.com/da-viper updated
https://github.com/llvm/llvm-project/pull/142439
>From 69efc48d722d18600018f25db0f9ea46b9fd1d97 Mon Sep 17 00:00:00 2001
From: Ebuka Ezike
Date: Wed, 28 May 2025 14:02:01 +0100
Subject: [PATCH 1/3] [lldb-dap] Use structured types for stepInTargets request
--
https://github.com/da-viper approved this pull request.
https://github.com/llvm/llvm-project/pull/142508
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/Michael137 created
https://github.com/llvm/llvm-project/pull/142620
None
>From bd2788fb6c4ba71c400d615f723d9f8e9522450c Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Tue, 3 Jun 2025 15:51:40 +0100
Subject: [PATCH] [lldb][TypeSystem][NFC] CreateFunctionType to take parame
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Michael Buch (Michael137)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/142620.diff
10 Files Affected:
- (modified)
lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionDeclMap.cpp (+2-2)
- (modified)
lldb/sou
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/142620
>From f69c768795f647d97b85bce5416b2302eb6b5a6d Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Tue, 3 Jun 2025 15:51:40 +0100
Subject: [PATCH] [lldb][TypeSystem][NFC] CreateFunctionType to take parameters
https://github.com/Michael137 commented:
My original idea was that we can just call `CalculateNumChildren` and bubble up
that error when something goes wrong. But if we're just going to consume it
anyway, then keeping them as `CalculateNumChildrenIgnoringErrors` seems fine
(though I would stil
charles-zablit wrote:
> But if we're just going to consume it anyway, then keeping them as
> `CalculateNumChildrenIgnoringErrors` seems fine (though I would still prefer
> bubbling up errors, but that brings us back to the discussion about how we
> should be using `llvm::Expected`)
Agreed, th
https://github.com/vogelsgesang updated
https://github.com/llvm/llvm-project/pull/141516
>From 06a2754bfbd434b6a957c6b252be15aac2bf8173 Mon Sep 17 00:00:00 2001
From: Adrian Vogelsgesang
Date: Fri, 23 May 2025 01:20:46 +
Subject: [PATCH] [lldb] Show coro_frame in `std::coroutine_handle` pre
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/142620
>From f69c768795f647d97b85bce5416b2302eb6b5a6d Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Tue, 3 Jun 2025 15:51:40 +0100
Subject: [PATCH 1/2] [lldb][TypeSystem][NFC] CreateFunctionType to take
paramet
Author: Dmitry Vasilyev
Date: 2025-06-03T20:18:16+04:00
New Revision: a90145e0282fb9eef0ad9ff61f505aff4e30c01d
URL:
https://github.com/llvm/llvm-project/commit/a90145e0282fb9eef0ad9ff61f505aff4e30c01d
DIFF:
https://github.com/llvm/llvm-project/commit/a90145e0282fb9eef0ad9ff61f505aff4e30c01d.dif
https://github.com/slydiman closed
https://github.com/llvm/llvm-project/pull/142573
___
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-03T09:19:50-07:00
New Revision: a9032712c453bda70449dedcaf00bead0fea6e88
URL:
https://github.com/llvm/llvm-project/commit/a9032712c453bda70449dedcaf00bead0fea6e88
DIFF:
https://github.com/llvm/llvm-project/commit/a9032712c453bda70449dedcaf00bead0fea6e88.d
https://github.com/JDevlieghere closed
https://github.com/llvm/llvm-project/pull/142424
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/JDevlieghere approved this pull request.
https://github.com/llvm/llvm-project/pull/142587
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -33,6 +33,23 @@ static void SendThreadExitedEvent(DAP &dap, lldb::tid_t tid)
{
dap.SendJSON(llvm::json::Value(std::move(event)));
}
+void SendAdditionalCapabilities(DAP &dap) {
+ if (dap.target.IsValid()) {
+// FIXME: stepInTargets request is only supported by the x
https://github.com/JDevlieghere approved this pull request.
https://github.com/llvm/llvm-project/pull/142583
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -33,6 +33,23 @@ static void SendThreadExitedEvent(DAP &dap, lldb::tid_t tid)
{
dap.SendJSON(llvm::json::Value(std::move(event)));
}
+void SendAdditionalCapabilities(DAP &dap) {
da-viper wrote:
Will update was struggling to think of a good name.
https:/
Author: Dave Lee
Date: 2025-06-03T09:38:22-07:00
New Revision: 20ca8958604dc26d5b480cf9109b861d05341ac8
URL:
https://github.com/llvm/llvm-project/commit/20ca8958604dc26d5b480cf9109b861d05341ac8
DIFF:
https://github.com/llvm/llvm-project/commit/20ca8958604dc26d5b480cf9109b861d05341ac8.diff
LOG:
@@ -414,6 +415,34 @@ bool fromJSON(const llvm::json::Value &,
SteppingGranularity &,
llvm::json::Path);
llvm::json::Value toJSON(const SteppingGranularity &);
+/// A `StepInTarget` can be used in the `stepIn` request and determines into
+/// which single target
@@ -33,6 +33,23 @@ static void SendThreadExitedEvent(DAP &dap, lldb::tid_t tid)
{
dap.SendJSON(llvm::json::Value(std::move(event)));
}
+void SendAdditionalCapabilities(DAP &dap) {
+ if (dap.target.IsValid()) {
da-viper wrote:
I wanted to do it like that b
@@ -78,3 +78,31 @@ def test_basic(self):
leaf_frame = self.dap_server.get_stackFrame()
self.assertIsNotNone(leaf_frame, "expect a leaf frame")
self.assertEqual(step_in_targets[1]["label"], leaf_frame["name"])
+
+def test_supported_capability(self):
+
@@ -78,3 +78,31 @@ def test_basic(self):
leaf_frame = self.dap_server.get_stackFrame()
self.assertIsNotNone(leaf_frame, "expect a leaf frame")
self.assertEqual(step_in_targets[1]["label"], leaf_frame["name"])
+
+def test_supported_capability(self):
+
@@ -33,6 +33,23 @@ static void SendThreadExitedEvent(DAP &dap, lldb::tid_t tid)
{
dap.SendJSON(llvm::json::Value(std::move(event)));
}
+void SendAdditionalCapabilities(DAP &dap) {
+ if (dap.target.IsValid()) {
ashgti wrote:
We can always update it in the
https://github.com/chelcassanova updated
https://github.com/llvm/llvm-project/pull/142051
>From b47eaa64397da7ea5d2a7ca46bea4513a37755f0 Mon Sep 17 00:00:00 2001
From: Chelsea Cassanova
Date: Wed, 28 May 2025 15:45:45 -0700
Subject: [PATCH 1/2] [lldb][headers] Create Python script to fix up fra
https://github.com/bulbazord approved this pull request.
https://github.com/llvm/llvm-project/pull/142583
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/JDevlieghere approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/141116
___
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/142587
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/JDevlieghere commented:
I think this could benefit from following the pattern of the other scripts,
which is running on a single file and having CMake doing the globbing,
presumably that means the build system can do all of this in parallel instead
of making this a single-th
jimingham wrote:
This also seems like an awkward way to do what we've wanted for a while, which
is the equivalent of stop hooks for "launch", "attach" and "module loaded",
since you have to do all three, even if you only wanted to do one of the set.
https://github.com/llvm/llvm-project/pull/14
jimingham wrote:
I changed the stop-hooks recently so they optionally fire when lldb first gets
control of the process, so you can already write python code that intervenes
when your "did_attach" and "did_launch" callbacks fire.
https://github.com/llvm/llvm-project/pull/142514
https://github.com/da-viper updated
https://github.com/llvm/llvm-project/pull/142439
>From 69efc48d722d18600018f25db0f9ea46b9fd1d97 Mon Sep 17 00:00:00 2001
From: Ebuka Ezike
Date: Wed, 28 May 2025 14:02:01 +0100
Subject: [PATCH 1/4] [lldb-dap] Use structured types for stepInTargets request
--
@@ -414,6 +415,34 @@ bool fromJSON(const llvm::json::Value &,
SteppingGranularity &,
llvm::json::Path);
llvm::json::Value toJSON(const SteppingGranularity &);
+/// A `StepInTarget` can be used in the `stepIn` request and determines into
+/// which single target
https://github.com/ashgti approved this pull request.
LGTM!
https://github.com/llvm/llvm-project/pull/142439
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/da-viper updated
https://github.com/llvm/llvm-project/pull/142439
>From 69efc48d722d18600018f25db0f9ea46b9fd1d97 Mon Sep 17 00:00:00 2001
From: Ebuka Ezike
Date: Wed, 28 May 2025 14:02:01 +0100
Subject: [PATCH 1/5] [lldb-dap] Use structured types for stepInTargets request
--
jimingham wrote:
But if you don't care so much about shared state, then I think a better way of
doing what you want is just to add `target module-hook add`...
https://github.com/llvm/llvm-project/pull/142514
___
lldb-commits mailing list
lldb-commits@
jimingham wrote:
One thing that's nicer about this approach than independent callbacks for each
of the hooks is that it allows you to group the three callbacks in the same
class, so that you can more easily keep shared state.
But that's a general problem with these affordances. For instance,
https://github.com/da-viper created
https://github.com/llvm/llvm-project/pull/142652
The current implementation hides any possible error from performing a step
command.
>From 7e940dcb0cfde1bc9be73c7cf2a40ba7f08d12e5 Mon Sep 17 00:00:00 2001
From: Ebuka Ezike
Date: Mon, 2 Jun 2025 17:07:50 +01
@@ -414,6 +415,34 @@ bool fromJSON(const llvm::json::Value &,
SteppingGranularity &,
llvm::json::Path);
llvm::json::Value toJSON(const SteppingGranularity &);
+/// A `StepInTarget` can be used in the `stepIn` request and determines into
+/// which single target
https://github.com/da-viper edited
https://github.com/llvm/llvm-project/pull/142652
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/da-viper edited
https://github.com/llvm/llvm-project/pull/142652
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
chelcassanova wrote:
> I think this could benefit from following the pattern of the other scripts,
> which is running on a single file and having CMake doing the globbing,
> presumably that means the build system can do all of this in parallel instead
> of making this a single-threaded operati
https://github.com/medismailben approved this pull request.
Hey @clayborg, this is pretty cool. I'm glad you were able to use and extend
the ScriptedPythonInterface to implement this, hopefully it wasn't too
complicated. LGTM!
https://github.com/llvm/llvm-project/pull/142514
__
https://github.com/bulbazord approved this pull request.
https://github.com/llvm/llvm-project/pull/142620
___
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: Ebuka Ezike (da-viper)
Changes
The current implementation hides any possible error from performing a step
command.
---
Full diff: https://github.com/llvm/llvm-project/pull/142652.diff
3 Files Affected:
- (modified) lldb/tools/lldb-dap/H
https://github.com/vogelsgesang edited
https://github.com/llvm/llvm-project/pull/142514
___
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/141116
___
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/139170
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/ashgti approved this pull request.
For these three request handlers, should we also add a sanity check of:
```
if (!SBDebugger::StateIsStoppedState(process.GetState()))
return make_error();
```
To the `Run(...)` so we can get a specific error if the process isn't stoppe
https://github.com/ashgti approved this pull request.
LGTM!
https://github.com/llvm/llvm-project/pull/142652
___
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/139170
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/da-viper updated
https://github.com/llvm/llvm-project/pull/142652
>From 7e940dcb0cfde1bc9be73c7cf2a40ba7f08d12e5 Mon Sep 17 00:00:00 2001
From: Ebuka Ezike
Date: Mon, 2 Jun 2025 17:07:50 +0100
Subject: [PATCH 1/2] [lldb-dap] Forward any error from stepping.
---
lldb/tools/l
https://github.com/JDevlieghere updated
https://github.com/llvm/llvm-project/pull/142489
>From 6076f7778f3f10d7360d8f0b156992809de73094 Mon Sep 17 00:00:00 2001
From: Jonas Devlieghere
Date: Tue, 3 Jun 2025 10:44:43 -0700
Subject: [PATCH] [lldb] Fix data race in statusline format handling
This
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/Core/Debugger.h lldb/include/lldb
bulbazord wrote:
> This seems like a generic module loading observer. I don't see anything JIT
> specific about it. Not saying a generic module loading observer is not a good
> idea. But calling it a JITLoader seems pretty confusing to me.
+1 on the name. The design seems very general, so mayb
medismailben wrote:
> Hey @clayborg, this is pretty cool. I'm glad you were able to use and extend
> the ScriptedPythonInterface to implement this, hopefully it wasn't too
> complicated. LGTM!
I meant LGTM on the scripting side of things. I still thing we should address
Jim's comments:
> For
https://github.com/JDevlieghere approved this pull request.
Nice
https://github.com/llvm/llvm-project/pull/142652
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/chelcassanova closed
https://github.com/llvm/llvm-project/pull/141116
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: Chelsea Cassanova
Date: 2025-06-03T13:06:46-07:00
New Revision: d204aa9deb72b8dcaf5e5b5550871d0ebe982825
URL:
https://github.com/llvm/llvm-project/commit/d204aa9deb72b8dcaf5e5b5550871d0ebe982825
DIFF:
https://github.com/llvm/llvm-project/commit/d204aa9deb72b8dcaf5e5b5550871d0ebe982825.d
https://github.com/JDevlieghere updated
https://github.com/llvm/llvm-project/pull/142489
>From 6076f7778f3f10d7360d8f0b156992809de73094 Mon Sep 17 00:00:00 2001
From: Jonas Devlieghere
Date: Tue, 3 Jun 2025 10:44:43 -0700
Subject: [PATCH 1/2] [lldb] Fix data race in statusline format handling
https://github.com/JDevlieghere commented:
Can you add a `Thread` serialization/deserialization unit test?
https://github.com/llvm/llvm-project/pull/142510
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman
https://github.com/JDevlieghere edited
https://github.com/llvm/llvm-project/pull/142510
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/kastiglione closed
https://github.com/llvm/llvm-project/pull/142215
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -8,72 +8,43 @@
#include "DAP.h"
#include "EventHelper.h"
-#include "JSONUtils.h"
+#include "Protocol/ProtocolRequests.h"
+#include "ProtocolUtils.h"
#include "RequestHandler.h"
+#include "lldb/API/SBDebugger.h"
+#include "lldb/API/SBDefines.h"
+#include "llvm/Support/Error
@@ -116,77 +116,79 @@ void SendProcessEvent(DAP &dap, LaunchMethod
launch_method) {
// Send a thread stopped event for all threads as long as the process
// is stopped.
-void SendThreadStoppedEvent(DAP &dap) {
+void SendThreadStoppedEvent(DAP &dap, bool on_entry) {
--
@@ -414,6 +415,34 @@ bool fromJSON(const llvm::json::Value &,
SteppingGranularity &,
llvm::json::Path);
llvm::json::Value toJSON(const SteppingGranularity &);
+/// A `StepInTarget` can be used in the `stepIn` request and determines into
+/// which single target
https://github.com/chelcassanova updated
https://github.com/llvm/llvm-project/pull/142051
>From b47eaa64397da7ea5d2a7ca46bea4513a37755f0 Mon Sep 17 00:00:00 2001
From: Chelsea Cassanova
Date: Wed, 28 May 2025 15:45:45 -0700
Subject: [PATCH 1/2] [lldb][headers] Create Python script to fix up fra
https://github.com/JDevlieghere approved this pull request.
LGTM. Might be worth making this match the other scripts and doing this once
instead of line-by-line.
https://github.com/llvm/llvm-project/pull/138028
___
lldb-commits mailing list
lldb-commi
https://github.com/da-viper closed
https://github.com/llvm/llvm-project/pull/134562
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
jimingham wrote:
This seems like a generic module loading observer. I don't see anything JIT
specific about it. Not saying a generic module loading observer is not a good
idea. But calling it a JITLoader seems pretty confusing to me.
https://github.com/llvm/llvm-project/pull/142514
https://github.com/JDevlieghere approved this pull request.
https://github.com/llvm/llvm-project/pull/142439
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/rupprecht created
https://github.com/llvm/llvm-project/pull/142676
Test case added by f669b9c3eca9438d33259aefb8156f977f1df382 / #137793. Note
that the `DemanglingParts` case above also frees the buffer; this new test case
is inconsistent.
>From 270b38ccfe58011c2b22e89e5b57
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Jordan Rupprecht (rupprecht)
Changes
Test case added by f669b9c3eca9438d33259aefb8156f977f1df382 / #137793.
Note that the `DemanglingParts` case above also frees the buffer; this new test
case is inconsistent.
---
Full diff: https://githu
https://github.com/JDevlieghere approved this pull request.
LGTM. Maybe an opportunity to use a unique_ptr with a custom deleter? Something
like:
```
struct TrackingOutputBufferDeleter {
void operator()(TrackingOutputBuffer* TOB) {
if (!TOB)
return;
std::free(TOB->getBuffer());
https://github.com/JDevlieghere approved this pull request.
https://github.com/llvm/llvm-project/pull/142129
___
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: Ebuka Ezike (da-viper)
Changes
Fixes #142581
---
Full diff: https://github.com/llvm/llvm-project/pull/142684.diff
6 Files Affected:
- (modified) lldb/source/Commands/CommandObjectProcess.cpp (+1-1)
- (modified) lldb/source/Symbol/SaveC
Author: Michael Buch
Date: 2025-06-03T23:37:39+01:00
New Revision: cb56e15bb3e92c8aab2b7fd74a7683ffd83ac10b
URL:
https://github.com/llvm/llvm-project/commit/cb56e15bb3e92c8aab2b7fd74a7683ffd83ac10b
DIFF:
https://github.com/llvm/llvm-project/commit/cb56e15bb3e92c8aab2b7fd74a7683ffd83ac10b.diff
Author: Ebuka Ezike
Date: 2025-06-03T23:40:00+01:00
New Revision: 33fae0840562ae7e93dd7b4bc6dd4a41150eee01
URL:
https://github.com/llvm/llvm-project/commit/33fae0840562ae7e93dd7b4bc6dd4a41150eee01
DIFF:
https://github.com/llvm/llvm-project/commit/33fae0840562ae7e93dd7b4bc6dd4a41150eee01.diff
L
1 - 100 of 128 matches
Mail list logo