labath added a comment. In D115662#3192651 <https://reviews.llvm.org/D115662#3192651>, @ljmf00 wrote:
> In D115662#3192064 <https://reviews.llvm.org/D115662#3192064>, @labath wrote: > >> I don't have any issues with this per se, but you may want to sync up with >> @zequanwu, as his D115308 <https://reviews.llvm.org/D115308> tries to delete >> the second instance. > > Maybe he can reach out here, but how is this related to D115308 > <https://reviews.llvm.org/D115308> ? You're right -- it isn't as related as I originally thought -- you're deduplicating the dietotype map, and he's doing it for the type list. But it still is related at least on a conceptual level, as both patches deal with recording information about the type, and would be good if those things could happen in a central place (and you seem to have picked different centralization points). Maybe there is a reason it has to be that way (I honestly don't know) but it at least seems like to be aware of each other's efforts. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115662/new/ https://reviews.llvm.org/D115662 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits