[PATCH] D76293: [WIP][DebugInfo] refactor DIE classes. Remove dependency on AsmPrinter.

Alexey Lapshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 18 08:42:19 PDT 2020


avl added a comment.

> Could you point to where this is a requirement? (has this already been discussed in review/found to be an unacceptable dependency?

Actually, I did not discuss it with LLD folks. 
It looked obvious from the LLD code structure.
But, that my assumption could be wrong. I`ve asked now in D74169 <https://reviews.llvm.org/D74169>.

So I will wait for feedback from them. If AsmPrinter is OK - then no need to do refactoring now.

> llvm-dsymutil uses the existing AsmPrinter-based APIs to write the output, yes?)

yes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76293





More information about the llvm-commits mailing list