https://github.com/jimingham updated
https://github.com/llvm/llvm-project/pull/171236
>From b9ebcb580ea13ee2f1a6491e03249062889eb12d Mon Sep 17 00:00:00 2001
From: Jim Ingham
Date: Mon, 8 Dec 2025 16:26:51 -0800
Subject: [PATCH 1/2] Add a _regexp-break-add and some more tests for the b
alias.
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Sergei Druzhkov (DrSergei)
Changes
This patch migrates `locations` request into structured types and adds test for
it.
---
Full diff: https://github.com/llvm/llvm-project/pull/171099.diff
5 Files Affected:
- (modified) lldb/tools/lldb-d
https://github.com/DrSergei created
https://github.com/llvm/llvm-project/pull/171099
This patch migrates `locations` request into structured types and adds test for
it.
>From 5a8427e3a9e515a47d68588d05d6147660c98b2f Mon Sep 17 00:00:00 2001
From: Druzhkov Sergei
Date: Sun, 7 Dec 2025 17:23:49
https://github.com/DrSergei updated
https://github.com/llvm/llvm-project/pull/171096
>From 679bb3f40b7c8961841b8d73bae0c8fa7e8cd904 Mon Sep 17 00:00:00 2001
From: Druzhkov Sergei
Date: Sun, 7 Dec 2025 13:55:04 +0300
Subject: [PATCH 1/2] [lldb-dap] Migrate pause request to structured types
---
Nicolai =?utf-8?q?Hähnle?= ,jimingham
,Matt Arsenault ,Priyanshu
Kumar <[email protected]>,Aiden Grossman
,Maksim
Levental
Message-ID:
In-Reply-To:
https://github.com/anonymouspc converted_to_draft
https://github.com/llvm/llvm-project/pull/171109
_
Nicolai =?utf-8?q?Hähnle?= ,jimingham
,Matt Arsenault ,Priyanshu
Kumar <[email protected]>,Aiden Grossman
,Maksim
Levental
Message-ID:
In-Reply-To:
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this lo
https://github.com/vogelsgesang auto_merge_enabled
https://github.com/llvm/llvm-project/pull/167803
___
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/Michael137 created
https://github.com/llvm/llvm-project/pull/171138
We used to set it to `true` up until recently, see TBD. That's incorrect
because `SuppressInlineNamespace` is actually an enum. What probably happened
is that `SuppressInlineNamespace` used to be a boolean b
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Michael Buch (Michael137)
Changes
We used to set it to `true` up until recently, see TBD. That's incorrect
because `SuppressInlineNamespace` is actually an enum. What probably happened
is that `SuppressInlineNamespace` used to be a boolean
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/171138
___
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/JDevlieghere updated
https://github.com/llvm/llvm-project/pull/170772
>From 38d27e4313ba38b5a54ffaab2ee68f53b21a3f2e Mon Sep 17 00:00:00 2001
From: Jonas Devlieghere
Date: Thu, 4 Dec 2025 12:00:11 -0800
Subject: [PATCH 1/4] Move GetBuildConfiguration from SBDebugger -> Debugg
https://github.com/JDevlieghere approved this pull request.
https://github.com/llvm/llvm-project/pull/171096
___
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -8,21 +8,71 @@
#include "CommandObjectVersion.h"
+#include "lldb/Core/Debugger.h"
#include "lldb/Interpreter/CommandReturnObject.h"
#include "lldb/Version/Version.h"
using namespace lldb;
using namespace lldb_private;
-// CommandObjectVersion
+#define LLDB_OPTIONS_
Author: Jonas Devlieghere
Date: 2025-12-08T10:11:39-08:00
New Revision: bc9f96a5e62d0b89588092343d137995811a9b80
URL:
https://github.com/llvm/llvm-project/commit/bc9f96a5e62d0b89588092343d137995811a9b80
DIFF:
https://github.com/llvm/llvm-project/commit/bc9f96a5e62d0b89588092343d137995811a9b80.d
https://github.com/JDevlieghere closed
https://github.com/llvm/llvm-project/pull/170772
___
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -1184,6 +1184,41 @@ struct EvaluateResponseBody {
};
llvm::json::Value toJSON(const EvaluateResponseBody &);
+/// Arguments for `locations` request.
+struct LocationsArguments {
+ /// Location reference to resolve.
+ uint64_t locationReference = LLDB_DAP_INVALID_VALUE_LOC
@@ -18,167 +19,64 @@
namespace lldb_dap {
-// "LocationsRequest": {
-// "allOf": [ { "$ref": "#/definitions/Request" }, {
-// "type": "object",
-// "description": "Looks up information about a location reference
-// previously returned by the deb
@@ -18,167 +19,64 @@
namespace lldb_dap {
-// "LocationsRequest": {
-// "allOf": [ { "$ref": "#/definitions/Request" }, {
-// "type": "object",
-// "description": "Looks up information about a location reference
-// previously returned by the deb
@@ -18,167 +19,64 @@
namespace lldb_dap {
-// "LocationsRequest": {
-// "allOf": [ { "$ref": "#/definitions/Request" }, {
-// "type": "object",
-// "description": "Looks up information about a location reference
-// previously returned by the deb
@@ -18,167 +19,64 @@
namespace lldb_dap {
-// "LocationsRequest": {
-// "allOf": [ { "$ref": "#/definitions/Request" }, {
-// "type": "object",
-// "description": "Looks up information about a location reference
-// previously returned by the deb
@@ -1184,6 +1184,41 @@ struct EvaluateResponseBody {
};
llvm::json::Value toJSON(const EvaluateResponseBody &);
+/// Arguments for `locations` request.
+struct LocationsArguments {
+ /// Location reference to resolve.
+ uint64_t locationReference = LLDB_DAP_INVALID_VALUE_LOC
https://github.com/DrSergei approved this pull request.
https://github.com/llvm/llvm-project/pull/170890
___
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/DrSergei created
https://github.com/llvm/llvm-project/pull/171096
This patch migrates `pause` request into structured types and adds test for it.
>From 679bb3f40b7c8961841b8d73bae0c8fa7e8cd904 Mon Sep 17 00:00:00 2001
From: Druzhkov Sergei
Date: Sun, 7 Dec 2025 13:55:04 +030
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Sergei Druzhkov (DrSergei)
Changes
This patch migrates `pause` request into structured types and adds test for it.
---
Full diff: https://github.com/llvm/llvm-project/pull/171096.diff
5 Files Affected:
- (modified) lldb/tools/lldb-dap/Ha
@@ -1184,6 +1184,41 @@ struct EvaluateResponseBody {
};
llvm::json::Value toJSON(const EvaluateResponseBody &);
+/// Arguments for `locations` request.
+struct LocationsArguments {
+ /// Location reference to resolve.
+ uint64_t locationReference = LLDB_DAP_INVALID_VALUE_LOC
https://github.com/DrSergei updated
https://github.com/llvm/llvm-project/pull/171099
>From 5a8427e3a9e515a47d68588d05d6147660c98b2f Mon Sep 17 00:00:00 2001
From: Druzhkov Sergei
Date: Sun, 7 Dec 2025 17:23:49 +0300
Subject: [PATCH 1/2] [lldb-dap] Migrate locations request to structured types
https://github.com/DavidSpickett edited
https://github.com/llvm/llvm-project/pull/170932
___
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -8,53 +8,19 @@
#include "DAP.h"
#include "EventHelper.h"
-#include "JSONUtils.h"
+#include "Protocol/ProtocolRequests.h"
#include "RequestHandler.h"
namespace lldb_dap {
-// "PauseRequest": {
-// "allOf": [ { "$ref": "#/definitions/Request" }, {
-// "type": "obj
https://github.com/DavidSpickett edited
https://github.com/llvm/llvm-project/pull/170932
___
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
DavidSpickett wrote:
> We're describing it as empty, but the documentation does imply that empty can
> equal success. This confuses me a bit because if I pass an SBError to a
> function that is expected to return an Status it should populate the
> underlying status and thus IsValid() would ret
da-viper wrote:
>My intention here is that a big timeout can cause problems on CI when flaky
>tests are failed (e.g. waiting for events)
The goal is to make the test no longer flaky, IMO if there is a wait for event
longer than 50 seconds there is something else wrong with the test than the
t
DavidSpickett wrote:
> Thus that makes SBError kinda weird where IsValid() only returns false if it
> hasn't been exercised?
Yes.
Though if you don't have the API source code to read, it can look like
IsValid() can be false *even if* it has been exercised.
https://github.com/llvm/llvm-projec
Author: Sergei Druzhkov
Date: 2025-12-08T21:35:24+03:00
New Revision: 371da58cfa05caa6ff654e538b913b059d9a1f44
URL:
https://github.com/llvm/llvm-project/commit/371da58cfa05caa6ff654e538b913b059d9a1f44
DIFF:
https://github.com/llvm/llvm-project/commit/371da58cfa05caa6ff654e538b913b059d9a1f44.dif
https://github.com/da-viper approved this pull request.
https://github.com/llvm/llvm-project/pull/171096
___
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -1646,326 +1730,317 @@ def __str__(self):
return f"lldb-dap returned non-zero exit status {self.returncode}."
-def attach_options_specified(options):
-if options.pid is not None:
+def attach_options_specified(opts):
+if opts.pid is not None:
re
https://github.com/DrSergei updated
https://github.com/llvm/llvm-project/pull/171099
>From c2d2546e4631f47150db7a0957d61f108d633760 Mon Sep 17 00:00:00 2001
From: Druzhkov Sergei
Date: Sun, 7 Dec 2025 17:23:49 +0300
Subject: [PATCH 1/3] [lldb-dap] Migrate locations request to structured types
@@ -695,4 +695,23 @@ llvm::json::Value toJSON(const EvaluateResponseBody &Body)
{
return result;
}
+bool fromJSON(const llvm::json::Value &Params, LocationsArguments &Args,
+ llvm::json::Path Path) {
+ json::ObjectMapper O(Params, Path);
+ return O && O.map("
https://github.com/DrSergei closed
https://github.com/llvm/llvm-project/pull/171096
___
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: David Spickett
Date: 2025-12-08T13:53:55Z
New Revision: 7fbd443491db62a3fe316f9cc4e3e47036f5730b
URL:
https://github.com/llvm/llvm-project/commit/7fbd443491db62a3fe316f9cc4e3e47036f5730b
DIFF:
https://github.com/llvm/llvm-project/commit/7fbd443491db62a3fe316f9cc4e3e47036f5730b.diff
LOG
mizvekov wrote:
As I pointed out in the original patch, maybe there was a confusion on my part
where I assumed we would have a three valued enum, but the direction actually
would make it four valued, and these would be controlling the suppression of
the keyword in orthogonal places.
Can you c
https://github.com/ashgti approved this pull request.
https://github.com/llvm/llvm-project/pull/171099
___
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -695,4 +695,23 @@ llvm::json::Value toJSON(const EvaluateResponseBody &Body)
{
return result;
}
+bool fromJSON(const llvm::json::Value &Params, LocationsArguments &Args,
+ llvm::json::Path Path) {
+ json::ObjectMapper O(Params, Path);
+ return O && O.map("
https://github.com/ashgti approved this pull request.
https://github.com/llvm/llvm-project/pull/171096
___
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Michael Buch (Michael137)
Changes
Split out from https://github.com/llvm/llvm-project/pull/169445 where we
introduce an additional mode to `SuppressTagKeywordMode`.
Prepares `PrintingPolicy::SuppressTagKeyword` to take enum va
https://github.com/Michael137 created
https://github.com/llvm/llvm-project/pull/171160
Split out from https://github.com/llvm/llvm-project/pull/169445 where we
introduce an additional mode to `SuppressTagKeywordMode`.
Prepares `PrintingPolicy::SuppressTagKeyword` to take enum values. Currently
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/171160
>From b111e73c03d84d1315ba9e52c4e6348d5dc866ca Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Fri, 5 Dec 2025 15:10:42 +0800
Subject: [PATCH 1/2] [clang][TypePrinter][NFC] Turn SuppressTagKeyword into an
@@ -121,6 +123,7 @@ struct PrintingPolicy {
/// \endcode
LLVM_PREFERRED_TYPE(bool)
unsigned SuppressTagKeyword : 1;
+ unsigned SuppressTagKeywordInAnonymousTagNames : 1;
mizvekov wrote:
Yeah hold on, I thought this would be a three valued enum, but I se
https://github.com/vogelsgesang updated
https://github.com/llvm/llvm-project/pull/167803
>From 27934f7d6c5ed7f421c7864918c5f2d398cc7fe7 Mon Sep 17 00:00:00 2001
From: Adrian Vogelsgesang
Date: Thu, 13 Nov 2025 01:10:09 +
Subject: [PATCH] [lldb] Fix command line of `target frame-provider reg
https://github.com/adrian-prantl approved this pull request.
You may need to add a minimum clang version to these test decorators?
https://github.com/llvm/llvm-project/pull/170804
___
lldb-commits mailing list
[email protected]
https://lists.
https://github.com/adrian-prantl approved this pull request.
https://github.com/llvm/llvm-project/pull/171138
___
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/felipepiovezan closed
https://github.com/llvm/llvm-project/pull/170834
___
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: Felipe de Azevedo Piovezan
Date: 2025-12-08T16:59:32Z
New Revision: f27fbca37c4a66d4e237bae1256d775ca7471cb9
URL:
https://github.com/llvm/llvm-project/commit/f27fbca37c4a66d4e237bae1256d775ca7471cb9
DIFF:
https://github.com/llvm/llvm-project/commit/f27fbca37c4a66d4e237bae1256d775ca7471c
Author: Adrian Vogelsgesang
Date: 2025-12-08T13:14:41Z
New Revision: 7c832fca5374cde2804cebc2ba3c5ad635fb76a1
URL:
https://github.com/llvm/llvm-project/commit/7c832fca5374cde2804cebc2ba3c5ad635fb76a1
DIFF:
https://github.com/llvm/llvm-project/commit/7c832fca5374cde2804cebc2ba3c5ad635fb76a1.diff
https://github.com/vogelsgesang closed
https://github.com/llvm/llvm-project/pull/167803
___
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/ashgti approved this pull request.
I'm okay with this if we think it will improve stability.
However, I think there may still be a few places in tests where we wait for a
negative assertion to time out. We may need to make sure we find anymore of
those and make them positive
@@ -71,6 +71,11 @@ class Interpreter : Visitor {
std::shared_ptr ctx,
const IntegerLiteralNode *literal);
+ llvm::Expected
cmtice wrote:
Comment added. Yes they are all the reference parameters are 'out' parameters,
i.e.
@@ -46,6 +46,13 @@ enum class CastKind {
eNone,///< Type promotion casting
};
+/// Promotions allowed for type casts in DIL.
+enum CastPromoKind {
cmtice wrote:
Done.
https://github.com/llvm/llvm-project/pull/170332
@@ -740,16 +776,236 @@ Interpreter::Visit(const BooleanLiteralNode *node) {
return ValueObject::CreateValueObjectFromBool(m_target, value, "result");
}
+llvm::Expected
+Interpreter::VerifyCastType(lldb::ValueObjectSP &operand, CompilerType
&op_type,
+
@@ -21,6 +21,42 @@
namespace lldb_private::dil {
+lldb::ValueObjectSP
+GetDynamicOrSyntheticValue(lldb::ValueObjectSP in_valobj_sp,
+ lldb::DynamicValueType use_dynamic,
+ bool use_synthetic) {
+ Status error;
+ if (!in_va
https://github.com/cmtice updated
https://github.com/llvm/llvm-project/pull/170332
>From 8aab34e35f8622422e1b82147d059b1b6d1ce161 Mon Sep 17 00:00:00 2001
From: Caroline Tice
Date: Tue, 2 Dec 2025 09:16:01 -0800
Subject: [PATCH 1/3] [LLDB] Add type casting to DIL, part 2 or 3
This PR implement
https://github.com/cmtice edited
https://github.com/llvm/llvm-project/pull/170332
___
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: Michael Buch
Date: 2025-12-09T07:48:20Z
New Revision: f53f6f725085996662055a669407f388c3ba2486
URL:
https://github.com/llvm/llvm-project/commit/f53f6f725085996662055a669407f388c3ba2486
DIFF:
https://github.com/llvm/llvm-project/commit/f53f6f725085996662055a669407f388c3ba2486.diff
LOG:
https://github.com/Michael137 closed
https://github.com/llvm/llvm-project/pull/170804
___
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/Michael137 approved this pull request.
https://github.com/llvm/llvm-project/pull/171213
___
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Michael137 wrote:
> You may need to add a minimum clang version to these test decorators?
Yea probably, but I was going to see what the matrix bot looks like after
merging this and then adjust the decorators
https://github.com/llvm/llvm-project/pull/170804
_
Author: Michael Buch
Date: 2025-12-09T07:47:39Z
New Revision: 0bb0e26aaf25f0f34bd2c5f2bc90540639fda4b7
URL:
https://github.com/llvm/llvm-project/commit/0bb0e26aaf25f0f34bd2c5f2bc90540639fda4b7
DIFF:
https://github.com/llvm/llvm-project/commit/0bb0e26aaf25f0f34bd2c5f2bc90540639fda4b7.diff
LOG:
https://github.com/Michael137 closed
https://github.com/llvm/llvm-project/pull/171138
___
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
67 matches
Mail list logo
