Nerixyz wrote:

> dumps dynamic type info, but I'm not sure if it's actually used anywhere.

This seems to be unused. clangd says it's only used in 
[`DumpTypeDescription`](https://github.com/llvm/llvm-project/blob/9d7449a82b83ee589b8af8d6f86525727788b3b9/lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp#L8821-L8823).
 However, that's the overload that always prints to stdout. This (and the 
`CompilerType` method) are unused.

The output of `ClangASTMetadata::Dump` is also missing spaces after the UID and 
ISA ptr - so we'd get `uid=0x1234is_dynamic_cxx=2`. 

Maybe it's better if I include these changes when adding a Microsoft ABI 
runtime, since that would allow for easy testing.

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

Reply via email to