https://github.com/Michael137 approved this pull request.

Seems reasonable to me

Yea not entirely sure how to test this either. As Greg points out, type 
mangling isn't really a thing LLDB cared about for C++. Maybe if we had this 
option exposed somehow (e.g., through `lldb-test`) we could come up with a 
DWARF yaml that has a type with a linkage name in the accelerator tables? But 
if that's too much hassle and we have a test on the Apple fork that exercises 
this API maybe we're ok

https://github.com/llvm/llvm-project/pull/113007
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to