Michael137 wrote:

> That's the case, and also why just checking for __Z isn't enough. For one, 
> this doesn't play nice with Rust, which also has two underscores on i686. And 
> secondly, C names are displayed incorrectly right now. For example, you'd see 
> _main instead of main and a function like void Zone() would become _Zone and 
> interpreted as an itanium name.

I see. So just checking `__Z` in `GetManglingScheme` doesn't work because Rust 
and other mangling schemes suffer from this issue. Then yea, I think we can do 
the stripping in PDB.

I don't think we need to in DWARF though? At least I haven't seen the prefix be 
added into DWARF `DW_AT_linkage_name`s before.

https://github.com/llvm/llvm-project/pull/160930
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to