https://github.com/dzhidzhoev commented:
> Thanks for the review @dzhidzhoev ! You're right about the placeholder > situation. I've given fixing it a go - it seems to work, but it's not very > pretty Thank you for the fix! This looks like a good approach to me. I've taken a look at the DWARF changes in Clang ThinLTO build with this patch, and they all seem expected (just some references to forward declarations of types are replaced with references to full definitions, or vice versa, in subprograms' DW_AT_type attributes). There is a couple of cases that need to be addressed. https://github.com/llvm/llvm-project/pull/217042 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
