jmorse wrote:

It sounds like there's agreement that the "before" approach was 
better/acceptable, i.e. having a CU-level variable that refers by 
`DW_AT_specification` to a variable in the class type. Doing so would also 
avoid the customisation for vtable-addresses in the latest patch with the 
`createGlobalVariableVTableDIE` method, which'd be neater. With that in mind, 
we'll head back in that direction.

It's also worth noting that this has spawned some DWARF issues such as 
https://dwarfstd.org/issues/250506.2.html , but I feel that's "future work".

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

Reply via email to