[Lldb-commits] [PATCH] D115662: [lldb][DWARF] Remove duplicate DIE type assignment

Tue Dec 14 10:06:02 PST 2021

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