This is an archive of the discontinued LLVM Phabricator instance.

ReleaseNotes: add lld/ELF notes
ClosedPublic

Authored by MaskRay on Aug 1 2022, 10:13 PM.

Details

Summary

For the release/15.x branch.

Diff Detail

Event Timeline

MaskRay created this revision.Aug 1 2022, 10:13 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 1 2022, 10:13 PM
Herald added a subscriber: StephenFan. · View Herald Transcript
MaskRay requested review of this revision.Aug 1 2022, 10:13 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 1 2022, 10:13 PM
smeenai resigned from this revision.Aug 2 2022, 9:16 AM

LGTM, but James usually has good feedback on these, so I'm leaving the accept for him.

lld/docs/ReleaseNotes.rst
50

"performed in parallel", maybe?

smeenai added a subscriber: smeenai.Aug 2 2022, 9:16 AM

Can't spot anything obviously wrong. Thanks for writing up.

oontvoo added inline comments.
lld/docs/ReleaseNotes.rst
59

wrong link? Should be https://reviews.llvm.org/D119074?

62

wrong link here too?

MaskRay updated this revision to Diff 449701.Aug 3 2022, 10:24 AM
MaskRay marked 3 inline comments as done.

address comments
fix wrong links

This revision is now accepted and ready to land.Aug 3 2022, 10:26 AM
MaskRay closed this revision.Aug 3 2022, 10:26 AM

All looks good to me, thanks!