[Lldb-commits] [lldb] [lldb-mcp] Auto connect to the first running lldb mcp instance. (PR #157503)

2025-09-11 Thread via lldb-commits
llvmbot wrote: @llvm/pr-subscribers-lldb Author: John Harrison (ashgti) Changes This improves the flow by automatically connecting to an exisitng lldb instance, if one is found. Future improvements include: * Launching a binary if an instance isn't detected. * Multiplexing if multiple i

[Lldb-commits] [lldb] [lldb][TypeSystemClang] Added unique builtins types for __bf16 and _Float16 (PR #157674)

2025-09-11 Thread Michael Buch via lldb-commits
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

[Lldb-commits] [lldb] [RISCV-LLDB] RISCV feature attribute support and allows overriding additional(default) feature (PR #147990)

2025-09-11 Thread David Spickett via lldb-commits
DavidSpickett wrote: Ultimately I won't be casting the deciding vote here as I don't work with RISC-V systems. Someone from Apple or perhaps others like @dlav-sc who do use them should have the final approvals. https://github.com/llvm/llvm-project/pull/147990 __

[Lldb-commits] [lldb] [lldb][test] Prevent TestqOffsets.py picking up host binaries (PR #157432)

2025-09-11 Thread Pavel Labath via lldb-commits
https://github.com/labath commented: would `runCmd("platform select remote-linux")` have the same effect? That would make the test more realistic, as that's how you would debug a non-local process. The problem right now is that lldb selects the host platform by default (you could argue that it

[Lldb-commits] [lldb] [lldb][DWARFASTParserClang] Added a check for the specialization existence (PR #154123)

2025-09-11 Thread Michael Buch via 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" &&

[Lldb-commits] [lldb] [lldb][DWARFASTParserClang] Added a check for the specialization existence (PR #154123)

2025-09-11 Thread Michael Buch via lldb-commits
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

[Lldb-commits] [lldb] 54b3dc1 - [lldb][NFC] Fix style issues with StackID.h (#157483)

2025-09-11 Thread via lldb-commits
Author: Felipe de Azevedo Piovezan Date: 2025-09-08T08:47:42-07:00 New Revision: 54b3dc1dad9a28709e880b54a416cdcdb624cad4 URL: https://github.com/llvm/llvm-project/commit/54b3dc1dad9a28709e880b54a416cdcdb624cad4 DIFF: https://github.com/llvm/llvm-project/commit/54b3dc1dad9a28709e880b54a416cdcdb

[Lldb-commits] [clang] [lldb] [lldb][Expression] Add structor variant to LLDB's function call labels (PR #149827)

2025-09-11 Thread Michael Buch via 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

[Lldb-commits] [lldb] [lldb][DWARFASTParserClang] Added a check for the specialization existence (PR #154123)

2025-09-11 Thread via lldb-commits
@@ -599,6 +599,39 @@ TEST_F(DWARFASTParserClangTests, TestDefaultTemplateParamParsing) { } } +TEST_F(DWARFASTParserClangTests, TestSpecDeclExistsError) { + // Tests checking error if ClassTemplateSpecializationDecl already exists. tgs-sc wrote: Addressed

[Lldb-commits] [lldb] [lldb][Instrumentation] Set selected frame to outside sanitizer libraries (PR #133079)

2025-09-11 Thread Michael Buch via lldb-commits
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

[Lldb-commits] [lldb] [lldb][DWARFASTParserClang] Added a check for the specialization existence (PR #154123)

2025-09-11 Thread via 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 +//

[Lldb-commits] [lldb] [RISCV-LLDB] RISCV feature attribute support and allows overriding additional(default) feature (PR #147990)

2025-09-11 Thread David Spickett via lldb-commits
@@ -0,0 +1,10 @@ +# REQUIRES: target=riscv32-{{.*}} + +# RUN: %lldb -b -o "disassemble -b -n main" %p/Inputs/riscv_xqci.out | FileCheck %s + +# CHECK: 051f 0058 qc.e.li a0, 0x58 +# CHECK: 1516 qc.extu a0, a0, 0x6, 0x0 + +# RUN: %lldb -b -o "disassemble -b -n ma

[Lldb-commits] [lldb] [lldb][test] StepUntil disable test for unsupported linkers. (PR #157474)

2025-09-11 Thread via 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 origin/main...HEAD lldb/packages/Python/lldbsuite/test/lldbplatformutil.py lldb/test/API/

[Lldb-commits] [lldb] [lldb][Expression] Reject languages not supported by TypeSystems for expression evaluation (PR #156648)

2025-09-11 Thread via lldb-commits
jimingham wrote: > > I don't remember how that works in swift, but note, there are two separate > > issues with the C family expression parser. One was making local > > declarations override ivar references in name resolution in the context of > > the expression. That is why we inject variable

[Lldb-commits] [lldb] [lldb][DWARFASTParserClang] Added a check for the specialization existence (PR #154123)

2025-09-11 Thread Michael Buch via 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

[Lldb-commits] [lldb] [lldb][Instrumentation] Set selected frame to outside sanitizer libraries (PR #133079)

2025-09-11 Thread Michael Buch via lldb-commits
@@ -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

[Lldb-commits] [lldb] [lldb-dap] Add invalidated event (PR #157530)

2025-09-11 Thread Walter Erquinigo via lldb-commits
https://github.com/walter-erquinigo edited https://github.com/llvm/llvm-project/pull/157530 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][test] Re-enable import-std-module tests on Linux (PR #157649)

2025-09-11 Thread via lldb-commits
llvmbot wrote: @llvm/pr-subscribers-lldb Author: Michael Buch (Michael137) Changes 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