[PATCH] D111658: Add JSON output skeleton to llvm-readelf

[Jayson Yan via Phabricator via llvm-commits]

Jaysonyan added a comment.

@jhenderson I was wondering if you had any more thoughts on the design on `ScopedPrinter`/ `ScopedPrinterBase`/`JSONScopedPrinter`? As I see it right now, it's not very feasible to treat `ScopedPrinter` and `JSONScopedPrinter` uniformly due to the usage of methods like `startLine()`. So a parent-child structure between `ScopedPrinter` and `JSONScopedPrinter` might not be suitable.


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

https://reviews.llvm.org/D111658