Author: David Spickett
Date: 2025-08-06T09:31:09+01:00
New Revision: 4077e66432a1d17543d0cef4b5a9280caff6e974
URL:
https://github.com/llvm/llvm-project/commit/4077e66432a1d17543d0cef4b5a9280caff6e974
DIFF:
https://github.com/llvm/llvm-project/commit/4077e66432a1d17543d0cef4b5a9280caff6e974.diff
https://github.com/DavidSpickett closed
https://github.com/llvm/llvm-project/pull/151973
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
DavidSpickett wrote:
Feels like someone stored a uint64_t into a void* that's 32-bit on Arm 32-bit.
Thanks for reverting, I'm looking into it now.
https://github.com/llvm/llvm-project/pull/151460
___
lldb-commits mailing list
lldb-commits@lists.llvm.o
https://github.com/labath approved this pull request.
https://github.com/llvm/llvm-project/pull/151973
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: Michael Buch
Date: 2025-08-06T09:17:33+01:00
New Revision: 753885eaaf8745a64f502a3a65fa89456d632cd5
URL:
https://github.com/llvm/llvm-project/commit/753885eaaf8745a64f502a3a65fa89456d632cd5
DIFF:
https://github.com/llvm/llvm-project/commit/753885eaaf8745a64f502a3a65fa89456d632cd5.diff
mstorsjo wrote:
> @chelcassanova @mstorsjo
>
> > But this seems to assume that everybody's cross builds are done in some
> > specific sequence, with regards to how LLVM_HOST_TRIPLE and
> > LLVM_DEFAULT_TARGET_TRIPLE are set (in my case, I just cross build with one
> > single step, and both of
@@ -2482,6 +2485,134 @@ bool SymbolFileDWARF::ResolveFunction(const DWARFDIE
&orig_die,
return false;
}
+static int ClangToItaniumCtorKind(clang::CXXCtorType kind) {
+ switch (kind) {
+ case clang::CXXCtorType::Ctor_Complete:
+return 1;
+ case clang::CXXCtorType::Cto
@@ -250,11 +250,52 @@ static unsigned GetCXXMethodCVQuals(const DWARFDIE
&subprogram,
return cv_quals;
}
+static const char *GetMangledOrStructorName(const DWARFDIE &die) {
+ const char *name = die.GetMangledName(/*substitute_name_allowed*/ false);
+ if (name)
+return
https://github.com/Nerixyz created
https://github.com/llvm/llvm-project/pull/152295
This adds the ability for functions to be matched by their basename.
Before, the globals were searched for the name. This works if the full name is
available but fails for basenames.
PDB only includes the full
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
@@ -301,6 +301,14 @@ Variable CreateVariable(lldb::SBValue v, int64_t var_ref,
bool format_hex,
if (lldb::addr_t addr = v.GetLoadAddress(); addr != LLDB_INVALID_ADDRESS)
var.memoryReference = addr;
+ bool is_readonly = v.GetType().IsAggregateType() ||
@@ -301,6 +301,14 @@ Variable CreateVariable(lldb::SBValue v, int64_t var_ref,
bool format_hex,
if (lldb::addr_t addr = v.GetLoadAddress(); addr != LLDB_INVALID_ADDRESS)
var.memoryReference = addr;
+ bool is_readonly = v.GetType().IsAggregateType() ||
@@ -2482,6 +2485,134 @@ bool SymbolFileDWARF::ResolveFunction(const DWARFDIE
&orig_die,
return false;
}
+static int ClangToItaniumCtorKind(clang::CXXCtorType kind) {
+ switch (kind) {
+ case clang::CXXCtorType::Ctor_Complete:
+return 1;
+ case clang::CXXCtorType::Cto
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/149827
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -2482,6 +2485,134 @@ bool SymbolFileDWARF::ResolveFunction(const DWARFDIE
&orig_die,
return false;
}
+static int ClangToItaniumCtorKind(clang::CXXCtorType kind) {
+ switch (kind) {
+ case clang::CXXCtorType::Ctor_Complete:
+return 1;
+ case clang::CXXCtorType::Cto
https://github.com/kastiglione created
https://github.com/llvm/llvm-project/pull/152334
None
>From e7ea7c066193c52700035a87a4ee08950c21da99 Mon Sep 17 00:00:00 2001
From: Dave Lee
Date: Tue, 5 Aug 2025 15:39:17 -0700
Subject: [PATCH] [lldb] Use const ref for looping over frame recognizers (NFC
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/149827
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/149827
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/149827
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/kuhar commented:
Can you move the ADT changes to a separate PR?
https://github.com/llvm/llvm-project/pull/152308
___
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: Jonas Devlieghere (JDevlieghere)
Changes
This is a continuation of #152188, which started splitting up the MCP
implementation into a generic implementation in Protocol/MCP that will be
shared between LLDB and lldb-mcp.
For now I kept all
vvereschaka wrote:
>Right - I see. But why does this build configuration have to affect whether it
>should build lldb-rpc-gen or not? Is it because the clang in this build can't
>manage to process the lldb headers (which are built for the current host)?
Right, "the clang in this build can't ma
https://github.com/cmtice updated
https://github.com/llvm/llvm-project/pull/151605
>From 0d51e89d663bfd85f3b503452d9c6221be0f7fb7 Mon Sep 17 00:00:00 2001
From: Caroline Tice
Date: Thu, 31 Jul 2025 15:07:25 -0700
Subject: [PATCH 1/2] [LLDB] Update DIL handling of array subscripting.
This updat
https://github.com/cmtice updated
https://github.com/llvm/llvm-project/pull/151605
>From 0d51e89d663bfd85f3b503452d9c6221be0f7fb7 Mon Sep 17 00:00:00 2001
From: Caroline Tice
Date: Thu, 31 Jul 2025 15:07:25 -0700
Subject: [PATCH 1/3] [LLDB] Update DIL handling of array subscripting.
This updat
https://github.com/walter-erquinigo approved this pull request.
this overall looks good to me
https://github.com/llvm/llvm-project/pull/148061
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lld
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
https://github.com/JDevlieghere created
https://github.com/llvm/llvm-project/pull/152396
This is a continuation of #152188, which started splitting up the MCP
implementation into a generic implementation in Protocol/MCP that will be
shared between LLDB and lldb-mcp.
For now I kept all the net
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/149827
___
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/152338
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/ashgti created
https://github.com/llvm/llvm-project/pull/152367
Reapply "[lldb] Update JSONTransport to use MainLoop for reading." (#152155)
This reverts commit cd40281685f642ad879e33f3fda8d1faa136ebf4.
This also includes some updates to try to address the platforms with fai
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
ashgti wrote:
You may not need these to be extra commands.
If you register a debug configuration resolve you can have the provider change
the type of the debug configuration during resolution.
```
// example changing configuration type
vscode.registerDebugConfigurationProvider('my-type', {
Author: David Spickett
Date: 2025-08-06T09:25:48Z
New Revision: 49d5dd37f8bdd961d11cdf4df95d26982b253e97
URL:
https://github.com/llvm/llvm-project/commit/49d5dd37f8bdd961d11cdf4df95d26982b253e97
DIFF:
https://github.com/llvm/llvm-project/commit/49d5dd37f8bdd961d11cdf4df95d26982b253e97.diff
LOG
Author: Michael Buch
Date: 2025-08-06T11:54:41+01:00
New Revision: 2b4b3fd03f716b9ddbb2a69ccfbe144312bedd12
URL:
https://github.com/llvm/llvm-project/commit/2b4b3fd03f716b9ddbb2a69ccfbe144312bedd12
DIFF:
https://github.com/llvm/llvm-project/commit/2b4b3fd03f716b9ddbb2a69ccfbe144312bedd12.diff
Author: Ilia Kuklin
Date: 2025-08-06T14:32:19+05:00
New Revision: dace67e941f309318b5ce200c1f4e180a4471d20
URL:
https://github.com/llvm/llvm-project/commit/dace67e941f309318b5ce200c1f4e180a4471d20
DIFF:
https://github.com/llvm/llvm-project/commit/dace67e941f309318b5ce200c1f4e180a4471d20.diff
L
DavidSpickett wrote:
Actually was an alignment issue trying to load from a struct -
https://github.com/llvm/llvm-project/commit/49d5dd37f8bdd961d11cdf4df95d26982b253e97.
https://github.com/llvm/llvm-project/pull/151460
___
lldb-commits mailing list
ll
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
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/source/Plugins/Process/Utility/NativeRegisterC
DavidSpickett wrote:
https://github.com/llvm/llvm-project/pull/152284
https://github.com/llvm/llvm-project/pull/147198
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/DavidSpickett updated
https://github.com/llvm/llvm-project/pull/152284
>From b1a421f6c94057bf7f4ca375907a6ab7081d5a33 Mon Sep 17 00:00:00 2001
From: David Spickett
Date: Mon, 16 Dec 2024 16:02:33 +
Subject: [PATCH 1/3] [lldb][ARM] Port Arm Linux to use
NativeRegisterCont
DavidSpickett wrote:
@b10902118 please try this out in the scenarios you fixed previously.
In theory I could have run the test suite with everything AArch32 but my worry
is that too much in lldb will make assumptions about the host being AArch64.
Plus, I want you to test this anyway.
https://
https://github.com/labath approved this pull request.
Seems reasonable to me.
https://github.com/llvm/llvm-project/pull/152284
___
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/152226
___
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: Dave Lee (kastiglione)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/152334.diff
1 Files Affected:
- (modified) lldb/source/Target/StackFrameRecognizer.cpp (+1-1)
``diff
diff --git a/lldb/source/Target/Stac
https://github.com/kastiglione created
https://github.com/llvm/llvm-project/pull/152338
None
>From aa9613b6e7a328af6764184e5bbd4a0f33a6d16f Mon Sep 17 00:00:00 2001
From: Dave Lee
Date: Tue, 5 Aug 2025 15:37:30 -0700
Subject: [PATCH] wip: add "settings modified" command
---
.../lldb/Interpre
https://github.com/kastiglione updated
https://github.com/llvm/llvm-project/pull/152338
>From a298abcf82d56a1ccc10b6bc4ac0fd11765ba1a6 Mon Sep 17 00:00:00 2001
From: Dave Lee
Date: Tue, 5 Aug 2025 15:37:30 -0700
Subject: [PATCH] [lldb] Add "settings modified" command
---
.../lldb/Interpreter/
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Ilia Kuklin (kuilpd)
Changes
This patch introduces `ScalarLiteralNode` without any uses by other nodes yet.
It also includes lexing and parsing for integer and floating point numbers, and
makes a function `getAutoSenseRadix` in `StringRef`
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
https://github.com/walter-erquinigo approved this pull request.
https://github.com/llvm/llvm-project/pull/151884
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/walter-erquinigo approved this pull request.
thanks!
https://github.com/llvm/llvm-project/pull/151828
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/kuilpd closed
https://github.com/llvm/llvm-project/pull/151350
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
DavidSpickett wrote:
This is a lot of change so it isn't a great review experience. A decent
strategy would be to look at each of the strange details we had before, and
check there is an equivalent in the new code. I leaned a lot on existing tests
to validate this.
https://github.com/llvm/llv
Author: Michael Buch
Date: 2025-08-06T12:19:04+01:00
New Revision: b242150b075a8a720b00821682a9469258bbcd30
URL:
https://github.com/llvm/llvm-project/commit/b242150b075a8a720b00821682a9469258bbcd30
DIFF:
https://github.com/llvm/llvm-project/commit/b242150b075a8a720b00821682a9469258bbcd30.diff
https://github.com/kuilpd created
https://github.com/llvm/llvm-project/pull/152308
This patch introduces `ScalarLiteralNode` without any uses by other nodes yet.
It also includes lexing and parsing for integer and floating point numbers, and
makes a function `getAutoSenseRadix` in `StringRef`
kuilpd wrote:
`PickLiteralType` follows C++ logic, but this can later be put in a plugin if
the logic doesn't apply everywhere. I'm not sure how to test this though, since
the type is picked by width, so the resulting type name can be different on
different machines.
I also split lexing numbe
@@ -2482,6 +2485,134 @@ bool SymbolFileDWARF::ResolveFunction(const DWARFDIE
&orig_die,
return false;
}
+static int ClangToItaniumCtorKind(clang::CXXCtorType kind) {
+ switch (kind) {
+ case clang::CXXCtorType::Ctor_Complete:
+return 1;
+ case clang::CXXCtorType::Cto
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Dave Lee (kastiglione)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/152338.diff
3 Files Affected:
- (modified) lldb/include/lldb/Interpreter/OptionValueProperties.h (+3)
- (modified) lldb/source/Commands/CommandObje
https://github.com/kastiglione edited
https://github.com/llvm/llvm-project/pull/152338
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
https://github.com/kuilpd closed
https://github.com/llvm/llvm-project/pull/147064
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
Author: Jonas Devlieghere
Date: 2025-08-06T09:06:54-07:00
New Revision: 3686e5b52f2a02c1c19050479d1dd0fd9d1dd4f8
URL:
https://github.com/llvm/llvm-project/commit/3686e5b52f2a02c1c19050479d1dd0fd9d1dd4f8
DIFF:
https://github.com/llvm/llvm-project/commit/3686e5b52f2a02c1c19050479d1dd0fd9d1dd4f8.d
Author: royitaqi
Date: 2025-08-06T10:24:09-07:00
New Revision: bb2642fab70fb4d59e431e01e319dcf6b90d88d8
URL:
https://github.com/llvm/llvm-project/commit/bb2642fab70fb4d59e431e01e319dcf6b90d88d8
DIFF:
https://github.com/llvm/llvm-project/commit/bb2642fab70fb4d59e431e01e319dcf6b90d88d8.diff
LOG:
https://github.com/royitaqi closed
https://github.com/llvm/llvm-project/pull/151828
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -301,6 +301,14 @@ Variable CreateVariable(lldb::SBValue v, int64_t var_ref,
bool format_hex,
if (lldb::addr_t addr = v.GetLoadAddress(); addr != LLDB_INVALID_ADDRESS)
var.memoryReference = addr;
+ bool is_readonly = v.GetType().IsAggregateType() ||
https://github.com/DavidSpickett created
https://github.com/llvm/llvm-project/pull/152284
Which is also used by AArch64 and LoongArch.
To do this I had to make some adjustments to NativeRegisterContextDBReg:
* New method AdjustBreakpoint. This is like AdjustWatchpoint, but
only Arm needs to
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: David Spickett (DavidSpickett)
Changes
Which is also used by AArch64 and LoongArch.
To do this I had to make some adjustments to NativeRegisterContextDBReg:
* New method AdjustBreakpoint. This is like AdjustWatchpoint, but
only Arm needs
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
https://github.com/JDevlieghere commented:
General feedback: this patch is using `auto` in a bunch of places that don't
align with the [LLVM Coding
Standards](https://llvm.org/docs/CodingStandards.html#use-auto-type-deduction-to-make-code-more-readable).
https://github.com/llvm/llvm-project/
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
@@ -125,19 +126,43 @@ ExecutionContext::ExecutionContext(const
ExecutionContextRef *exe_ctx_ref_ptr,
}
}
-ExecutionContext::ExecutionContext(const ExecutionContextRef *exe_ctx_ref_ptr,
- std::unique_lock
&lock)
+ExecutionContext::Execution
@@ -755,11 +755,237 @@ size_t ModuleList::GetIndexForModule(const Module
*module) const {
}
namespace {
+/// A wrapper around ModuleList for shared modules. Provides fast lookups for
+/// file-based ModuleSpec queries.
+class SharedModuleList {
+public:
+ /// Finds all the m
@@ -125,19 +126,43 @@ ExecutionContext::ExecutionContext(const
ExecutionContextRef *exe_ctx_ref_ptr,
}
}
-ExecutionContext::ExecutionContext(const ExecutionContextRef *exe_ctx_ref_ptr,
- std::unique_lock
&lock)
+ExecutionContext::Execution
https://github.com/JDevlieghere edited
https://github.com/llvm/llvm-project/pull/152054
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/felipepiovezan edited
https://github.com/llvm/llvm-project/pull/152020
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -125,19 +126,43 @@ ExecutionContext::ExecutionContext(const
ExecutionContextRef *exe_ctx_ref_ptr,
}
}
-ExecutionContext::ExecutionContext(const ExecutionContextRef *exe_ctx_ref_ptr,
- std::unique_lock
&lock)
+ExecutionContext::Execution
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: nerix (Nerixyz)
Changes
This adds the ability for functions to be matched by their basename.
Before, the globals were searched for the name. This works if the full name is
available but fails for basenames.
PDB only includes the full names
https://github.com/labath approved this pull request.
Makes sense. I gave this a spin on our internal build, and it seems to work
fine.
https://github.com/llvm/llvm-project/pull/152226
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://
https://github.com/felipepiovezan edited
https://github.com/llvm/llvm-project/pull/152020
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
royitaqi wrote:
@ashgti Wow! This is very useful information (and powerful feature). Good to
know. Thanks~! I will keep these options in mind.
Just curious, is there good documentation to read about all these? I read [this
page from VS
Code](https://code.visualstudio.com/api/extension-guides/
100 matches
Mail list logo
