[PATCH] D71839: [Dsymutil][Debuginfo][NFC] Reland: Refactor dsymutil to separate DWARF optimizing part. #2.

Mon Dec 30 10:23:46 PST 2019

dblaikie added a subscriber: probinson.
dblaikie added a comment.

In D71839#1795483 <https://reviews.llvm.org/D71839#1795483>, @avl wrote:

> > Not sure if this is better/worse off as a subdirectory of lib/CodeGen, rather than as a top level library on its own (lib/DWARFOptimizer). Figured I'd mention it, but mostly leave it up to the other dsymutil folks to make the choices here.
>
> My idea was that DWARFOptimizer does generation task. F.e. AsmPrinter - generates asm and it is under CodeGen.  DWARFOptimizer generates optimized DWARF. That is why I put it under CodeGen.
>  Though if it is better to put it on top level - lib/DWARFOptimizer - I am OK to do this.

@aprantl @JDevlieghere @probinson any preferences here? If I've got to make the call I'll probably pick lib/DWARFOptimizer or lib/DWARFLinker (probably the latter)

(totally random aside: I Imagine we could have a mode where LLVM's debug info is aware it will be linked with a DWARF aware linker, so it doesn't need to use relocations in some cases (even with DWARFv5, the string relocations (in str_offsets) are still pretty significant & would be nice to skip (as split DWARF manages to skip))

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71839/new/

https://reviews.llvm.org/D71839