https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/157674
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -959,6 +959,9 @@ CompilerType
TypeSystemClang::GetBuiltinTypeForDWARFEncodingAndBitSize(
if (type_name == "long double" &&
QualTypeMatchesBitSize(bit_size, ast, ast.LongDoubleTy))
return GetType(ast.LongDoubleTy);
+if (type_name == "__bf16" &&
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/154123
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Michael137 wrote:
> Might be fixed next build, I saw the Linux ones fail then go green right
> after.
Yea I hoped so too, but looks like that test run had both my changes. I think
this is just not supported on Windows for now (i doubt this worked before the
tests were added either). I XFAILed
https://github.com/Michael137 closed
https://github.com/llvm/llvm-project/pull/133079
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Michael137 wrote:
Can you rebase your PR on `main` and resolve the conflict in
`TypeSystemClang.cpp`?
https://github.com/llvm/llvm-project/pull/154123
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/lis
@@ -218,6 +222,10 @@ def compiler_rt_asan_tests(self):
self.check_traces()
+# Make sure we're not stopped in the sanitizer library but instead at
the
Michael137 wrote:
Should be fixed now. Let me know if you're still seeing issues
https://g
@@ -1873,6 +1873,18 @@ DWARFASTParserClang::ParseStructureLikeDIE(const
SymbolContext &sc,
clang_type =
m_ast.CreateClassTemplateSpecializationType(class_specialization_decl);
+if (!class_specialization_decl) {
+ if (log) {
+dwarf->GetObjectFile()
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/149827
>From f979958249187a1b89b4b87c46e90d63b6dc4042 Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Mon, 18 Aug 2025 15:14:40 +0100
Subject: [PATCH 1/3] [clang][DebugInfo] Emit unified (Itanium) mangled name to
@@ -0,0 +1,29 @@
+/// Check that LLDB does not emit "GetDIE for DIE {{0x[0-9a-f]+}} is outside
of its CU"
+/// error message when user is searching for a matching symbol from
.debug_names
+/// and fail to locate the corresponding .dwo file.
+
+/// -gsplit-dwarf is supported only
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/149827
>From f979958249187a1b89b4b87c46e90d63b6dc4042 Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Mon, 18 Aug 2025 15:14:40 +0100
Subject: [PATCH 1/2] [clang][DebugInfo] Emit unified (Itanium) mangled name to
https://github.com/Michael137 auto_merge_enabled
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 approved this pull request.
Seems to align with how the PDB plugin does it, so LGTM
https://github.com/llvm/llvm-project/pull/154121
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/
@@ -959,6 +959,9 @@ CompilerType
TypeSystemClang::GetBuiltinTypeForDWARFEncodingAndBitSize(
if (type_name == "long double" &&
QualTypeMatchesBitSize(bit_size, ast, ast.LongDoubleTy))
return GetType(ast.LongDoubleTy);
+if (type_name == "__bf16" &&
@@ -74,6 +74,7 @@ def test_hitting(self):
@skipIf(oslist=lldbplatformutil.getDarwinOSTriples() + ["windows"])
@skipIf(archs=no_match(["x86_64", "aarch64"]))
+@skipTestIfFn(linker_script_syntax_unsupported)
Michael137 wrote:
Did you try wrapping th
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/149827
>From f979958249187a1b89b4b87c46e90d63b6dc4042 Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Mon, 18 Aug 2025 15:14:40 +0100
Subject: [PATCH 1/2] [clang][DebugInfo] Emit unified (Itanium) mangled name to
https://github.com/Michael137 closed
https://github.com/llvm/llvm-project/pull/155485
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/157649
>From 0939db4e9adf79cc02342d66d2402ab72babb64b Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Tue, 9 Sep 2025 12:14:02 +0100
Subject: [PATCH 1/2] [lldb][test] Re-enable import-std-module tests on Linux
T
https://github.com/Michael137 created
https://github.com/llvm/llvm-project/pull/157649
This reverts commit 76bd5da3248fd4affedfefec3b2a3e1acbe0a94a.
These were originally skipped for
https://github.com/llvm/llvm-project/issues/137046 because they were failing on
Fuchsia buildbots.
I couldn't
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/157474
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -0,0 +1,29 @@
+/// Check that LLDB does not emit "GetDIE for DIE {{0x[0-9a-f]+}} is outside
of its CU"
+/// error message when user is searching for a matching symbol from
.debug_names
+/// and fail to locate the corresponding .dwo file.
+
+/// -gsplit-dwarf is supported only
@@ -1725,6 +1725,7 @@ DWARFASTParserClang::ParseStructureLikeDIE(const
SymbolContext &sc,
const dw_tag_t tag = die.Tag();
SymbolFileDWARF *dwarf = die.GetDWARF();
LanguageType cu_language = SymbolFileDWARF::GetLanguage(*die.GetCU());
+ ModuleSP module_sp = dwarf->GetObj
@@ -599,6 +599,39 @@ TEST_F(DWARFASTParserClangTests,
TestDefaultTemplateParamParsing) {
}
}
+TEST_F(DWARFASTParserClangTests, TestSpecDeclExistsError) {
+ // Tests checking error if ClassTemplateSpecializationDecl already exists.
+ auto BufferOrError = llvm::MemoryBuffer
@@ -51,41 +46,28 @@ class StackID {
void Dump(Stream *s);
- // Operators
- const StackID &operator=(const StackID &rhs) {
-if (this != &rhs) {
- m_pc = rhs.m_pc;
- m_cfa = rhs.m_cfa;
- m_symbol_scope = rhs.m_symbol_scope;
-}
-return *this;
- }
Michael137 wrote:
> @Michael137, probably we should return finding and adding specialization in
> DWARFASTParserClang.cpp?
That would be equivalent to the way your current version handles this right? I
don't see how that would change anything.
I now realize this is trickier than expected. E.g
https://github.com/Michael137 auto_merge_disabled
https://github.com/llvm/llvm-project/pull/154123
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/Michael137 closed
https://github.com/llvm/llvm-project/pull/133078
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Michael137 wrote:
> As far as I know, there isn't anything in the debug info that says "this
> variable is the this pointer" other than the name/type of a variable and the
> type of the function.
How does the DIA PDB plugin do it then? Why can't the native plugin use
`PDB_DataKind::ObjectPtr`
@@ -1873,6 +1874,24 @@ DWARFASTParserClang::ParseStructureLikeDIE(const
SymbolContext &sc,
clang_type =
m_ast.CreateClassTemplateSpecializationType(class_specialization_decl);
+// Try to find an existing specialization with these template arguments and
+//
https://github.com/Michael137 approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/157483
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Michael137 wrote:
Based on the test failure in `TestCppTemplateArguments.py` maybe we need to
return the `ClassTemplateDeclSpecialization` that we found, instead of
returning `nullptr`. We would create a new `TypeSP` for it, but that shouldn't
be an issue i think
https://github.com/llvm/llvm-
@@ -1873,6 +1873,17 @@ DWARFASTParserClang::ParseStructureLikeDIE(const
SymbolContext &sc,
clang_type =
m_ast.CreateClassTemplateSpecializationType(class_specialization_decl);
Michael137 wrote:
lets move the check before we call `CreateClassTemplat
@@ -1676,6 +1676,13 @@ TypeSystemClang::CreateClassTemplateSpecializationDecl(
class_template_specialization_decl->setInstantiationOf(class_template_decl);
class_template_specialization_decl->setTemplateArgs(
TemplateArgumentList::CreateCopy(ast, args));
+ // Special
Michael137 wrote:
> @Michael137, do you think that this PR(#154134) is still needed?
I'll comment on that PR
https://github.com/llvm/llvm-project/pull/154123
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mail
@@ -1676,6 +1676,13 @@ TypeSystemClang::CreateClassTemplateSpecializationDecl(
class_template_specialization_decl->setInstantiationOf(class_template_decl);
class_template_specialization_decl->setTemplateArgs(
TemplateArgumentList::CreateCopy(ast, args));
+ // Special
@@ -1873,6 +1873,17 @@ DWARFASTParserClang::ParseStructureLikeDIE(const
SymbolContext &sc,
clang_type =
m_ast.CreateClassTemplateSpecializationType(class_specialization_decl);
+if (!class_specialization_decl) {
+ if (log) {
+dwarf->GetObjectFile()
Michael137 wrote:
friendly ping
https://github.com/llvm/llvm-project/pull/155485
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -1873,6 +1873,19 @@ DWARFASTParserClang::ParseStructureLikeDIE(const
SymbolContext &sc,
clang_type =
m_ast.CreateClassTemplateSpecializationType(class_specialization_decl);
+// If we got nullptr, then such specialization already exists.
Mic
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/154123
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -1873,6 +1874,24 @@ DWARFASTParserClang::ParseStructureLikeDIE(const
SymbolContext &sc,
clang_type =
m_ast.CreateClassTemplateSpecializationType(class_specialization_decl);
+// Try to find an existing specialization with these template arguments and
+//
@@ -2125,9 +2125,9 @@ bool DWARFASTParserClang::ParseTemplateDIE(
if (tag == DW_TAG_template_value_parameter && uval64_valid) {
if (auto value = MakeAPValue(ast, clang_type, uval64)) {
template_param_infos.InsertArg(
- name, clang::TemplateA
@@ -48,7 +48,7 @@ int main() {
// COMMON-SAME: templateParams: ![[PARAM:[0-9]+]]
// COMMON: ![[PARAM]] = !{![[TEMPL_TYPE_PARAM:[0-9]+]]}
// GDB: ![[TEMPL_TYPE_PARAM]] = !DITemplateTypeParameter(name: "T",
type: ![[BAR_INT_TY]])
-// LLDB:![[TEMPL_TYPE_PARAM
@@ -1873,6 +1874,24 @@ DWARFASTParserClang::ParseStructureLikeDIE(const
SymbolContext &sc,
clang_type =
m_ast.CreateClassTemplateSpecializationType(class_specialization_decl);
+// Try to find an existing specialization with these template arguments and
+//
@@ -0,0 +1,675 @@
+# struct Type {};
+#
+# template struct _Optional_payload;
+#
+# template struct _Optional_payload<_Tp, true, false, false> {};
+#
+# template
+# struct _Optional_payload<_Tp, false, _Copy, _Move>
+# : _Optional_payload<_Tp, true, false, false> {};
+#
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/154123
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -1873,6 +1874,24 @@ DWARFASTParserClang::ParseStructureLikeDIE(const
SymbolContext &sc,
clang_type =
m_ast.CreateClassTemplateSpecializationType(class_specialization_decl);
+// Try to find an existing specialization with these template arguments and
+//
https://github.com/Michael137 closed
https://github.com/llvm/llvm-project/pull/156050
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/133079
>From 2314f9e584d736ce2093cc196c7c57c2087cde42 Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Wed, 26 Mar 2025 12:54:36 +
Subject: [PATCH 1/3] [lldb][Instrumentation] Set selected frame to outside
san
https://github.com/Michael137 milestoned
https://github.com/llvm/llvm-project/pull/156050
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Michael137 wrote:
> Some targets like PowerPC store their
Is there some text missing here?
https://github.com/llvm/llvm-project/pull/157198
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-
https://github.com/Michael137 approved this pull request.
https://github.com/llvm/llvm-project/pull/157196
___
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/157198
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Michael137 wrote:
/cherry-pick 4b362f152e58abd6aeed5d603a6dfc10115ed1ab
https://github.com/llvm/llvm-project/pull/156050
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Michael137 wrote:
> The other scenario is accessing an extant C global variable (maybe even one
> you don't have debug info for) called "class" or "namespace". But more
> importantly, having this sort of secret out that is applied inconsistently
> will just make lldb more confusing. It's fine
@@ -30,9 +30,14 @@ class ArchitecturePPC64 : public Architecture {
void AdjustBreakpointAddress(const Symbol &func,
Address &addr) const override;
+ lldb::ByteOrder GetVectorElementOrder() const override;
+
private:
static std::unique_ptr
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/155000
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Michael137 wrote:
> We could potentially also drop the char_traits thingy.
Yea was thinking the same. Might do that as a follow-up. That seems like a more
niche use-case, but probably something people do
https://github.com/llvm/llvm-project/pull/156050
_
@@ -644,7 +723,25 @@ bool IRInterpreter::CanInterpret(llvm::Module &module,
llvm::Function &function,
switch (operand_type->getTypeID()) {
default:
break;
-case Type::FixedVectorTyID:
+case Type::FixedVectorTyID: {
Mic
Michael137 wrote:
> Any word on this? Dead/can't be done? Lowered priority/might revisit later?
> Any conclusions/lessons that'd be handy to record for posterity?
The performance implications of the way this was implemented turned out to be
quite difficult to overcome. The way we wanted to add
https://github.com/Michael137 requested changes to this pull request.
Marking as needs changes so we don't forget to move the PDB support code into
LLVM
https://github.com/llvm/llvm-project/pull/149701
___
lldb-commits mailing list
lldb-commits@lists.
https://github.com/Michael137 commented:
For testing the endianess, I'm not sure we have PPC LLDB buildbots that would
test this (@DavidSpickett or @JDevlieghere might know). Any chance this can be
tested with unit-tests somehow? Any chance we can just pass an LLVM IR module
into the IRInterpr
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/155000
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -367,7 +379,67 @@ class InterpreterStackFrame {
return true;
}
- bool ResolveConstant(lldb::addr_t process_address, const Constant *constant)
{
+ bool ResolveVectorConstant(lldb::addr_t process_address,
+ const Constant *constant,
+
@@ -1564,7 +1660,103 @@ bool IRInterpreter::Interpret(llvm::Module &module,
llvm::Function &function,
returnVal = value.GetScalar();
// Push the return value as the result
-frame.AssignValue(inst, returnVal, module);
+frame.AssignValue(inst, re
@@ -1564,7 +1660,103 @@ bool IRInterpreter::Interpret(llvm::Module &module,
llvm::Function &function,
returnVal = value.GetScalar();
// Push the return value as the result
-frame.AssignValue(inst, returnVal, module);
+frame.AssignValue(inst, re
@@ -30,9 +30,14 @@ class ArchitecturePPC64 : public Architecture {
void AdjustBreakpointAddress(const Symbol &func,
Address &addr) const override;
+ lldb::ByteOrder GetVectorElementOrder() const override;
+
private:
static std::unique_ptr
https://github.com/Michael137 ready_for_review
https://github.com/llvm/llvm-project/pull/156050
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/156050
>From 35999d8d509864795dd36565d12ddea425a98c22 Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Fri, 29 Aug 2025 16:57:35 +0100
Subject: [PATCH 1/6] [lldb][DataFormatter] Allow std::string formatters to
mat
Michael137 wrote:
I suspect this broke `UnsupportedLanguage.test` on the lldb-x86_64-win bot:
https://lab.llvm.org/buildbot/#/builders/211/builds/1840
Though don't have a machine to confirm that.
```
TEST 'lldb-shell :: Process/UnsupportedLanguage.test'
FAILED ***
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/156050
>From 35999d8d509864795dd36565d12ddea425a98c22 Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Fri, 29 Aug 2025 16:57:35 +0100
Subject: [PATCH 1/5] [lldb][DataFormatter] Allow std::string formatters to
mat
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/156050
>From 35999d8d509864795dd36565d12ddea425a98c22 Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Fri, 29 Aug 2025 16:57:35 +0100
Subject: [PATCH 1/7] [lldb][DataFormatter] Allow std::string formatters to
mat
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/133079
>From 2314f9e584d736ce2093cc196c7c57c2087cde42 Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Wed, 26 Mar 2025 12:54:36 +
Subject: [PATCH 1/4] [lldb][Instrumentation] Set selected frame to outside
san
Michael137 wrote:
Latest commit narrows the heuristics so it specifically applies to `CallExpr`s
generated for structured bindings. Here's what the AST looks like when stopped
in `IgnoreImplicitCallSingleStep` for a structured binding:
```
(lldb) p E->dump()
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/156050
>From 35999d8d509864795dd36565d12ddea425a98c22 Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Fri, 29 Aug 2025 16:57:35 +0100
Subject: [PATCH 1/2] [lldb][DataFormatter] Allow std::string formatters to
mat
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/156050
>From 35999d8d509864795dd36565d12ddea425a98c22 Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Fri, 29 Aug 2025 16:57:35 +0100
Subject: [PATCH 1/4] [lldb][DataFormatter] Allow std::string formatters to
mat
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/133079
>From 3d5ee1252efe24bf2c72fb4e155f8042e7334aba Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Wed, 26 Mar 2025 12:54:36 +
Subject: [PATCH 1/2] [lldb][Instrumentation] Set selected frame to outside
san
Michael137 wrote:
> The only bad effect I can see just from looking at the patch is if I had
> selected frame 5 of a given thread, and then had run an expression. I want to
> make sure that when the expression is done running, we don't set the selected
> frame back to 0 (or whatever the most r
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/122265
>From a9e13ad8d2a7a95d431dddcced611bea1e83b99a Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Thu, 9 Jan 2025 10:01:31 +
Subject: [PATCH 01/13] [clang][DebugInfo] Expand detection of structured
bindin
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/122265
>From a9e13ad8d2a7a95d431dddcced611bea1e83b99a Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Thu, 9 Jan 2025 10:01:31 +
Subject: [PATCH 01/12] [clang][DebugInfo] Expand detection of structured
bindin
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/122265
>From a9e13ad8d2a7a95d431dddcced611bea1e83b99a Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Thu, 9 Jan 2025 10:01:31 +
Subject: [PATCH 01/11] [clang][DebugInfo] Expand detection of structured
bindin
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/122265
>From a9e13ad8d2a7a95d431dddcced611bea1e83b99a Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Thu, 9 Jan 2025 10:01:31 +
Subject: [PATCH 01/10] [clang][DebugInfo] Expand detection of structured
bindin
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/133078
>From 4a0d13ef2751071505ab797c63c2ee20d14a6c61 Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Wed, 26 Mar 2025 13:20:24 +
Subject: [PATCH 1/4] [lldb][Target] Clear selected frame index after a
StopInf
Michael137 wrote:
> How do we explain that for my global variable `class`,
>
> (lldb) expr class = 110
>
> works when I happen to be stopped in a C frame, but if I can't find a C frame
> around there's no way I can replicate that behavior?
> How do we explain that for my global variable `clas
https://github.com/Michael137 closed
https://github.com/llvm/llvm-project/pull/156681
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Michael137 wrote:
/cherry-pick a862225813c251c28b085603b7d32d4b111dbc57
https://github.com/llvm/llvm-project/pull/156681
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/Michael137 milestoned
https://github.com/llvm/llvm-project/pull/156681
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -44,18 +44,21 @@ Status
CommandObjectExpression::CommandOptions::SetOptionValue(
const int short_option = GetDefinitions()[option_idx].short_option;
switch (short_option) {
- case 'l':
+ case 'l': {
language = Language::GetLanguageTypeFromString(option_arg);
-
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/156648
>From 5689f9e8489c66237097891e98aba93571f8583f Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Wed, 3 Sep 2025 12:19:17 +0100
Subject: [PATCH 1/3] [lldb][Expression] Reject languages not supported by
TypeS
Michael137 wrote:
> BTW, if this patch breaks that feature w/o some test catching it, we should
> add a test for these allowed keywords.
This patch doesn't break said feature. It only breaks if you explicitly pass
`expression --language C`, and expect it to work. If it's a C frame, then
`expr
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/156050
>From 35999d8d509864795dd36565d12ddea425a98c22 Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Fri, 29 Aug 2025 16:57:35 +0100
Subject: [PATCH 1/3] [lldb][DataFormatter] Allow std::string formatters to
mat
https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/122265
>From a9e13ad8d2a7a95d431dddcced611bea1e83b99a Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Thu, 9 Jan 2025 10:01:31 +
Subject: [PATCH 1/9] [clang][DebugInfo] Expand detection of structured
bindings
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/156648
___
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/156648
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/Michael137 closed
https://github.com/llvm/llvm-project/pull/156642
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Michael137 wrote:
> I think this is fine, but that FIXME is curious. We don't support running in
> non-C++ modes, so switching on `frame_lang` would only get us a more precise
> C++ standard version, but do we use that?
Yea it was me who added the FIXME back when I was refactoring some of this
Michael137 wrote:
> I'm still happy with this. I did not notice it was not merged
Yea I was just clearing out my open PRs and noticed this was still open. There
is still that clang-tidy failure I haven't figured out. Might need a more
targeted heuristic for expressions coming from binding decl
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/133079
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/Michael137 ready_for_review
https://github.com/llvm/llvm-project/pull/133079
___
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/133079
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Michael137 wrote:
Closing in favour of https://github.com/llvm/llvm-project/pull/156681
https://github.com/llvm/llvm-project/pull/130768
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-comm
1 - 100 of 2350 matches
Mail list logo
