[Lldb-commits] [lldb] [LLDB] Process minidump is in memory check command (PR #149401)

2025-07-24 Thread David Spickett via lldb-commits
DavidSpickett wrote: Would this be helped with better error messages? ``` $ ./bin/lldb /tmp/test.o --core core.test.o.2152361.tcwg-jade-03-dev.1753366442 (lldb) target create "/tmp/test.o" --core "core.test.o.2152361.tcwg-jade-03-dev.1753366442" Core file '/home/david.spickett/build-llvm-aarc

[Lldb-commits] [lldb] [lldb] Add WebAssembly Process Plugin (PR #150143)

2025-07-24 Thread David Spickett via lldb-commits
@@ -0,0 +1,81 @@ +//===--===// +// +// 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: Apac

[Lldb-commits] [lldb] [lldb] Add WebAssembly Process Plugin (PR #150143)

2025-07-24 Thread David Spickett via lldb-commits
@@ -0,0 +1,127 @@ +//===--===// +// +// 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: Apa

[Lldb-commits] [lldb] [lldb] Add WebAssembly Process Plugin (PR #150143)

2025-07-24 Thread David Spickett via lldb-commits
@@ -0,0 +1,127 @@ +//===--===// +// +// 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: Apa

[Lldb-commits] [lldb] [lldb] Add WebAssembly Process Plugin (PR #150143)

2025-07-24 Thread David Spickett via lldb-commits
DavidSpickett wrote: Certainly lldb being in this conversation early would be a good way to prevent the reliance on GDB specific behaviour that often happens in the native debugging world. https://github.com/llvm/llvm-project/pull/150143 ___ lldb-com

[Lldb-commits] [lldb] [lldb] Add WebAssembly Process Plugin (PR #150143)

2025-07-24 Thread David Spickett via lldb-commits
DavidSpickett wrote: Long term, if the WASM standard makes recommendations for debugging then that's great. You are correct there. Short term, users will need to know what to expect so we need a short, simple, answer to "I tried lldb with whatever runtime and it didn't work". Which ideally is

[Lldb-commits] [lldb] [lldb] Allow building using Mingw-w64 on Windows. (PR #150382)

2025-07-24 Thread David Spickett via lldb-commits
DavidSpickett wrote: Please configure your email settings as explained here https://llvm.org/docs/DeveloperPolicy.html#email-addresses, and then we will merge this. https://github.com/llvm/llvm-project/pull/150382 ___ lldb-commits mailing list lldb-c

[Lldb-commits] [lldb] [lldb] Allow building using Mingw-w64 on Windows. (PR #150382)

2025-07-24 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett approved this pull request. Seems harmless enough, thanks for the fix. Surprised @mstorsjo didn't already hit this, but maybe they will appreciate the fix. https://github.com/llvm/llvm-project/pull/150382 ___ lldb-com

[Lldb-commits] [lldb] [lldb] Add WebAssembly Process Plugin (PR #150143)

2025-07-24 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett deleted https://github.com/llvm/llvm-project/pull/150143 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Add WebAssembly Process Plugin (PR #150143)

2025-07-24 Thread David Spickett via lldb-commits
@@ -323,6 +323,11 @@ ProcessGDBRemote::~ProcessGDBRemote() { KillDebugserverProcess(); } +std::shared_ptr DavidSpickett wrote: `static` https://github.com/llvm/llvm-project/pull/150143 ___ lldb-commits mailing li

[Lldb-commits] [lldb] [lldb] Add WebAssembly Process Plugin (PR #150143)

2025-07-24 Thread David Spickett via lldb-commits
DavidSpickett wrote: > process connect --plugin wasm connect://localhost:4567 On the original review, auto-detection of the plugin wasn't working which meant this `plugin wasm` was needed. Did you ever find out what was the cause? I presume lldb still has a local copy of the program file, so i

[Lldb-commits] [lldb] [lldb] Add WebAssembly Process Plugin (PR #150143)

2025-07-24 Thread David Spickett via lldb-commits
DavidSpickett wrote: Also can we get a vague idea of how many more PRs are in the stack for this? Just so save me asking "why is X missing" over and over. https://github.com/llvm/llvm-project/pull/150143 ___ lldb-commits mailing list lldb-commits@list

[Lldb-commits] [lldb] [lldb] Add WebAssembly Process Plugin (PR #150143)

2025-07-24 Thread David Spickett via lldb-commits
DavidSpickett wrote: This is an overly harsh way of phrasing it, but hopefully it gives you an idea of why I'm asking for an overview of this effort: * We already have at least one barely tested architecture, I would like to avoid picking up another one. I think WASM is cool (all WASM userspac

[Lldb-commits] [lldb] [lldb] Add WebAssembly Process Plugin (PR #150143)

2025-07-24 Thread David Spickett via lldb-commits
DavidSpickett wrote: > I've been told that the runtime can have a different stack (separate from the > native stack) for the Wasm code and also that the format of call frames may > not match native stack frame ABI. Sure, so this is like qemu-user? You want to debug the wasm process being host

[Lldb-commits] [clang] [lldb] [lldb-dap] Add performance optimization options to improve launch times (PR #150365)

2025-07-24 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett requested changes to this pull request. I'm going to go out on a limb and suggest that a certain sort of assistive interactive tool was used to generate this. Which I don't venture an opinion on in general but in this case, it's didn't do so well. A glance at t

[Lldb-commits] [clang] [clang-tools-extra] [lldb] [llvm] [mlir] [NFC] Fix typos 'seperate' -> 'separate' (PR #144368)

2025-07-24 Thread David Spickett via lldb-commits
DavidSpickett wrote: Please change your email settings as explained in https://llvm.org/docs/DeveloperPolicy.html#email-addresses. Right now it would be merged with a GitHub noreply address. https://github.com/llvm/llvm-project/pull/144368 ___ lldb-c

[Lldb-commits] [clang] [clang-tools-extra] [lldb] [llvm] [mlir] [NFC] Fix typos 'seperate' -> 'separate' (PR #144368)

2025-07-24 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett approved this pull request. LGTM, thanks! https://github.com/llvm/llvm-project/pull/144368 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Add WebAssembly Process Plugin (PR #150143)

2025-07-23 Thread David Spickett via lldb-commits
DavidSpickett wrote: How does this relate to / overlap with existing PRs https://github.com/llvm/llvm-project/pull/77949 and https://github.com/llvm/llvm-project/pull/78977? They are unlikely to get merged by the authors, so I assume you'll make your own equivalent. > qWasmCallStack This is

[Lldb-commits] [lldb] [lldb] account for registers being host endian when casting values (PR #150011)

2025-07-22 Thread David Spickett via lldb-commits
DavidSpickett wrote: First commit is https://github.com/llvm/llvm-project/pull/148836, please review the second onward. https://github.com/llvm/llvm-project/pull/150011 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/c

[Lldb-commits] [lldb] [lldb] account for registers being host endian when casting values (PR #150011)

2025-07-22 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett created https://github.com/llvm/llvm-project/pull/150011 Fixes https://github.com/llvm/llvm-project/issues/135707 Follow up to https://github.com/llvm/llvm-project/pull/148836 which fixed some of this issue but not all of it. Our Value/ValueObject system does

[Lldb-commits] [lldb] [lldb][rpc] Use Clang attributes to keep track of pointer plus len (PR #148981)

2025-07-22 Thread David Spickett via lldb-commits
@@ -69,6 +69,7 @@ class LLDB_API SBData { const char *GetString(lldb::SBError &error, lldb::offset_t offset); + [[clang::annotate("lldb-rpc-gen pointer plus len")]] DavidSpickett wrote: > This commit changes this by using the Clang annotation attribute to

[Lldb-commits] [lldb] [lldb][AArch64][Linux] Show MTE store only setting in mte_ctrl (PR #145033)

2025-07-21 Thread David Spickett via lldb-commits
DavidSpickett wrote: ping! https://github.com/llvm/llvm-project/pull/145033 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][AArch64] Add HWCAP3 to register field detection (PR #145029)

2025-07-21 Thread David Spickett via lldb-commits
DavidSpickett wrote: ping! https://github.com/llvm/llvm-project/pull/145029 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][AArch64] Fix arm64 hardware breakpoint/watchpoint to arm32 process. (PR #147198)

2025-07-21 Thread David Spickett via lldb-commits
@@ -0,0 +1,23 @@ +#include "Plugins/Process/Linux/NativeProcessLinux.h" +#include "Plugins/Process/Utility/NativeRegisterContextDBReg.h" +#include "lldb/Utility/Status.h" +#include +#include +#include +#include +#include + +namespace lldb_private { +namespace process_linux {

[Lldb-commits] [lldb] [lldb][AArch64] Fix arm64 hardware breakpoint/watchpoint to arm32 process. (PR #147198)

2025-07-21 Thread David Spickett via lldb-commits
@@ -102,18 +105,8 @@ class NativeRegisterContextLinux_arm : public NativeRegisterContextLinux { uint32_t m_gpr_arm[k_num_gpr_registers_arm]; RegisterInfoPOSIX_arm::FPU m_fpr; - // Debug register info for hardware breakpoints and watchpoints management. - struct DREG { -

[Lldb-commits] [lldb] [lldb][AArch64] Fix arm64 hardware breakpoint/watchpoint to arm32 process. (PR #147198)

2025-07-21 Thread David Spickett via lldb-commits
@@ -0,0 +1,63 @@ +#include "NativeRegisterContextLinuxArm64Shared.h" + +using namespace lldb_private::process_linux::arm64; + +namespace lldb_private { +namespace process_linux { +namespace arm64 { DavidSpickett wrote: You don't need `using namespace` if you're t

[Lldb-commits] [lldb] [lldb][AArch64] Fix arm64 hardware breakpoint/watchpoint to arm32 process. (PR #147198)

2025-07-21 Thread David Spickett via lldb-commits
@@ -74,8 +75,10 @@ class NativeRegisterContextLinux_arm : public NativeRegisterContextLinux { bool WatchpointIsEnabled(uint32_t wp_index); - // Debug register type select - enum DREGType { eDREGTypeWATCH = 0, eDREGTypeBREAK }; + using DREGType = NativeRegisterContextDBR

[Lldb-commits] [lldb] [lldb][AArch64] Fix arm64 hardware breakpoint/watchpoint to arm32 process. (PR #147198)

2025-07-21 Thread David Spickett via lldb-commits
@@ -0,0 +1,63 @@ +#include "NativeRegisterContextLinuxArm64Shared.h" DavidSpickett wrote: All new .cpp and .h files need the license header, you can copy one from an existing file and modify it. https://github.com/llvm/llvm-project/pull/147198 _

[Lldb-commits] [lldb] [lldb][AArch64] Fix arm64 hardware breakpoint/watchpoint to arm32 process. (PR #147198)

2025-07-21 Thread David Spickett via lldb-commits
@@ -0,0 +1,63 @@ +#include "NativeRegisterContextLinuxArm64Shared.h" + +using namespace lldb_private::process_linux::arm64; + +namespace lldb_private { +namespace process_linux { +namespace arm64 { + +namespace { +Status ReadHardwareDebugInfoHelper(int regset, ::pid_t tid, ---

[Lldb-commits] [lldb] [lldb][AArch64] Fix arm64 hardware breakpoint/watchpoint to arm32 process. (PR #147198)

2025-07-21 Thread David Spickett via lldb-commits
@@ -8,6 +8,7 @@ add_lldb_library(lldbPluginProcessLinux NativeRegisterContextLinux.cpp NativeRegisterContextLinux_arm.cpp NativeRegisterContextLinux_arm64.cpp + NativeRegisterContextLinuxArm64Shared.cpp DavidSpickett wrote: We have the term "dbreg" in u

[Lldb-commits] [lldb] [lldb][AArch64] Fix arm64 hardware breakpoint/watchpoint to arm32 process. (PR #147198)

2025-07-21 Thread David Spickett via lldb-commits
@@ -0,0 +1,63 @@ +#include "NativeRegisterContextLinuxArm64Shared.h" DavidSpickett wrote: Also the top of this file needs a comment explaining why it needed to be split out. https://github.com/llvm/llvm-project/pull/147198 __

[Lldb-commits] [lldb] [lldb] Proofread formatting.rst (PR #149768)

2025-07-21 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett approved this pull request. With the lack of apostrophes fixed, this LGTM. https://github.com/llvm/llvm-project/pull/149768 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailma

[Lldb-commits] [lldb] [lldb] Proofread formatting.rst (PR #149768)

2025-07-21 Thread David Spickett via lldb-commits
@@ -203,7 +203,7 @@ Scoping Many times the information that you might have in your prompt might not be available and you won``t want it to print out if it isn``t valid. To take care DavidSpickett wrote: You can fix the uses double back tick instead of apostro

[Lldb-commits] [lldb] [lldb] Correct spacing of = {...} when depth limit is hit (PR #149480)

2025-07-18 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett closed https://github.com/llvm/llvm-project/pull/149480 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [LLDB] Run API tests with native PDB too (PR #149305)

2025-07-18 Thread David Spickett via lldb-commits
DavidSpickett wrote: https://github.com/llvm/llvm-project/issues/114906 as well. https://github.com/llvm/llvm-project/pull/149305 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [LLDB] Run API tests with native PDB too (PR #149305)

2025-07-18 Thread David Spickett via lldb-commits
DavidSpickett wrote: Opened https://github.com/llvm/llvm-project/issues/149498 so you can refer to it. I think it's a reasonable pitch to say: * Formatters that work with PDB are a good idea. * If we opt-in those formatters for testing using PDB, they will be run on at least Linaro's Windows o

[Lldb-commits] [lldb] [LLDB] Run API tests with native PDB too (PR #149305)

2025-07-18 Thread David Spickett via lldb-commits
DavidSpickett wrote: I ran this on Linaro's bot machine and got these results: ``` Unresolved Tests (5): lldb-api :: functionalities/data-formatter/data-formatter-stl/generic/string/TestDataFormatterStdString.py lldb-api :: lang/c/vla/TestVLA.py lldb-api :: python_api/

[Lldb-commits] [lldb] [lldb][windows] force the console to use a UTF-8 codepage (PR #149493)

2025-07-18 Thread David Spickett via lldb-commits
DavidSpickett wrote: Windows Terminal is the default on Windows 10 at least. I think buildbot launches things in conhost, but if utf-8 there was a problem for tests, we would have seen it before now. https://github.com/llvm/llvm-project/pull/149493 _

[Lldb-commits] [lldb] [lldb][windows] force the console to use a UTF-8 codepage (PR #149493)

2025-07-18 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett edited https://github.com/llvm/llvm-project/pull/149493 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [windows][lldb] force the console to use a UTF-8 codepage (PR #149493)

2025-07-18 Thread David Spickett via lldb-commits
DavidSpickett wrote: I opened an issue for this https://github.com/llvm/llvm-project/issues/142568. Where @Nerixyz mentions that SetConsoleOutputCP might have problems in cmd.exe (which probably means conhost, the original windows terminal host, as opposed to "windows terminal", the new one).

[Lldb-commits] [lldb] [lldb][Expression][NFC] Make LoadAddressResolver::m_target a reference (PR #149490)

2025-07-18 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/149490 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Correct spacing of = {...} when depth limit is hit (PR #149480)

2025-07-18 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett created https://github.com/llvm/llvm-project/pull/149480 In some places it was printing "= {...}" and some "={...}" with no space. I think the space looks nicer so do that in both cases. >From 3c6241d1ed5e5b8d8c86447a98bed2f407f50ffb Mon Sep 17 00:00:00 2001 Fr

[Lldb-commits] [lldb] 88721d6 - [lldb][test] Fix PDB UdtLayoutTest

2025-07-18 Thread David Spickett via lldb-commits
Author: David Spickett Date: 2025-07-18T08:26:00Z New Revision: 88721d63d482cf8f95deb66e74462b2cf583be8f URL: https://github.com/llvm/llvm-project/commit/88721d63d482cf8f95deb66e74462b2cf583be8f DIFF: https://github.com/llvm/llvm-project/commit/88721d63d482cf8f95deb66e74462b2cf583be8f.diff LOG

[Lldb-commits] [lldb] [lldb][test] Fix TestChildCountTruncation on Windows (PR #149322)

2025-07-17 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett closed https://github.com/llvm/llvm-project/pull/149322 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][test] Fix TestChildCountTruncation on Windows (PR #149322)

2025-07-17 Thread David Spickett via lldb-commits
DavidSpickett wrote: @Michael137 Does this actually need DWARF, or will any debug info format do? https://github.com/llvm/llvm-project/pull/149322 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo

[Lldb-commits] [lldb] [lldb][test] Fix TestChildCountTruncation on Windows (PR #149322)

2025-07-17 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett edited https://github.com/llvm/llvm-project/pull/149322 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][test] Fix TestChildCountTruncation on Windows (PR #149322)

2025-07-17 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett created https://github.com/llvm/llvm-project/pull/149322 By not forcing the DWARF debug info format. When left as the default, the tests pass. >From 092168660b280c0e1a1f6956bddd11899d5bafcc Mon Sep 17 00:00:00 2001 From: David Spickett Date: Thu, 17 Jul 2025 1

[Lldb-commits] [lldb] ba5f31c - [lldb][test] Disable TestChildCountTruncation on Windows

2025-07-17 Thread David Spickett via lldb-commits
Author: David Spickett Date: 2025-07-17T14:30:50Z New Revision: ba5f31cfaa2452a4a94a482b53d899d6f2ee0e66 URL: https://github.com/llvm/llvm-project/commit/ba5f31cfaa2452a4a94a482b53d899d6f2ee0e66 DIFF: https://github.com/llvm/llvm-project/commit/ba5f31cfaa2452a4a94a482b53d899d6f2ee0e66.diff LOG

[Lldb-commits] [lldb] 756e077 - [lldb][test] Adjust TestTypeList.py on Windows with exceptions

2025-07-17 Thread David Spickett via lldb-commits
Author: David Spickett Date: 2025-07-17T14:24:05Z New Revision: 756e07734b4f2aa64bd1bd5b6a29389c4c642323 URL: https://github.com/llvm/llvm-project/commit/756e07734b4f2aa64bd1bd5b6a29389c4c642323 DIFF: https://github.com/llvm/llvm-project/commit/756e07734b4f2aa64bd1bd5b6a29389c4c642323.diff LOG

[Lldb-commits] [lldb] [LLDB] Run API tests with native PDB too (PR #149305)

2025-07-17 Thread David Spickett via lldb-commits
DavidSpickett wrote: Yes this is what I was thinking of. Don't go diving into the failures yet, I'll try this on the bot machine first and see if we need other adjustments. https://github.com/llvm/llvm-project/pull/149305 ___ lldb-commits mailing list

[Lldb-commits] [lldb] Reland "[lldb][RPC] Upstream lldb-rpc-gen tool" (#146969)" Attempt 2 (PR #148996)

2025-07-17 Thread David Spickett via lldb-commits
DavidSpickett wrote: You should email the worker admin and see if they can help. "Worker:" tab then "admin". It is doing remote testing but the failure seems to be before that step so I think it's something to do with the host. https://github.com/llvm/llvm-project/pull/148996

[Lldb-commits] [lldb] [lldb][rpc] Use Clang attributes to keep track of pointer plus len (PR #148981)

2025-07-17 Thread David Spickett via lldb-commits
@@ -0,0 +1,469 @@ +//===-- RPCCommon.cpp -===// +// +// 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

[Lldb-commits] [lldb] [lldb][rpc] Use Clang attributes to keep track of pointer plus len (PR #148981)

2025-07-17 Thread David Spickett via lldb-commits
@@ -0,0 +1,469 @@ +//===-- RPCCommon.cpp -===// +// +// 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

[Lldb-commits] [lldb] [lldb][rpc] Use Clang attributes to keep track of pointer plus len (PR #148981)

2025-07-17 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett commented: What happens when you don't put this annotation on one of these? It generates something I suppose, but it fails when you use lldb-rpc? https://github.com/llvm/llvm-project/pull/148981 ___ lldb-commits mailin

[Lldb-commits] [lldb] [lldb][rpc] Use Clang attributes to keep track of pointer plus len (PR #148981)

2025-07-17 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett edited https://github.com/llvm/llvm-project/pull/148981 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][rpc] Upstream RPC Client Library Emitters (PR #147655)

2025-07-17 Thread David Spickett via lldb-commits
@@ -0,0 +1,124 @@ +#include "RPCLibraryHeaderEmitter.h" +#include "RPCCommon.h" + +#include "clang/AST/AST.h" +#include "clang/AST/Mangle.h" +#include "clang/Frontend/CompilerInstance.h" + +#include "llvm/ADT/StringRef.h" +#include "llvm/Support/ToolOutputFile.h" +#include "llvm/S

[Lldb-commits] [lldb] [lldb][rpc] Upstream RPC Client Library Emitters (PR #147655)

2025-07-17 Thread David Spickett via lldb-commits
@@ -0,0 +1,542 @@ +#include "RPCLibrarySourceEmitter.h" +#include "RPCCommon.h" + +#include "clang/AST/AST.h" +#include "clang/Frontend/CompilerInstance.h" + +#include "llvm/ADT/STLExtras.h" +#include "llvm/ADT/SmallString.h" +#include "llvm/ADT/StringRef.h" +#include "llvm/Suppor

[Lldb-commits] [lldb] [lldb][rpc] Upstream RPC Client Library Emitters (PR #147655)

2025-07-17 Thread David Spickett via lldb-commits
@@ -0,0 +1,20 @@ +#ifndef LLDB_API_SBRPC_CHECKARRAYPTR_H +#define LLDB_API_SBRPC_CHECKARRAYPTR_H + +#include +#include + +#include "lldb/API/SBDefines.h" + +namespace lldb { +class LLDB_API SBRPC_CHECKARRAYPTR { DavidSpickett wrote: Not sure about the ALLCAPS n

[Lldb-commits] [lldb] [lldb][rpc] Upstream RPC Client Library Emitters (PR #147655)

2025-07-17 Thread David Spickett via lldb-commits
@@ -0,0 +1,542 @@ +#include "RPCLibrarySourceEmitter.h" +#include "RPCCommon.h" + +#include "clang/AST/AST.h" +#include "clang/Frontend/CompilerInstance.h" + +#include "llvm/ADT/STLExtras.h" +#include "llvm/ADT/SmallString.h" +#include "llvm/ADT/StringRef.h" +#include "llvm/Suppor

[Lldb-commits] [lldb] [lldb][rpc] Upstream RPC Client Library Emitters (PR #147655)

2025-07-17 Thread David Spickett via lldb-commits
@@ -0,0 +1,542 @@ +#include "RPCLibrarySourceEmitter.h" +#include "RPCCommon.h" + +#include "clang/AST/AST.h" +#include "clang/Frontend/CompilerInstance.h" + +#include "llvm/ADT/STLExtras.h" +#include "llvm/ADT/SmallString.h" +#include "llvm/ADT/StringRef.h" +#include "llvm/Suppor

[Lldb-commits] [lldb] [lldb][rpc] Upstream RPC Client Library Emitters (PR #147655)

2025-07-17 Thread David Spickett via lldb-commits
@@ -0,0 +1,20 @@ +#ifndef LLDB_API_SBRPC_CHECKARRAYPTR_H +#define LLDB_API_SBRPC_CHECKARRAYPTR_H + +#include +#include + +#include "lldb/API/SBDefines.h" + +namespace lldb { +class LLDB_API SBRPC_CHECKARRAYPTR { DavidSpickett wrote: So these are just test input

[Lldb-commits] [lldb] [lldb][rpc] Upstream RPC Client Library Emitters (PR #147655)

2025-07-17 Thread David Spickett via lldb-commits
@@ -0,0 +1,124 @@ +#include "RPCLibraryHeaderEmitter.h" +#include "RPCCommon.h" + +#include "clang/AST/AST.h" +#include "clang/AST/Mangle.h" +#include "clang/Frontend/CompilerInstance.h" + +#include "llvm/ADT/StringRef.h" +#include "llvm/Support/ToolOutputFile.h" +#include "llvm/S

[Lldb-commits] [lldb] [lldb][rpc] Upstream RPC Client Library Emitters (PR #147655)

2025-07-17 Thread David Spickett via lldb-commits
@@ -0,0 +1,542 @@ +#include "RPCLibrarySourceEmitter.h" +#include "RPCCommon.h" + +#include "clang/AST/AST.h" +#include "clang/Frontend/CompilerInstance.h" + +#include "llvm/ADT/STLExtras.h" +#include "llvm/ADT/SmallString.h" +#include "llvm/ADT/StringRef.h" +#include "llvm/Suppor

[Lldb-commits] [lldb] [lldb][rpc] Upstream RPC Client Library Emitters (PR #147655)

2025-07-17 Thread David Spickett via lldb-commits
@@ -0,0 +1,542 @@ +#include "RPCLibrarySourceEmitter.h" +#include "RPCCommon.h" + +#include "clang/AST/AST.h" +#include "clang/Frontend/CompilerInstance.h" + +#include "llvm/ADT/STLExtras.h" +#include "llvm/ADT/SmallString.h" +#include "llvm/ADT/StringRef.h" +#include "llvm/Suppor

[Lldb-commits] [lldb] [lldb][rpc] Upstream RPC Client Library Emitters (PR #147655)

2025-07-17 Thread David Spickett via lldb-commits
@@ -0,0 +1,124 @@ +#include "RPCLibraryHeaderEmitter.h" +#include "RPCCommon.h" + +#include "clang/AST/AST.h" +#include "clang/AST/Mangle.h" +#include "clang/Frontend/CompilerInstance.h" + +#include "llvm/ADT/StringRef.h" +#include "llvm/Support/ToolOutputFile.h" +#include "llvm/S

[Lldb-commits] [lldb] [lldb][rpc] Upstream RPC Client Library Emitters (PR #147655)

2025-07-17 Thread David Spickett via lldb-commits
@@ -0,0 +1,542 @@ +#include "RPCLibrarySourceEmitter.h" +#include "RPCCommon.h" + +#include "clang/AST/AST.h" +#include "clang/Frontend/CompilerInstance.h" + +#include "llvm/ADT/STLExtras.h" +#include "llvm/ADT/SmallString.h" +#include "llvm/ADT/StringRef.h" +#include "llvm/Suppor

[Lldb-commits] [lldb] [lldb][rpc] Upstream RPC Client Library Emitters (PR #147655)

2025-07-17 Thread David Spickett via lldb-commits
@@ -0,0 +1,20 @@ +#ifndef LLDB_API_SBRPC_CHECKARRAYPTR_H +#define LLDB_API_SBRPC_CHECKARRAYPTR_H + +#include +#include DavidSpickett wrote: Oh these are just test inputs. So the include can be removed entirely? https://github.com/llvm/llvm-project/pull/147655

[Lldb-commits] [lldb] [lldb][rpc] Upstream RPC Client Library Emitters (PR #147655)

2025-07-17 Thread David Spickett via lldb-commits
@@ -0,0 +1,15 @@ +RUN: mkdir -p %t/server +RUN: mkdir -p %t/lib +RUN: %lldb-rpc-gen --output-dir=%t %S/../../Inputs/CheckSBPointer.h + +RUN: cat %t/lib/CheckSBPointer.cpp | FileCheck %s + +# Pointers to SB objects must be checked to +# see if they're null. If so, then a new object

[Lldb-commits] [lldb] [lldb][rpc] Upstream RPC Client Library Emitters (PR #147655)

2025-07-17 Thread David Spickett via lldb-commits
@@ -0,0 +1,20 @@ +#ifndef LLDB_API_SBRPC_CHECKARRAYPTR_H +#define LLDB_API_SBRPC_CHECKARRAYPTR_H + +#include +#include DavidSpickett wrote: I think all the cstdio includes can move into the cpp files. https://github.com/llvm/llvm-project/pull/147655 __

[Lldb-commits] [lldb] d871702 - [lldb][test] Remove XFAIL from some Windows tests

2025-07-17 Thread David Spickett via lldb-commits
Author: David Spickett Date: 2025-07-17T09:17:25Z New Revision: d87170211dc780341e42f7719c7332416f9cc290 URL: https://github.com/llvm/llvm-project/commit/d87170211dc780341e42f7719c7332416f9cc290 DIFF: https://github.com/llvm/llvm-project/commit/d87170211dc780341e42f7719c7332416f9cc290.diff LOG

[Lldb-commits] [lldb] [LLDB] Add formatters for MSVC STL std::variant (PR #148554)

2025-07-17 Thread David Spickett via lldb-commits
DavidSpickett wrote: We had a discussion about preferring one or the other PDB implementation but I don't recall the conclusion. I think this is just for reading PDB, right? Clang is going to use llvm's libraries to *write* PDB and of course MSVC is using Microsoft's implementation. I think a

[Lldb-commits] [lldb] [LLDB] Add formatters for MSVC STL std::variant (PR #148554)

2025-07-16 Thread David Spickett via lldb-commits
DavidSpickett wrote: What does running API tests with PDB look like? I assume Linaro is not doing that because we don't pass any extra flags, and often have to disable tests because link.exe is mangling the DWARF sections. Or put another way, @Nerixyz how do you make sure PDB is being used whe

[Lldb-commits] [lldb] [lldb][docs] Add section on testing with QEMU user (PR #149057)

2025-07-16 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett closed https://github.com/llvm/llvm-project/pull/149057 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb] Document MCP tools & resources (PR #148708)

2025-07-16 Thread David Spickett via lldb-commits
DavidSpickett wrote: Also, the server made it into 21.x but the docs obviously didn't and there's no release note for it. Is it complete "enough" to cherry-pick this and add a release note? https://github.com/llvm/llvm-project/pull/148708 ___ lldb-co

[Lldb-commits] [lldb] [lldb] Document MCP tools & resources (PR #148708)

2025-07-16 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett approved this pull request. LGTM with whatever is the correct thing to document for debugger IDs. https://github.com/llvm/llvm-project/pull/148708 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.

[Lldb-commits] [lldb] [lldb][docs] Add section on testing with QEMU user (PR #149057)

2025-07-16 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett edited https://github.com/llvm/llvm-project/pull/149057 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][docs] Add section on testing with QEMU user (PR #149057)

2025-07-16 Thread David Spickett via lldb-commits
@@ -167,3 +167,49 @@ The result of this is that: Your VM configuration should have ports ``54321`` and ``49140`` forwarded for this to work. + +QEMU user mode emulation + + +Serious testing of LLDB should be done using system mode emulation. The follow

[Lldb-commits] [lldb] [lldb][docs] Add section on testing with QEMU user (PR #149057)

2025-07-16 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett updated https://github.com/llvm/llvm-project/pull/149057 >From eab94453a4275cdbc5fe60607fb18a7869a2d3a8 Mon Sep 17 00:00:00 2001 From: David Spickett Date: Wed, 16 Jul 2025 10:20:59 + Subject: [PATCH 1/2] [lldb][docs] Add section on testing with QEMU user T

[Lldb-commits] [lldb] [llvm] [LLDB] Switch to using DIL as default implementation for 'frame var'. (PR #147887)

2025-07-16 Thread David Spickett via lldb-commits
DavidSpickett wrote: Perhaps with a release note saying this happened and how to switch back to the non-default way? https://github.com/llvm/llvm-project/pull/147887 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-

[Lldb-commits] [lldb] [lldb][docs] Add section on testing with QEMU user (PR #149057)

2025-07-16 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett edited https://github.com/llvm/llvm-project/pull/149057 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][docs] Add section on testing with QEMU user (PR #149057)

2025-07-16 Thread David Spickett via lldb-commits
DavidSpickett wrote: Hopefully there are enough "here be dragons, fighting them is your problem" in the text. WASM runtimes is another class of simulator I was thinking of in future. I know we don't have support for that yet but if that one PR ever gets revived, it might be a thing. So I'm le

[Lldb-commits] [lldb] [lldb][docs] Add section on testing with QEMU user (PR #149057)

2025-07-16 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett created https://github.com/llvm/llvm-project/pull/149057 This is not recommended to basically anyone but on occasion it's useful and could be used for testing with other simulator programs for example bare metal simulators. Adding this is also an excuse to doc

[Lldb-commits] [lldb] [lldb] Convert registers values into target endian for expressions (PR #148836)

2025-07-15 Thread David Spickett via lldb-commits
@@ -1376,29 +1376,26 @@ class EntityRegister : public Materializer::Entity { return; } -DataExtractor register_data; - -if (!reg_value.GetData(register_data)) { - err = Status::FromErrorStringWithFormat( - "couldn't get the data for register %s"

[Lldb-commits] [lldb] [lldb] Convert registers values into target endian for expressions (PR #148836)

2025-07-15 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett created https://github.com/llvm/llvm-project/pull/148836 Relates to https://github.com/llvm/llvm-project/issues/135707 Where it was reported that reading the PC using "register read" had different results to an expression "$pc". This was happening because regi

[Lldb-commits] [lldb] [lldb] Improve setting of program for filtering disassembly (PR #148823)

2025-07-15 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett updated https://github.com/llvm/llvm-project/pull/148823 >From ba64d72e000bd14125b202a70bd7d34831e12cc0 Mon Sep 17 00:00:00 2001 From: David Spickett Date: Tue, 15 Jul 2025 10:30:21 + Subject: [PATCH] [lldb] Improve setting of program for filtering disassemb

[Lldb-commits] [lldb] a64bfd8 - [lldb] Fix Disasembler build error on 32-bit systems

2025-07-15 Thread David Spickett via lldb-commits
Author: David Spickett Date: 2025-07-15T09:54:58Z New Revision: a64bfd8a5186bc5b967878689509db81b9d1922c URL: https://github.com/llvm/llvm-project/commit/a64bfd8a5186bc5b967878689509db81b9d1922c DIFF: https://github.com/llvm/llvm-project/commit/a64bfd8a5186bc5b967878689509db81b9d1922c.diff LOG

[Lldb-commits] [lldb] [lldb] Document MCP tools & resources (PR #148708)

2025-07-15 Thread David Spickett via lldb-commits
@@ -75,7 +75,69 @@ Configuration example for [Visual Studio Code](https://code.visualstudio.com/doc } ``` -### Troubleshooting +## Tools + +Tools are a primitive in the Model Context Protocol that enable servers to +expose functionality to clients. + +LLDB's MCP integration e

[Lldb-commits] [lldb] [lldb] Document MCP tools & resources (PR #148708)

2025-07-15 Thread David Spickett via lldb-commits
@@ -75,7 +75,69 @@ Configuration example for [Visual Studio Code](https://code.visualstudio.com/doc } ``` -### Troubleshooting +## Tools + +Tools are a primitive in the Model Context Protocol that enable servers to +expose functionality to clients. + +LLDB's MCP integration e

[Lldb-commits] [lldb] [lldb] Document MCP tools & resources (PR #148708)

2025-07-15 Thread David Spickett via lldb-commits
@@ -75,7 +75,69 @@ Configuration example for [Visual Studio Code](https://code.visualstudio.com/doc } ``` -### Troubleshooting +## Tools + +Tools are a primitive in the Model Context Protocol that enable servers to +expose functionality to clients. + +LLDB's MCP integration e

[Lldb-commits] [lldb] [lldb] Update the String table offset based on the DWARF format (PR #147054)

2025-07-14 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett approved this pull request. LGTM. https://github.com/llvm/llvm-project/pull/147054 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [llvm] [lldb] Support disassembling RISC-V proprietary instructions (PR #145793)

2025-07-14 Thread David Spickett via lldb-commits
DavidSpickett wrote: Please fix the formatting before landing - https://github.com/llvm/llvm-project/actions/runs/16232166455/job/45836933279?pr=145793. https://github.com/llvm/llvm-project/pull/145793 ___ lldb-commits mailing list lldb-commits@lists.

[Lldb-commits] [lldb] [llvm] [lldb] Support disassembling RISC-V proprietary instructions (PR #145793)

2025-07-14 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/145793 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [llvm] [lldb] Support disassembling RISC-V proprietary instructions (PR #145793)

2025-07-14 Thread David Spickett via lldb-commits
@@ -90,3 +91,64 @@ TEST_F(TestMCDisasmInstanceRISCV, TestRISCV32Instruction) { EXPECT_FALSE(inst_sp->IsCall()); EXPECT_TRUE(inst_sp->DoesBranch()); } + +TEST_F(TestMCDisasmInstanceRISCV, TestOpcodeBytePrinter) { + ArchSpec arch("riscv32-*-linux"); + + const unsigned num_o

[Lldb-commits] [lldb] [llvm] [lldb] Support disassembling RISC-V proprietary instructions (PR #145793)

2025-07-14 Thread David Spickett via lldb-commits
@@ -304,6 +304,10 @@ Changes to LLDB stop reason = SIGSEGV: sent by tkill system call (sender pid=649752, uid=2667987) ``` * ELF Cores can now have their siginfo structures inspected using `thread siginfo`. +* Disassembly of unknown instructions now produces "" instead

[Lldb-commits] [lldb] [lldb][AArch64] Fix arm64 hardware breakpoint/watchpoint to arm32 process. (PR #147198)

2025-07-09 Thread David Spickett via lldb-commits
DavidSpickett wrote: > So the solution is copying the implementation in > NativeRegisterContextLinux_arm64.cpp. Yes, try putting the 64-bit code in a separate file in Process/Linux then using it in both places. Currently arm64 includes the arm header, for reasons I am not sure, so if there i

[Lldb-commits] [lldb] [lldb][AArch64] Fix arm64 hardware breakpoint/watchpoint to arm32 process. (PR #147198)

2025-07-09 Thread David Spickett via lldb-commits
DavidSpickett wrote: > the test By this you mean it skips any test that requires hardware breakpoints? Or you mean that in the general course of using lldb it doesn't use hardware breakpoints because of this? (I assume you could make the test suite launch all debugees as AArch32 but seems un

[Lldb-commits] [lldb] [llvm] [lldb] Support disassembling RISC-V proprietary instructions (PR #145793)

2025-07-09 Thread David Spickett via lldb-commits
https://github.com/DavidSpickett edited https://github.com/llvm/llvm-project/pull/145793 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [llvm] [lldb] Support disassembling RISC-V proprietary instructions (PR #145793)

2025-07-09 Thread David Spickett via lldb-commits
@@ -0,0 +1,28 @@ +# REQUIRES: x86 + +# This test verifies that disassemble -b prints out the correct bytes and +# format for x86_64 instructions of various sizes, and that an unknown +# instruction shows the opcode and disassembles as "" + +# RUN: llvm-mc -filetype=obj --triple=x8

[Lldb-commits] [lldb] [llvm] [lldb] Support disassembling RISC-V proprietary instructions (PR #145793)

2025-07-09 Thread David Spickett via lldb-commits
@@ -0,0 +1,30 @@ +# REQUIRES: riscv + +# This test verifies that disassemble -b prints out the correct bytes and +# format for standard and unknown riscv instructions of various sizes, +# and that unknown instructions show opcodes and disassemble as "". +# It also tests that the f

[Lldb-commits] [lldb] [llvm] [lldb] Support disassembling RISC-V proprietary instructions (PR #145793)

2025-07-09 Thread David Spickett via lldb-commits
@@ -0,0 +1,30 @@ +# REQUIRES: riscv + +# This test verifies that disassemble -b prints out the correct bytes and +# format for standard and unknown riscv instructions of various sizes, +# and that unknown instructions show opcodes and disassemble as "". +# It also tests that the f

  1   2   3   4   5   6   7   8   9   10   >