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

Alexey Lapshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 24 03:01:48 PST 2019


avl added a comment.

> 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.


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

https://reviews.llvm.org/D71839





More information about the llvm-commits mailing list