[Lldb-commits] [lldb] [LLDB] Handle i686 mingw32 mangling prefix (PR #160930)

[Martin Storsjö via lldb-commits]

mstorsjo wrote:

> For the time being, we could strip only the __cdecl prefix in PDB to match DWARF and open an issue for the mangling of other calling conventions.

Sounds reasonble. If we add good test coverage for these cases, we should be kinda free to adjust the exact implementation later anyway.

> A bit related: It seems like debugging x86 executables on x86_64 Windows (WOW64) doesn't work right now, because `STATUS_WX86_BREAKPOINT` is received there as opposed to `EXCEPTION_BREAKPOINT`. Windows also sends two events for the initial breakpoint (`EXCEPTION_BREAKPOINT` followed by `STATUS_WX86_BREAKPOINT`).

Yes, this matches my experience - the code is meant to work for x86 debugging with an x86_64 debugger, but in practice, it doesn't.

For cases like these, I presume you don't actually need to test live debugging though, I guess it should be enough with just tests that load an executable and the matching debug info and inspect it? And such tests should be able to run on any platform.

https://github.com/llvm/llvm-project/pull/160930