[PATCH] D114225: Add JSONScopedPrinter to llvm-readelf

Fri Dec 10 02:31:59 PST 2021

jhenderson accepted this revision.
jhenderson added a comment.
This revision is now accepted and ready to land.

Looks good!

In D114225#3184814 <https://reviews.llvm.org/D114225#3184814>, @Jaysonyan wrote:

> Just as a heads up, Dec 10th is the last day of my internship so unfortunately after that I won't be able to work on this change as actively as before. If there is still feedback to address a member of my team may be able to pick this up from where I left off. And thank you for taking the time to review all of my code changes these past months!

No problem. I think this is now in a good state to land the first sequence of patches. Hopefully you can get that done today.

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

https://reviews.llvm.org/D114225