https://github.com/mizvekov edited
https://github.com/llvm/llvm-project/pull/136162
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/mizvekov closed
https://github.com/llvm/llvm-project/pull/136158
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
tbaederr wrote:
I broke it with https://github.com/llvm/llvm-project/pull/117671
https://github.com/llvm/llvm-project/pull/136269
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/boomanaiden154 approved this pull request.
https://github.com/llvm/llvm-project/pull/136158
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`clang-s390x-linux-multistage` running on `systemz-1` while building
`lldb,llvm` at step 5 "ninja check 1".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/98/builds/1285
Here is the relevant piece of
https://github.com/royitaqi edited
https://github.com/llvm/llvm-project/pull/136236
___
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/136236
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -749,10 +749,20 @@ TEST_F(SymtabTest, TestSymtabCreatedOnDemand) {
ASSERT_THAT_EXPECTED(ExpectedFile, llvm::Succeeded());
auto module_sp = std::make_shared(ExpectedFile->moduleSpec());
- // The symbol table should not be loaded by default.
+ // The symbol file should
https://github.com/mizvekov updated
https://github.com/llvm/llvm-project/pull/136158
>From ba2f21f3619a8079e1ae06a6a01d11be8d98163f Mon Sep 17 00:00:00 2001
From: Matheus Izvekov
Date: Thu, 17 Apr 2025 13:31:32 -0300
Subject: [PATCH] [ci] add dependencies for lldb python binding tests
Add corr
https://github.com/mizvekov edited
https://github.com/llvm/llvm-project/pull/136158
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/jimingham edited
https://github.com/llvm/llvm-project/pull/136236
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -749,10 +749,20 @@ TEST_F(SymtabTest, TestSymtabCreatedOnDemand) {
ASSERT_THAT_EXPECTED(ExpectedFile, llvm::Succeeded());
auto module_sp = std::make_shared(ExpectedFile->moduleSpec());
- // The symbol table should not be loaded by default.
+ // The symbol file should
@@ -749,10 +749,20 @@ TEST_F(SymtabTest, TestSymtabCreatedOnDemand) {
ASSERT_THAT_EXPECTED(ExpectedFile, llvm::Succeeded());
auto module_sp = std::make_shared(ExpectedFile->moduleSpec());
- // The symbol table should not be loaded by default.
+ // The symbol file should
https://github.com/royitaqi edited
https://github.com/llvm/llvm-project/pull/136236
___
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/136236
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/dzhidzhoev closed
https://github.com/llvm/llvm-project/pull/136309
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: Vladislav Dzhidzhoev
Date: 2025-04-18T22:16:35+02:00
New Revision: 058992ea9fc62a9f79ab90eaebecb267469795a7
URL:
https://github.com/llvm/llvm-project/commit/058992ea9fc62a9f79ab90eaebecb267469795a7
DIFF:
https://github.com/llvm/llvm-project/commit/058992ea9fc62a9f79ab90eaebecb267469795a
@@ -992,6 +997,8 @@ ModuleList::GetSharedModule(const ModuleSpec &module_spec,
ModuleSP &module_sp,
// By getting the object file we can guarantee that the architecture
// matches
if (module_sp && module_sp->GetObjectFile()) {
+module_sp->GetSymbolLoc
@@ -814,8 +814,16 @@ Status PlatformDarwinKernel::GetSharedModuleKernel(
// append ".dSYM" to the filename for the SymbolFile.
FileSpecList search_paths =
process->GetTarget().GetDebugFileSearchPaths();
-FileSpec dsym_fspec =
-Plu
@@ -297,6 +309,10 @@ Status ProcessKDP::DoConnectRemote(llvm::StringRef
remote_url) {
if (FileSystem::Instance().Exists(module_spec.GetFileSpec())) {
ModuleSP module_sp(new Module(module_spec));
+module_sp->GetSymbolLocatorStatist
@@ -122,6 +122,7 @@ struct ModuleStats {
double symtab_index_time = 0.0;
double debug_parse_time = 0.0;
double debug_index_time = 0.0;
+ llvm::StringMap symbol_locator_time;
clayborg wrote:
We should move `StatisticsMap` into this file and then have `St
@@ -917,9 +917,14 @@ ModuleList::GetSharedModule(const ModuleSpec &module_spec,
ModuleSP &module_sp,
// Fixup the incoming path in case the path points to a valid file, yet the
// arch or UUID (if one was passed in) don't match.
- ModuleSpec located_binary_modulespec =
-
@@ -276,12 +276,24 @@ Status ProcessKDP::DoConnectRemote(llvm::StringRef
remote_url) {
// Lookup UUID locally, before attempting dsymForUUID like action
FileSpecList search_paths =
Target::GetDefaultDebugFileSearchPaths();
-
@@ -243,15 +243,33 @@ ModuleSP DynamicLoader::LoadBinaryWithUUIDAndAddress(
// find an executable and symbol file.
if (!module_sp) {
FileSpecList search_paths = Target::GetDefaultDebugFileSearchPaths();
- module_spec.GetSymbolFileSpec() =
- PluginMan
@@ -243,15 +243,33 @@ ModuleSP DynamicLoader::LoadBinaryWithUUIDAndAddress(
// find an executable and symbol file.
if (!module_sp) {
FileSpecList search_paths = Target::GetDefaultDebugFileSearchPaths();
- module_spec.GetSymbolFileSpec() =
- PluginMan
@@ -956,6 +961,20 @@ class Module : public std::enable_shared_from_this,
bool m_match_name_after_lookup = false;
};
+ class StatisticsMap {
+ public:
+void add(const std::string &key, double value) {
+ if (key.empty())
+return;
+ auto it = map.fin
https://github.com/clayborg requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/134563
___
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/136309
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/GeorgeHuyubo edited
https://github.com/llvm/llvm-project/pull/134563
___
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: John Harrison (ashgti)
Changes
Reverse requests do have a 'seq' set still from VSCode. I incorrectly
interpreted
https://github.com/microsoft/vscode/blob/dede7bb4b7e9c9ec69155a243bb84037a40588fe/src/vs/workbench/contrib/debug/common/abstra
https://github.com/felipepiovezan closed
https://github.com/llvm/llvm-project/pull/136160
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: Felipe de Azevedo Piovezan
Date: 2025-04-17T12:53:25-07:00
New Revision: 9f9c1f93819d441c561d18a73a55bcb6d8a692df
URL:
https://github.com/llvm/llvm-project/commit/9f9c1f93819d441c561d18a73a55bcb6d8a692df
DIFF:
https://github.com/llvm/llvm-project/commit/9f9c1f93819d441c561d18a73a55bcb6d
https://github.com/kastiglione closed
https://github.com/llvm/llvm-project/pull/136025
___
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: None (royitaqi)
Changes
These stats are added or changed:
1. In summary:
1. Add `totalSymbolsLoaded`. The total number of symbols loaded in all
modules.
2. Add `totalSymbolTablesLoaded `. The total number symbol tables loaded in
al
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 cpp,h --
lldb/include/lldb/Symbol/ObjectFile.h lldb/include/
Author: Raul Tambre
Date: 2025-04-18T14:48:30+03:00
New Revision: c890b7376fc916c1818dc3c211a080601973aa0b
URL:
https://github.com/llvm/llvm-project/commit/c890b7376fc916c1818dc3c211a080601973aa0b
DIFF:
https://github.com/llvm/llvm-project/commit/c890b7376fc916c1818dc3c211a080601973aa0b.diff
L
@@ -749,10 +749,20 @@ TEST_F(SymtabTest, TestSymtabCreatedOnDemand) {
ASSERT_THAT_EXPECTED(ExpectedFile, llvm::Succeeded());
auto module_sp = std::make_shared(ExpectedFile->moduleSpec());
- // The symbol table should not be loaded by default.
+ // The symbol file should
https://github.com/royitaqi edited
https://github.com/llvm/llvm-project/pull/136236
___
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/136236
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -749,10 +749,20 @@ TEST_F(SymtabTest, TestSymtabCreatedOnDemand) {
ASSERT_THAT_EXPECTED(ExpectedFile, llvm::Succeeded());
auto module_sp = std::make_shared(ExpectedFile->moduleSpec());
- // The symbol table should not be loaded by default.
+ // The symbol file should
tambry wrote:
I'll return to this once I find the time.
https://github.com/llvm/llvm-project/pull/110557
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/tambry closed
https://github.com/llvm/llvm-project/pull/110557
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: Jonas Devlieghere
Date: 2025-04-17T17:57:00+02:00
New Revision: 83116209331ad6a5a45f1b8735ad5bce9e6ef761
URL:
https://github.com/llvm/llvm-project/commit/83116209331ad6a5a45f1b8735ad5bce9e6ef761
DIFF:
https://github.com/llvm/llvm-project/commit/83116209331ad6a5a45f1b8735ad5bce9e6ef761.d
felipepiovezan wrote:
Closing in favor of the stack of patches here:
[[lldb][nfc] Factor out code from ThreadPlanStepOut ctor
#136159](https://github.com/llvm/llvm-project/pull/136159)
[[lldb][nfc] Split the constructor of ThreadPlanStepOut
#136160](https://github.com/llvm/llvm-project/pull/13
https://github.com/JDevlieghere edited
https://github.com/llvm/llvm-project/pull/136065
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/charles-zablit updated
https://github.com/llvm/llvm-project/pull/135963
>From 6dd67fe4ad03f0ec0623717715b8cfcc9537ab3f Mon Sep 17 00:00:00 2001
From: Charles Zablit
Date: Wed, 16 Apr 2025 11:28:54 +0100
Subject: [PATCH 1/3] [lldb] Remove unused API
CompilerType::GetIndexOfFi
Author: Shubham Sandeep Rastogi
Date: 2025-04-18T09:34:48-07:00
New Revision: f4a47b4003b6cfbdd445615b044f0911d99a0e44
URL:
https://github.com/llvm/llvm-project/commit/f4a47b4003b6cfbdd445615b044f0911d99a0e44
DIFF:
https://github.com/llvm/llvm-project/commit/f4a47b4003b6cfbdd445615b044f0911d99a
https://github.com/rastogishubham closed
https://github.com/llvm/llvm-project/pull/136322
___
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: Shubham Sandeep Rastogi (rastogishubham)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/136322.diff
1 Files Affected:
- (modified)
lldb/test/API/commands/expression/import-std-module/deque-dbg-info-content/TestDbgInfo
https://github.com/rastogishubham created
https://github.com/llvm/llvm-project/pull/136322
None
>From f6fb1bcb7d6c05bc32a58d00f3bbdffb1c45ec94 Mon Sep 17 00:00:00 2001
From: Shubham Sandeep Rastogi
Date: Fri, 18 Apr 2025 09:20:28 -0700
Subject: [PATCH] Disable test on older compilers.
---
..
adrian-prantl wrote:
Why did this break?
https://github.com/llvm/llvm-project/pull/136269
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -749,10 +749,20 @@ TEST_F(SymtabTest, TestSymtabCreatedOnDemand) {
ASSERT_THAT_EXPECTED(ExpectedFile, llvm::Succeeded());
auto module_sp = std::make_shared(ExpectedFile->moduleSpec());
- // The symbol table should not be loaded by default.
+ // The symbol file should
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Vladislav Dzhidzhoev (dzhidzhoev)
Changes
TestStructuredDataAPI.py fails with Python debug build ver. 3.12+ due to call
to Py_XINCREF while GIL is not held.
---
Full diff: https://github.com/llvm/llvm-project/pull/136309.diff
1 Files Aff
https://github.com/dzhidzhoev created
https://github.com/llvm/llvm-project/pull/136309
TestStructuredDataAPI.py fails with Python debug build ver. 3.12+ due to call
to Py_XINCREF while GIL is not held.
>From 48538eb932283ef5652555402cc68a244e59c730 Mon Sep 17 00:00:00 2001
From: Vladislav Dzhi
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: Shubham Sandeep Rastogi (rastogishubham)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/136186.diff
1 Files Affected:
- (modified)
lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx-simulators/inva
https://github.com/tambry closed
https://github.com/llvm/llvm-project/pull/136115
___
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/136236
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
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 HEAD~1...HEAD
lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx-simul
github-actions[bot] wrote:
:warning: undef deprecator found issues in your code. :warning:
You can test this locally with the following command:
``bash
git diff -U0 --pickaxe-regex -S
'([^a-zA-Z0-9#_-]undef[^a-zA-Z0-9_-]|UndefValue::get)' 'HEAD~1' HEAD
lldb/include/lldb/Core/Dem
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/131836
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/tbaederr closed
https://github.com/llvm/llvm-project/pull/136269
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: Timm Baeder
Date: 2025-04-18T09:33:20+02:00
New Revision: 802e7309c0ab7e9966f8a98eede0231d146c6a97
URL:
https://github.com/llvm/llvm-project/commit/802e7309c0ab7e9966f8a98eede0231d146c6a97
DIFF:
https://github.com/llvm/llvm-project/commit/802e7309c0ab7e9966f8a98eede0231d146c6a97.diff
L
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/136269
>From bc65c24f328a45184cbded16b204c5c1aba78450 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Fri, 18 Apr 2025 08:43:37 +0200
Subject: [PATCH] [lldb] Try to fix TestExprDiagnostics test
---
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/136269
>From 04e355e7356acf6d7de7ab7e232b2a489ad4fffb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Fri, 18 Apr 2025 08:43:37 +0200
Subject: [PATCH] [lldb] Try to fix TestExprDiagnostics test
---
64 matches
Mail list logo