This is an archive of the discontinued LLVM Phabricator instance.

ReleaseNotes: add notes for binary utilities
ClosedPublic

Authored by MaskRay on Feb 11 2022, 6:52 PM.

Details

Summary

For the release/14.x branch.

Diff Detail

Event Timeline

MaskRay requested review of this revision.Feb 11 2022, 6:52 PM
MaskRay created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptFeb 11 2022, 6:52 PM
jhenderson accepted this revision.Feb 14 2022, 12:38 AM

One nit, otherwise looks good (NB: I haven't double-checked that there aren't more things worth release noting (@gbreynoo and others, it might be worth you taking a look through too to see if there's anything else that should be commented on).

llvm/docs/ReleaseNotes.rst
219

Otherwise, it sounds like --pretty-print is for non-JSON output formats too.

This revision is now accepted and ready to land.Feb 14 2022, 12:38 AM
MaskRay updated this revision to Diff 408529.Feb 14 2022, 11:42 AM

Address comment. Add some links.

MaskRay marked an inline comment as done.Feb 14 2022, 11:44 AM
MaskRay edited the summary of this revision. (Show Details)

Thanks for review. I'll push this tomorrow if there is no pushback. We can make incremental changes if needed.

jhenderson accepted this revision.Feb 15 2022, 12:14 AM

Latest looks good too.