Michael137 wrote:

> As an alternative, we might be able to make the die-to-type map shared 
> between all symbol files in a debug map (similar to how "unique dwarf ast 
> type map" is shared).

Yea I considered that, but I was concerned with the unforeseen complications 
this might have. Particularly I'm not sure how the `TypeSP` ownership should be 
managed. IIUC, a `TypeSP` is solely supposed to be owned by a single 
`SymbolFile` (`MakeType` kind of binds the two together). But then if we 
bundled `TypeSP`s from different modules into a single map on 
`SymbolFileDWARFDebugMap`, could we run into issues if one of the 
`SymbolFile`'s gets torn down (if that can happen)? Though I'm also not certain 
that we *only* store `TypeSP`s inside the `SymbolFileDWARF`. I don't think 
there's anything asserting/enforcing this.

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

Reply via email to