[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