clayborg wrote:
The main issue that this covers is when you have N .o files compiled with
.debug_names, and M .o files compiled without it, and then you link N+M into a
final executable. The linker will simply concatenate all of the .o files
section together, but the linker doesn't know anythi
DmT021 wrote:
> Could you run `ninja check-lldb` from your build folder to make sure this
> change doesn't break anything?
Already did. All tests are ok.
https://github.com/llvm/llvm-project/pull/102835
___
lldb-commits mailing list
lldb-commits@list
@@ -174,6 +176,83 @@ struct StatisticsOptions {
std::optional m_include_transcript;
};
+/// A class that represents statistics about a TypeSummaryProviders invocations
+class SummaryStatistics {
+public:
+ SummaryStatistics() = default;
+ SummaryStatistics(lldb_private::Co
https://github.com/bulbazord commented:
Thanks!
https://github.com/llvm/llvm-project/pull/102793
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -174,6 +176,83 @@ struct StatisticsOptions {
std::optional m_include_transcript;
};
+/// A class that represents statistics about a TypeSummaryProviders invocations
+class SummaryStatistics {
+public:
+ SummaryStatistics() = default;
+ SummaryStatistics(lldb_private::Co
augusto2112 wrote:
@DmT021 I think you can update this patch and set it ready for review then :)
https://github.com/llvm/llvm-project/pull/102835
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo
@@ -799,20 +803,21 @@ IRExecutionUnit::FindInSymbols(const
std::vector &names,
sc_list);
if (auto load_addr = resolver.Resolve(sc_list))
return *load_addr;
-}
-if (sc.target_sp) {
- SymbolContextList sc_list;
-
https://github.com/bulbazord commented:
Makes sense to me and the change looks good. Feel free to move this out of
draft mode :)
https://github.com/llvm/llvm-project/pull/102835
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.l
@@ -258,6 +258,7 @@ class TargetProperties : public Properties {
bool GetDebugUtilityExpression() const;
+
bulbazord wrote:
nit: spurious newline
https://github.com/llvm/llvm-project/pull/102708
___
lldb-commits
@@ -3193,6 +3193,14 @@ bool Target::SetSectionUnloaded(const lldb::SectionSP
§ion_sp,
void Target::ClearAllLoadedSections() { m_section_load_history.Clear(); }
+lldb_private::SummaryStatistics&
Target::GetSummaryStatisticsForProvider(lldb_private::ConstString
summary_provi
@@ -174,6 +176,83 @@ struct StatisticsOptions {
std::optional m_include_transcript;
};
+/// A class that represents statistics about a TypeSummaryProviders invocations
+class SummaryStatistics {
+public:
+ SummaryStatistics() = default;
+ SummaryStatistics(lldb_private::Co
@@ -174,6 +176,83 @@ struct StatisticsOptions {
std::optional m_include_transcript;
};
+/// A class that represents statistics about a TypeSummaryProviders invocations
+class SummaryStatistics {
+public:
+ SummaryStatistics() = default;
+ SummaryStatistics(lldb_private::Co
@@ -174,6 +176,83 @@ struct StatisticsOptions {
std::optional m_include_transcript;
};
+/// A class that represents statistics about a TypeSummaryProviders invocations
+class SummaryStatistics {
+public:
+ SummaryStatistics() = default;
+ SummaryStatistics(lldb_private::Co
https://github.com/clayborg requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/102708
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -174,6 +176,83 @@ struct StatisticsOptions {
std::optional m_include_transcript;
};
+/// A class that represents statistics about a TypeSummaryProviders invocations
+class SummaryStatistics {
+public:
+ SummaryStatistics() = default;
clayborg wrote:
Do
@@ -174,6 +176,83 @@ struct StatisticsOptions {
std::optional m_include_transcript;
};
+/// A class that represents statistics about a TypeSummaryProviders invocations
+class SummaryStatistics {
+public:
+ SummaryStatistics() = default;
+ SummaryStatistics(lldb_private::Co
@@ -174,6 +176,83 @@ struct StatisticsOptions {
std::optional m_include_transcript;
};
+/// A class that represents statistics about a TypeSummaryProviders invocations
+class SummaryStatistics {
+public:
+ SummaryStatistics() = default;
+ SummaryStatistics(lldb_private::Co
@@ -615,7 +615,16 @@ bool ValueObject::GetSummaryAsCString(TypeSummaryImpl
*summary_ptr,
m_synthetic_value->UpdateValueIfNeeded(); // the summary might depend on
// the synthetic children being
@@ -174,6 +176,83 @@ struct StatisticsOptions {
std::optional m_include_transcript;
};
+/// A class that represents statistics about a TypeSummaryProviders invocations
+class SummaryStatistics {
+public:
+ SummaryStatistics() = default;
+ SummaryStatistics(lldb_private::Co
@@ -174,6 +176,83 @@ struct StatisticsOptions {
std::optional m_include_transcript;
};
+/// A class that represents statistics about a TypeSummaryProviders invocations
+class SummaryStatistics {
+public:
+ SummaryStatistics() = default;
+ SummaryStatistics(lldb_private::Co
@@ -615,7 +615,16 @@ bool ValueObject::GetSummaryAsCString(TypeSummaryImpl
*summary_ptr,
m_synthetic_value->UpdateValueIfNeeded(); // the summary might depend on
// the synthetic children being
@@ -1,7 +1,13 @@
// Test that the lldb command `statistics` works.
+#include
+
+void foo() {
+ std::string str = "hello world";
clayborg wrote:
We want to test that this works for:
- built in C++ summary formatters (std::string)
- python summary formatters
- s
@@ -1,7 +1,13 @@
// Test that the lldb command `statistics` works.
+#include
+
+void foo() {
+ std::string str = "hello world";
clayborg wrote:
We should also test a type that uses a template as some summary providers match
based on a regex like:
```
std::vec
@@ -918,3 +920,24 @@ def test_order_of_options_do_not_matter(self):
debug_stats_1,
f"The order of options '{options[0]}' and '{options[1]}'
should not matter",
)
+
+def test_summary_statistics_providers(self):
+"""
+
@@ -615,7 +615,16 @@ bool ValueObject::GetSummaryAsCString(TypeSummaryImpl
*summary_ptr,
m_synthetic_value->UpdateValueIfNeeded(); // the summary might depend on
// the synthetic children being
@@ -174,6 +176,83 @@ struct StatisticsOptions {
std::optional m_include_transcript;
};
+/// A class that represents statistics about a TypeSummaryProviders invocations
+class SummaryStatistics {
+public:
+ SummaryStatistics() = default;
+ SummaryStatistics(lldb_private::Co
@@ -408,3 +410,23 @@ llvm::json::Value DebuggerStats::ReportStatistics(
return std::move(global_stats);
}
+
+llvm::json::Value SummaryStatistics::ToJSON() const {
+ json::Object body {{
+ {"invocationCount", GetSummaryCount()},
+ {"totalTime", GetTotalTime()},
+
@@ -258,6 +258,7 @@ class TargetProperties : public Properties {
bool GetDebugUtilityExpression() const;
+
clayborg wrote:
remove blank line
https://github.com/llvm/llvm-project/pull/102708
___
lldb-commits mail
https://github.com/DmT021 updated
https://github.com/llvm/llvm-project/pull/102835
>From 45c91e403a36360727a5cd51e1a84f86027a77bc Mon Sep 17 00:00:00 2001
From: Dmitrii Galimzianov
Date: Mon, 12 Aug 2024 01:09:33 +0200
Subject: [PATCH] Remove redundant symbol lookups in
IRExecutionUnit::FindIn
https://github.com/DmT021 ready_for_review
https://github.com/llvm/llvm-project/pull/102835
___
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: Dmitrii Galimzianov (DmT021)
Changes
When we search for a symbol, we first check if it is in the module_sp of the
current SymbolContext, and if not, we check in the target's modules. However,
the target's ModuleList also includes the alrea
@@ -174,6 +176,83 @@ struct StatisticsOptions {
std::optional m_include_transcript;
};
+/// A class that represents statistics about a TypeSummaryProviders invocations
+class SummaryStatistics {
+public:
+ SummaryStatistics() = default;
+ SummaryStatistics(lldb_private::Co
https://github.com/Jlalond edited
https://github.com/llvm/llvm-project/pull/102708
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Author: Greg Clayton
Date: 2024-08-12T10:57:04-07:00
New Revision: f1e2886261281e788e4faae406c24e787c1dbdd0
URL:
https://github.com/llvm/llvm-project/commit/f1e2886261281e788e4faae406c24e787c1dbdd0
DIFF:
https://github.com/llvm/llvm-project/commit/f1e2886261281e788e4faae406c24e787c1dbdd0.diff
https://github.com/clayborg closed
https://github.com/llvm/llvm-project/pull/102570
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/Michael137 commented:
Generally, makes sense to me. Out of curiosity, could you provide some context
around where/how this came up?
The ideal version of this would be a version of
`FindFunctions`/`FindSymbolsWithNameAndType` that took a `ModuleSP` hint (aka
where to search
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/102835
___
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/102835
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/augusto2112 approved this pull request.
https://github.com/llvm/llvm-project/pull/102835
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
jeffreytan81 wrote:
@labath, unfortunately the failing log does not provide much information to
troubleshoot. Is it possible to gather more information of the failure? Like is
it only this `lldb-x86_64-debian` bot timeout or including other bots? And is
it consistently failing or occasionally
Author: Sergei Barannikov
Date: 2024-08-12T21:46:47+03:00
New Revision: e26b42c70d37cbea47016984e1d2bfc347fb8818
URL:
https://github.com/llvm/llvm-project/commit/e26b42c70d37cbea47016984e1d2bfc347fb8818
DIFF:
https://github.com/llvm/llvm-project/commit/e26b42c70d37cbea47016984e1d2bfc347fb8818.d
https://github.com/s-barannikov closed
https://github.com/llvm/llvm-project/pull/102839
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
DmT021 wrote:
> Out of curiosity, could you provide some context around where/how this came
> up?
I just noticed this by accident in the `dwarf all` log when I was trying to
figure out why LLDB was evaluating expressions slowly in my project.
Here's a swift forum thread with some logs.
https:
https://github.com/clayborg commented:
While this does solve the problem, it means other clients that want to do the
same thing will need to create this same kind of code.
I would propose we modify `ModuleList::FindSymbolsWithNameAndType(...)` to take
an extra parameter `SymbolContext &sc` whi
https://github.com/clayborg edited
https://github.com/llvm/llvm-project/pull/102835
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -174,6 +176,83 @@ struct StatisticsOptions {
std::optional m_include_transcript;
};
+/// A class that represents statistics about a TypeSummaryProviders invocations
+class SummaryStatistics {
+public:
+ SummaryStatistics() = default;
+ SummaryStatistics(lldb_private::Co
https://github.com/Jlalond closed
https://github.com/llvm/llvm-project/pull/101272
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
augusto2112 wrote:
I agree that passing some sort of "hint" object to prioritize the search would
be a more complete solution, but since this is @DmT021's first contribution to
LLVM I think we should merge this change if there are no concerns with the
current implementation, and if @DmT021 wan
clayborg wrote:
> I agree that passing some sort of "hint" object to prioritize the search
> would be a more complete solution, but since this is @DmT021's first
> contribution to LLVM I think we should merge this change if there are no
> concerns with the current implementation, and if @DmT02
@@ -0,0 +1,459 @@
+//===-- DILAST.h *- C++
-*-===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Ap
https://github.com/clayborg commented:
Thanks for trying this, and yes, this is exactly what I was thinking!
After seeing all of the code changes that this created, I suggested in
ModuleList.h that we add a the symbol context as a default parameter to the
functions as `const SymbolContext *sc
https://github.com/clayborg edited
https://github.com/llvm/llvm-project/pull/102835
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -295,7 +295,7 @@ class ModuleList {
/// \see Module::FindFunctions ()
void FindFunctions(const RegularExpression &name,
const ModuleFunctionSearchOptions &options,
- SymbolContextList &sc_list);
+ const SymbolC
@@ -285,7 +285,7 @@ class ModuleList {
/// \see Module::FindFunctions ()
void FindFunctions(ConstString name, lldb::FunctionNameType name_type_mask,
const ModuleFunctionSearchOptions &options,
- SymbolContextList &sc_list) const;
+
@@ -355,6 +355,7 @@ class ModuleList {
void FindSymbolsWithNameAndType(ConstString name,
lldb::SymbolType symbol_type,
+ const SymbolContext &sc,
SymbolContextList &sc_list)
@@ -1764,8 +1764,8 @@ lldb::SBSymbolContextList SBTarget::FindFunctions(const
char *name,
function_options.include_inlines = true;
FunctionNameType mask = static_cast(name_type_mask);
- target_sp->GetImages().FindFunctions(ConstString(name), mask,
-
@@ -351,8 +351,8 @@ CommandObjectDisassemble::GetNameRanges(CommandReturnObject
&result) {
// Find functions matching the given name.
SymbolContextList sc_list;
- GetTarget().GetImages().FindFunctions(name, eFunctionNameTypeAuto,
-
clayborg wrote:
We should also document in the headerdoc this new `const SymbolContext *sc =
nullptr` argument and say something like:
```
/// \param[in] sc If the value is NULL, then all modules will be searched in
/// order. If the value is a valid pointer and if a module is specified in th
labath wrote:
> I guess what I was proposing was more about deleting code than adding it. We
> must always have at most _one_ index, but we have a lot of code allowing for
> the situation of two indices (manual & {apple, dwarf}).
Okay. I think I understand now what you meant -- which was to ha
labath wrote:
It's not just this bot. I also found failures on
[aarch64](https://lab.llvm.org/buildbot/#/builders/59/builds/2999) and
[arm](https://lab.llvm.org/buildbot/#/builders/18/builds/2079). I don't see it
on the mac bots, but I don't know how to look further into the history for
thos
@@ -0,0 +1,459 @@
+//===-- DILAST.h *- C++
-*-===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Ap
https://github.com/xusheng6 updated
https://github.com/llvm/llvm-project/pull/102873
>From 73c98df4baef99f96d9c67113ba2ed0d972e5a04 Mon Sep 17 00:00:00 2001
From: Xusheng
Date: Mon, 20 Mar 2023 20:24:11 +0800
Subject: [PATCH 1/6] [lldb] Claim to support swbreak and hwbreak packets when
debuggi
xusheng6 wrote:
> > or do we already have some infra that I can use?
>
> `lldb/test/API/functionalities/gdb_remote_client/TestStopPCs.py` is an
> example of that infrastructure. The `threadStopInfo` is where you would
> insert the swbreak hwbreak key.
>
> In fact, if i386 is an architecture w
101 - 163 of 163 matches
Mail list logo
