[PATCH] D86045: [llvm-dwarfdump] --statistics: switch to json::OStream. NFC

Eric Christopher via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 19 14:48:24 PDT 2020


echristo accepted this revision.
echristo added a comment.

I feel pretty OK approving this.

(Tradeoffs I'd contemplated and discarded was renaming OS rather than OS->J everywhere. I ended up liking the readability of J more. JS might be nice as well?)

-eric


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86045



More information about the llvm-commits mailing list